https://bugzilla.wikimedia.org/show_bug.cgi?id=71927

            Bug ID: 71927
           Summary: shouldSkip and transition do not properly execute on
                    next/prev
           Product: MediaWiki extensions
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Keywords: code-update-regression
          Severity: normal
          Priority: Unprioritized
         Component: GuidedTour
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected],
                    [email protected], [email protected],
                    [email protected], [email protected]
       Web browser: ---
   Mobile Platform: ---

shouldSkip used to execute correctly when you clicked next.  It would first get
the ID of the next guider, then see if skipping was needed.  See
https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FGuidedTour.git/40e856360e4ba377242c172b036477aebf740ad9/modules%2Fmediawiki.libs.guiders%2Fmediawiki.libs.guiders.js#L774
.

The same should happen with transition, but now it works for neither legacy
shouldSkip nor AFAICT transition.

For an example of the regression on legacy, try going to
https://en.wikipedia.org/wiki/Wikipedia:TWA/1/Start?tour=twa1&step=6 then click
next.

There is a shouldSkip on step 7 (the one you hit when you click next):

shouldSkip: function () {
                return mw.config.get( 'wgUserId' )  !== null;
}

Thus, it should automatically skip step 7, since you're logged in.  However,
this doesn't work.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to