Last modified: 2013-07-25 07:07:58 UTC
Before Mar 4, making a "null edit" (opening the editor and saving the page but making no changes) in the course of a tour like http://test2.wikipedia.org/wiki/Bottle?tour=gettingstarted by design did not show the "Looking for more to do" prompt. As of Mar 5, making a "null edit" in the course of the tour produces the "Looking for more to do" prompt. If this change in behavior is intended, please close the bug to let me know. It is causing an automated browser test to fail in all browsers.
This is probably due to a false positive bug in our core implementation of the wgPostEdit boolean.
Hmm. Yeah on English Wikipedia just yesterday I got the correct behavior (no final tour step if edit is null). We'll be disabling this particular tour tomorrow, so we will have time to figure out the interaction between the core implementation of wgPostEdit and the tour at our leisure later. Thanks for reporting Chris.
The PostEdit extension did not set wgPostEdit true for null edits. I've submitted a Gerrit change to core, awaiting review: https://gerrit.wikimedia.org/r/#/c/52459/ I considered changing the behavior of didSave too, but didn't for backwards compatibility.
Merged. The first WMF deployment branch with this fix will be 1.21wmf12.
Enabling the Selenium test: https://gerrit.wikimedia.org/r/#/c/56143/ Jenkins job passes: https://wmf.ci.cloudbees.com/job/_debug-browsertests-template/89/testReport/(root)/Guided%20Tour/ Screen shots and video of the test run: https://saucelabs.com/tests/278eff1eb33a4a58b26c9bb3d13065f9
