Opened 14 years ago

Closed 13 years ago

#535 closed defect (fixed)

New folder does not work in ImageManager

Reported by: ray Owned by: yermol
Priority: highest Milestone: Version 1.0
Component: Plugin_ImageManager Version: trunk
Severity: critical Keywords: new folder
Cc:

Description

You can't create a new folder in the ImageManager?, because the required popup-functions are not loaded from ImageManagers? own popup.js since [256].

The general popup-functions have to be used, but are not included in

/plugins/ImageManager/newFolder.html

so we have to add

<script type="text/javascript" src="../../popups/popup.js"></script>

see: http://xinha.gogo.co.nz/punbb/viewtopic.php?pid=2523

Attachments (1)

popup_imagemanager.patch (19.2 KB) - added by ray 14 years ago.
here's a patch for newFolder.html as well as editor.php

Download all attachments as: .zip

Change History (9)

comment:1 Changed 14 years ago by ray

  • Keywords new folder added
  • Milestone set to Version 1.0
  • Severity changed from normal to major
  • Version set to trunk

comment:2 Changed 14 years ago by niko

  • Priority changed from normal to highest

...this happens with the editor too!

Changed 14 years ago by ray

here's a patch for newFolder.html as well as editor.php

comment:3 Changed 14 years ago by anonymous

  • Severity changed from major to critical

comment:4 Changed 14 years ago by gocher

popup_imagemanager.patch :

Index: editor.php
===================================================================
--- editor.php	(revision 431)
+++ editor.php	(working copy)
@@ -25,6 +25,7 @@
 	<link href="<?php print $IMConfig['base_url'];?>assets/editor.css" rel="stylesheet" type="text/css" />	
 <script type="text/javascript" src="<?php print $IMConfig['base_url'];?>assets/slider.js"></script>
 <script type="text/javascript" src="<?php print $IMConfig['base_url'];?>assets/popup.js"></script>
+<script type="text/javascript" src="../../popups/popup.js"></script>
 <script type="text/javascript">
 /*<![CDATA[*/
 	window.resizeTo(673, 531);
@@ -145,4 +146,4 @@
 </div>
 <div id="bottom"></div>
 </body>
-</html>
\ No newline at end of file
+</html>
Index: newFolder.html
===================================================================
--- newFolder.html	(revision 431)
+++ newFolder.html	(working copy)
@@ -16,6 +16,7 @@
  /*]]>*/
  </style>
 <script type="text/javascript" src="assets/popup.js"></script>
+<script type="text/javascript" src="../../popups/popup.js"></script>
 <script type="text/javascript">
 /*<![CDATA[*/
 	window.resizeTo(300, 160);

comment:5 Changed 14 years ago by Wolfgang

I can confirm that this patch works.
Pls commit into Mainstream.

comment:6 Changed 13 years ago by mvanbaak

I can confirm too that this patch works.

comment:7 Changed 13 years ago by CrustyDOD

Patch works.

comment:8 Changed 13 years ago by gocher

  • Resolution set to fixed
  • Status changed from new to closed

look at Ticket #724
missing JS in ImageManager? plugin

fixed in Changeset 485

Note: See TracTickets for help on using tickets.