Configuration form not updating the changes

CommentFileSizeAuthor
#10 3015579-10-tablesorter-settings-form.patch1.19 KBdouggreen
#7 Screenshot from 2018-12-01 22-19-53.png40.57 KBmahipal46
#3 tablesorter-config-issue-1881938-1.patch779 bytesmadhan.718@gmail.com

Comments

madhan.718@gmail.com created an issue. See original summary.

madhan.718@gmail.com’s picture

madhan.718@gmail.com commented
Issue summary: View changes
madhan.718@gmail.com’s picture

madhan.718@gmail.com commented
StatusFileSize
new tablesorter-config-issue-1881938-1.patch779 bytes
madhan.718@gmail.com’s picture

madhan.718@gmail.com commented
Title: Tablesorter Drupal-8 porting issue » Configuration form is having Wrong Config variable
madhan.718@gmail.com’s picture

madhan.718@gmail.com commented
Status: Active » Needs review
mahipal46’s picture

mahipal46 commented
Assigned: Unassigned » mahipal46
mahipal46’s picture

mahipal46 commented
Assigned: mahipal46 » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2018-12-01 22-19-53.png40.57 KB

This patch works fine. Configuration form changes are updating properly.

monymirza’s picture

monymirza
Primary language Urdu
Location Islamabad
commented
Status: Reviewed & tested by the community » Closed (duplicate)

#3015479

douggreen’s picture

douggreen commented
Status: Closed (duplicate) » Active
Issue tags: -

I think this was closed by mistake, it is not a duplicate of #3015479: Js and css files not included for sorting the tables, re-opening.

douggreen’s picture

douggreen commented
StatusFileSize
new 3015579-10-tablesorter-settings-form.patch1.19 KB

New attached patch uses getValues() instead of getUserInput() to ensure that the values are sanitized.

douggreen’s picture

douggreen commented
Title: Configuration form is having Wrong Config variable » Configuration form doesn't save
douggreen’s picture

douggreen commented
Version: 8.x-1.x-dev » 2.0.0-beta1
Status: Active » Needs review
monymirza’s picture

monymirza
Primary language Urdu
Location Islamabad
commented
Version: 2.0.0-beta1 » 8.x-1.x-dev
Status: Needs review » Closed (outdated)