Closed (won't fix)
Project:
Ubercart AJAX Cart
Version:
6.x-2.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2011 at 17:30 UTC
Updated:
30 May 2012 at 23:16 UTC
Module administration page is not using the standard Drupal function system_settings_form. It should be used because is the santard Drupal way and makes easier to add new configurations to the module.
Comments
Comment #1
stewart.adam commentedThis is true, however some of the newer additions to the settings page require preprocessing so it's easiest to simply not change this and perform the preprocessing of form values in the submit handler before we do variable_set(). Otherwise we'd have to inject a submit handler for the particular form elements that require preprocessing via the form's #submit index.