Missing temp mode indicator in "form_submissions_config.php"

4 views
Skip to first unread message

Alexander Obuhovich

unread,
Dec 7, 2010, 2:35:42 PM12/7/10
to In-Portal Bugs
We have "QueryString" option in almost every unit. It consists of several elements:

  1. id
  2. Page
  3. PerPage
  4. event
  5. mode

All but "mode" parameters are specified in "form_submissions_config.php" file. Also adding "mode" parameter via "OnAfterConfigRead" event doesn't work (strange why).

Because of that any attempt to edit form submissions in temp table will result in data being edited in live table.


We need to fix both issues.

Dmitry A.

unread,
Dec 10, 2010, 1:02:10 PM12/10/10
to in-port...@googlegroups.com
Hello,


Here is a task for a 1st bug:

946: Missing Temp mode indicator in "form_submissions_config.php"



Alex, what about AfterConfigRead issue - I don't see it used anywhere in the code. Did you add custom code, if so did you figure out more here or we still need to look into this?

Cheers


DA

Dmitry A.

unread,
Dec 10, 2010, 1:18:26 PM12/10/10
to in-port...@googlegroups.com

Patch attached. To test please make sure you correctly change:

function edit()
{
// don't use temp tables, since we can receive user replies while reviewing form submission
std_edit_temp_item('formsubs', 'submissions/submission_view');
}

in submission_list.tpl so system uses TEMP mode.


DA
Fixes-minor-bug-with-editing-Form-Submissions.patch
Reply all
Reply to author
Forward
0 new messages