Simple patch that will expand the "File Attachments" form_group_collapsible() by default if there are files attached.

CommentFileSizeAuthor
#2 upload.module_2.patch1.19 KBeafarris
upload.module_1.patch1.21 KBeafarris
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Needs work

I think

$has_attachments ? FALSE : TRUE

can be replaced with

!$has_attachments
eafarris’s picture

eafarris CreditAttribution: eafarris commented
Status: Needs work » Needs review
FileSize
upload.module_2.patch1.19 KB

Agreed. Here's an updated patch.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
Status: Needs review » Reviewed & tested by the community

Tested, works, please apply. The current behaviour is highly annoying.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed a better solution (I hope) to HEAD.

Anonymous’s picture

(not verified) CreditAttribution: commented
Anonymous’s picture

(not verified) CreditAttribution: commented
Anonymous’s picture

(not verified) CreditAttribution: commented
Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)