Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
FileSize
7.68 KB

Here's a port of the D7 patch!

(FYI, the class_exists('DOMDocument') is because D6, unlike D7, doesn't require the PHP dom extension)

dsnopek’s picture

Status: Needs review » Fixed

Committed to the repo!

catch’s picture

Status: Fixed » Needs review
FileSize
2.9 KB
7.54 KB

The backport broke file uploads (silently) - the form submits OK but the file never makes it. Just an if statement that shouldn't be there, here's an interdiff and updated patch.

dsnopek’s picture

@catch Hrm. File uploads are working for me! I just retested it. Not sure why we'd see different behavior?

catch’s picture

Did you test only uploading a file and nothing else?

dsnopek’s picture

Aha! Ok, that did it. :-) Your patch fixes it. Thanks!

dsnopek’s picture

Status: Needs review » Reviewed & tested by the community
dsnopek’s picture

Status: Reviewed & tested by the community » Fixed

Committed!

Status: Fixed » Closed (fixed)

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