Closed (fixed)
Project:
One Click Upload
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Jul 2015 at 21:36 UTC
Updated:
9 Oct 2015 at 11:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
gbyteApparently the signature field stores its value differently to regular fields, this might be due to it not being translatable. Fixed it by adding a simple isset check - see attached patch.
Comment #2
gbyteNewest dev still has the issue and this patch still applies. I know it is not that common of a scenario to upload files into a signature field, but could someone just test this scenario and verfiry the bug / apply patch to change this issue to RTBC? White screen of death is not acceptable, hence changing to major.
Comment #3
xandeadx commentedIt is necessary to find out why there is a mistake, and not to hide it
Comment #4
gbyteI already found out mate, the signature field is no standard drupal text field and stored its value differently in $form_state['values']. Why would you assume this patch hides anything? Ocupload wokrs perfectly with the signature field after applying the patch.
Comment #5
xandeadx commentedNeed to find prime cause
Comment #6
gbyteNot sure what you mean, prime cause seems to me core signature functionality which by design stores its value differently. Please correct me if I'm mistaken.
Comment #7
xandeadx commentedHow to reproduce error?
Comment #8
gbyteError happens in _ocupload_field_value_by_html_name() which is called in ocupload_change_files_status() which is a submission handler called when submitting user account form.
Comment #10
xandeadx commentedfixed