Ticket #535 (closed defect: fixed)

Opened 8 years ago

Last modified 7 years ago

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

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

Change History

Changed 8 years ago by ray

  • keywords new folder added
  • version set to trunk
  • severity changed from normal to major
  • milestone set to Version 1.0

Changed 8 years ago by niko

  • priority changed from normal to highest

...this happens with the editor too!

Changed 8 years ago by ray

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

Changed 7 years ago by anonymous

  • severity changed from major to critical

Changed 7 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);

Changed 7 years ago by Wolfgang

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

Changed 7 years ago by mvanbaak

I can confirm too that this patch works.

Changed 7 years ago by CrustyDOD

Patch works.

Changed 7 years ago by gocher

  • status changed from new to closed
  • resolution set to fixed

look at Ticket #724 missing JS in ImageManager plugin

fixed in Changeset 485

Note: See TracTickets for help on using tickets.