Opened 15 years ago

Closed 15 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


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


so we have to add

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


Attachments (1)

popup_imagemanager.patch (19.2 KB) - added by ray 15 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 15 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 15 years ago by niko

  • Priority changed from normal to highest

...this happens with the editor too!

Changed 15 years ago by ray

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

comment:3 Changed 15 years ago by anonymous

  • Severity changed from major to critical

comment:4 Changed 15 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">
 	window.resizeTo(673, 531);
@@ -145,4 +146,4 @@
 <div id="bottom"></div>
\ No newline at end of file
Index: newFolder.html
--- newFolder.html	(revision 431)
+++ newFolder.html	(working copy)
@@ -16,6 +16,7 @@
 <script type="text/javascript" src="assets/popup.js"></script>
+<script type="text/javascript" src="../../popups/popup.js"></script>
 <script type="text/javascript">
 	window.resizeTo(300, 160);

comment:5 Changed 15 years ago by Wolfgang

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

comment:6 Changed 15 years ago by mvanbaak

I can confirm too that this patch works.

comment:7 Changed 15 years ago by CrustyDOD

Patch works.

comment:8 Changed 15 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.