Attachment list not displayed (patch)

Anselm Heaton - February 19, 2009 - 17:24
Project:Upload previews
Version:6.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hello,

In the Drupal 6 version, the attachment list is not displayed when viewing the node. That's because 'upload_preview_attachments' is not declared as a theme, and thus it is never called.

The attached patch fixes this by declaring 'upload_preview_attachments' along with the other themes in 'upload_preview_theme'.

Thanks.

#1

rmiddle - February 19, 2009 - 17:30

It looks like drupal.org is still having some issues. The attachment didn't make it.

Thanks
Robert

#2

Anselm Heaton - February 19, 2009 - 17:42

Oh yes, well I'll just include the patch here, it's fairly short :

Index: upload_preview.module
===================================================================
--- upload_preview.module (revision 1618)
+++ upload_preview.module (working copy)
@@ -341,7 +341,11 @@
     'upload_preview_image' => array(
       'arguments' => array('element' => NULL),
       /* 'file' => 'upload_preview.module' */
-    )
+    ),
+    'upload_preview_attachments' => array(
+      'arguments' => array('element' => NULL),
+      /* 'file' => 'upload_preview.module' */
+    ),
     );
}

#3

rmiddle - February 19, 2009 - 20:48
Status:needs review» fixed

Commited to CVS.

Thanks
Robert

#4

System Message - March 5, 2009 - 20:50
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.