Currently the setting form submit handler uses the $form element to get the form values. This should make use of the $firm_state object to get the value really

CommentFileSizeAuthor
#3 settings_form_doesn_t-2825173-3.patch2.02 KBmike.davis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mike.davis created an issue. See original summary.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis at Deeson commented
Title: Settings for doesn't use $form_state to get values » Settings form doesn't use $form_state to get values
mike.davis’s picture

mike.davis CreditAttribution: mike.davis at Deeson commented
Status: Active » Needs review
FileSize
settings_form_doesn_t-2825173-3.patch2.02 KB

Here is a patch which updates the form submit hander to use $form_state object.

  • mike.davis committed 707a705 on 8.x-3.x 
    Issue #2825173 by mike.davis: Settings form doesn't use $form_state to...
mike.davis’s picture

mike.davis CreditAttribution: mike.davis at Deeson commented
Status: Needs review » Fixed

This has been committed now

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.