When uploading or deleting a file at /admin/store/products/files, this error is displayed:

warning: Missing argument 3 for uc_file_admin_files_form_action_validate(), called in /[...]/includes/form.inc on line 775 and defined in /[...]/sites/all/modules/ubercart/uc_file/uc_file.admin.inc on line 337.

The file does get uploaded to the server. So that's good. :) The only problem is that this warning shows up.

Comments

Island Usurper’s picture

Island Usurper commented
Status: Active » Fixed

I may be going crazy. I thought for sure that validate and submit handlers took the $form_id as a third argument. I noticed that uc_file_admin_files_form_action_validate() didn't have one, so I thought I'd fix it in passing. I even looked at another function to be sure I was doing it right, so now I wonder if I have to go find and change it. I'm surprised I haven't seen this error more often.

Anyway, I've fixed this function, and committed it to Bazaar.

Status: Fixed » Closed (fixed)

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

andreib’s picture

andreib commented
Version: 6.x-2.0-rc3 » 6.x-2.x-dev
Status: Closed (fixed) » Needs work

The warning no longer appears but the file upload still fails: "An error occurred while uploading the file" with no other details.

jesse4216’s picture

jesse4216 commented

I think this may still be an issue, I am getting the same error as well though the file seems to be added .

http://www.ubercart.org/issue/11003/file_downloads_and_php_error_after_u...

I get the same error with ubercart installed 6.x-2.0-rc3 using druapl 6.12.

Thanks for the help.
J

jesse4216’s picture

jesse4216 commented
Version: 6.x-2.x-dev » 6.x-2.0-rc3
cha0s’s picture

cha0s commented
Status: Needs work » Fixed

It's been fixed in the code repository, it just wasn't in by the time rc3 came out.

Status: Fixed » Closed (fixed)

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