Comments on: Drupal: theme override for Upload.module's attachments list

By: damon

damon — Mon, 17 Aug 2009 20:50:02 +0000

I used the iTweak Upload (http://drupal.org/project/itweak_upload) module and it works great!

By: s

Thu, 25 Jun 2009 20:44:43 +0000

fYI if you are using “Garland” theme and have specified custom colors using the color picker you must physically delete the cached Garland theme at sites/default/files/color/garland-xxxxx

Then you must configure and save your theme to reset this cache. Note that garland “color picker” doesn’t work if your file method is Private.

If you are using private file method and Garland and have lots of files I do not suggest you attempt to use this file icon hack at all or you risk changing file paths for all your attachments.

By: Evgeny Pavlov

Evgeny Pavlov — Sat, 13 Jun 2009 17:32:26 +0000

If you have an issue with Search Module, you can move functions (as mentioned qrta) into template.php.

By: Andrew

Andrew — Wed, 27 May 2009 22:52:20 +0000

I love it. Thanks.

By: jamie

jamie — Sat, 25 Apr 2009 15:37:29 +0000

In additional change for this code to work with Drupal 5, the l() function which reads:

$file_link = ”. l(shiny_upload_file_icon($file) . $text, $file_url, array(‘html’=>’true’)) . ”;

Should be changed to:

$file_link = ”. l(shiny_upload_file_icon($file) . $text, $file_url, array(), null, null, false, true) . ”;

By: qrta

qrta — Sat, 10 Jan 2009 23:45:59 +0000

I have solved the problem:
you have to move the function declarations into one other file, good place is for example into one custom module. The upload_attachments.tpl.php only contains the print and drupal_add_css invocations.

By: MJ

MJ — Thu, 01 Jan 1970 00:00:00 +0000

*oops* my use of code tags didn’t work, so ignore the ‘it should read’ bit.

By: MJ

MJ — Thu, 01 Jan 1970 00:00:00 +0000

This is a great bit of code. In trying to control the look of the filename, I noted that there's a missing closing span tag on line 23 of upload_attachments.tpl.php, to close the class=file-name block. It should read:


$file_metadata = '

' . $file->filename . ' (' . format_size($file->filesize) .

Many thanks for this.

By: Jooblie

Jooblie — Thu, 01 Jan 1970 00:00:00 +0000

Whoa, this is slick.

Any way this can be made to work with 5.x? Any pointers for me to backport it?

Thanks for this!

By: Anonymous

Anonymous — Thu, 01 Jan 1970 00:00:00 +0000

This is a great bit of code. But really didn`t make any difference. Thanks anyway