This prevents it from showing up over at api.drupal.org: http://api.drupal.org/api/function/file_check_upload/5

THANK GOODNESS, this ugly, horrible function is not in Drupal 6 and 7, so it only needs to be fixed in 5.x.

CommentFileSizeAuthor
#6 drupal5.file_check_upload_.patch1.41 KBsun
#5 drupal5.file_check_upload.patch1.41 KBsun
#2 drupal5.file_check_upload.patch1.5 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Needs review » Active

Actually, I'll set this to active so someone new can roll a patch. :)

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Active » Needs review
FileSize
drupal5.file_check_upload.patch1.5 KB
drewish’s picture

drewish CreditAttribution: drewish commented

boy those @todo's could probably be dropped too since it's been addressed in D6.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

We need to be careful.. this might break the time/space continiuum :)

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community
FileSize
drupal5.file_check_upload.patch1.41 KB

Boy, you are driving me nuts -- this is an outdated version of Drupal core ;)

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
FileSize
drupal5.file_check_upload_.patch1.41 KB

So here's the tricky part: You should have mentioned the malformed indentation in the introduced PHPDoc :-D

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Sheesh. Well that's even better than I was hoping for. :)

Agree with RTBC. Thanks, sun!

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

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

Issue tags: +Quick fix

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.