Big wall of red when you upload a file attachments:



    * Notice: Undefined index: process_input in _form_builder_handle_input_element() (line 1088 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: process_input in _form_builder_handle_input_element() (line 1088 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: process_input in _form_builder_handle_input_element() (line 1088 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: process_input in _form_builder_handle_input_element() (line 1088 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: process_input in _form_builder_handle_input_element() (line 1088 of /home/catch/www/7/includes/form.inc).
    * Notice: Undefined index: complete form in form_builder() (line 970 of /home/catch/www/7/includes/form.inc).

CommentFileSizeAuthor
#7 530904_3.patch1.4 KBmgifford
#3 530904_2.patch1.47 KBmgifford
#1 530904.patch1.54 KBkjy07

Comments

kjy07’s picture

Assigned: Unassigned » kjy07
Priority: Critical » Minor
Status: Active » Needs review
StatusFileSize
new1.54 KB

Relatively minor as uploads still proceed without error and normal error reporting usually suppresses notices.

In any event, the attached patch should remedy the problem.

Status: Needs review » Needs work

The last submitted patch failed testing.

mgifford’s picture

StatusFileSize
new1.47 KB

I updated the patch, but it worked fine for me.

mgifford’s picture

Status: Needs work » Needs review

got rid of the notices that is.

Status: Needs review » Needs work

The last submitted patch failed testing.

mgifford’s picture

Interesting.. It does apply against head. But I get "Error message
Validation error, please try again. If this error persists, please contact the site administrator."

Every time on every form. Makes it less than useful for sure.

mgifford’s picture

StatusFileSize
new1.4 KB

Ok.. Forgot the previous very busted patch and have one hitting the notices more directly.

mgifford’s picture

Status: Needs work » Needs review

arg. forgot to set it to needs review

webchick’s picture

Status: Needs review » Closed (won't fix)

Don't need this anymore.