Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#40729 closed defect (bug) (fixed)

Customize: Tallying of settings modified during save starts before processing completes

Reported by: westonruter's profile westonruter Owned by: westonruter's profile westonruter
Milestone: 4.8 Priority: normal
Severity: normal Version: 4.6
Component: Customize Keywords: has-patch
Focuses: Cc:

Description

In #32941 logic was introduced to fix a situation where someone modifies a setting while saving is undergoing, so that once the save is completed the dirty state is restored. The logic however failed to take into account the processing state which is used for settings which require server processing before a save can be initiated. Widgets, for example, currently must undergo a update-widget request to obtain the instance data for the setting. The result is that currently if you make a change to a widget's title and then click Save & Publish while the update-widget request is being made, after the subsequent customize-save request completes the modifiedWhileSaving object will have the setting ID for that widget inside of it, and the resulting saved state will be set to false even though wp.customize.dirtyValues() is in fact empty.

Attachments (1)

40729.0.diff (734 bytes) - added by westonruter 7 years ago.

Download all attachments as: .zip

Change History (3)

@westonruter
7 years ago

#1 @westonruter
7 years ago

  • Keywords has-patch added

#2 @westonruter
7 years ago

  • Owner set to westonruter
  • Resolution set to fixed
  • Status changed from new to closed

In 40627:

Customize: Wait for processing state to clear before starting to captureSettingModifiedDuringSave.

See #32941.
Fixes #40729.

Note: See TracTickets for help on using tickets.