The settings form is not (correctly) saving values do to a few issues with the system settings from:

  • The $form['swfobject_api_settings'] fieldset is causing the swfoa_version and swfoa_express variables to be stored as an array of values named swfobject_api_settings
  • The default value for the swfoa_version field is being set to swfoa_express.
  • An extra submit button is being added to the form due to the inclusion of an unneeded $form['submit'] definiton in the form.

This patch fixes all of these issues with the form.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcmarco’s picture

The same patch ported to 5.x-2.0-beta2

jcmarco’s picture

Version: 5.x-1.x-dev » 5.x-2.0-beta2
seutje’s picture

2 months and still not rolled into a new beta?

anyway, cheers for the patch

<3

arthurf’s picture

Fixes are committed to CVS, thanks for the patches!

arthurf’s picture

Status: Needs review » Closed (fixed)