Page MenuHomePhabricator

Post-edit dialog is not shown for subsequent edits when topic hasn't been selected
Closed, ResolvedPublicBUG REPORT

Description

List of steps to reproduce (step by step, including full links if applicable):

  • If the account has already selected topics, clear the selection via:
new mw.Api().saveOptions({
    'growthexperiments-homepage-suggestededits-activated': '',
    'growthexperiments-homepage-se-ores-topic-filters' : '',
    'growthexperiments-homepage-se-filters': ''
})
  • Select an unstructured task without selecting any topics
  • Publish an edit
  • Close the post-edit dialog
  • Publish another edit

What happens?:

The post-edit dialog is not shown for the subsequent edit. This warning is present in the browser console.

load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets|oojs-ui.styles.icons-editing-advanced&skin=vector&version=e0r8n:52 jQuery.Deferred exception: Cannot read properties of null (reading 'getTopics') TypeError: Cannot read properties of null (reading 'getTopics')
    at setup (<anonymous>:7:707)
    at <anonymous>:8:597
    at mightThrow (http://localhost:8080/w/load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets%7Coojs-ui.styles.icons-editing-advanced&skin=vector&version=e0r8n:49:881)
    at process (http://localhost:8080/w/load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets%7Coojs-ui.styles.icons-editing-advanced&skin=vector&version=e0r8n:50:544) undefined

What should have happened instead?:
The post-edit dialog should be shown whenever an edit is published (for unstructured tasks only).

Software version (if not a Wikimedia wiki), browser information, screenshots, other information, etc.:
When the user is in this state, it looks like the promise for closing the post-edit dialog is never resolved so postEditDialogIsOpen is still true in suggestedEditSession so the post-edit dialog is not shown for the subsequent edit.

Event Timeline

Change 775293 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@master] Post-edit dialog: check for presence of preferences.topicFilters

https://gerrit.wikimedia.org/r/775293

Change 775293 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Post-edit dialog: check for presence of preferences.topicFilters

https://gerrit.wikimedia.org/r/775293

Change 775370 had a related patch set uploaded (by Sergio Gimeno; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@wmf/1.39.0-wmf.5] Post-edit dialog: check for presence of preferences.topicFilters

https://gerrit.wikimedia.org/r/775370

I'm not sure about the adjustment. I remember annoyingly breaking the "blue dot" functionality when not preserving the null value for the user preference. Did you test that scenario?

I'm not sure about the adjustment. I remember annoyingly breaking the "blue dot" functionality when not preserving the null value for the user preference. Did you test that scenario?

You're right, we do need it. I reverted that part of the patch.

Change 775370 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@wmf/1.39.0-wmf.5] Post-edit dialog: check for presence of preferences.topicFilters

https://gerrit.wikimedia.org/r/775370

Mentioned in SAL (#wikimedia-operations) [2022-03-31T08:19:30Z] <daniel@deploy1002> Synchronized php-1.39.0-wmf.5/extensions/GrowthExperiments/modules/ext.growthExperiments.PostEdit/index.js: Backport: [[gerrit:775370|Post-edit dialog: check for presence of preferences.topicFilters (T305057)]] (duration: 00m 53s)

Etonkovidova subscribed.

Checked in testwiki wmf.5 - all works as expected (the blue dot is present too). Also checked the structured task workflow - no issues were found.