Page MenuHomePhabricator

Enable data value normalization by default (remove tmpNormalizeDataValues repo setting)
Closed, ResolvedPublic

Description

As a third-party Wikibase admin, I want data values to be normalized on my wiki like they are on Wikidata.
As a Wikibase developer, I want tests to pass on my wiki without having to configure too many settings.

Problem:
In T251480, we implemented normalization of data values, but hid it behind a feature flag (tmpNormalizeDataValues). Now that we’ve rolled this out to all production wikis, we should remove that feature flag and unconditionally enable the feature. This is also relevant to developers, because we have an API integration test for the feature which assumes that the feature is enabled, and fails in a default Wikibase install.

Example:

Screenshots/mockups:

BDD
GIVEN
AND
WHEN
AND
THEN
AND

Acceptance criteria:

  • the tmpNormalizeDataValues repo setting is gone
  • data value normalization is enabled everywhere

Open questions:

Event Timeline

We probably want to backport this to the REL1_37 branch too.

Change 761903 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Enable data value normalization by default

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

Change 761903 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Enable data value normalization by default

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

(This is blocked on nothing other than some test failures that I haven’t had the time to look into yet.)

(This is blocked on nothing other than some test failures that I haven’t had the time to look into yet.)

(Fixed now.)

CCing @Evelien_WMDE – I’m not sure if this should be prioritized from the product or tech side, but if it’s product then it probably belongs with Wikibase more than Wikidata. (Also CC @ItamarWMDE for the tech side.)

CCing @jon_amar-WMDE so they could decide if this is a product priority (seems more like suite than cloud to me).

I just rebased the change again, it now adds the change to the release notes for 1.42 (since the branch for REL1_41 was cut the other day). Recall that when this task was originally filed, I had still hoped to get this into the 1.37 release (T292181#7391022), which in the meantime has gone through its entire lifecycle and is now EOL.

I don’t know what else I can do here. I don’t know why it’s seemingly impossible for us to remove this temporary feature flag. I don’t know how many more people need to be pinged to reach a decision about this. I don’t know why almost every third-party Wikibase for the last two years has still had data value normalization disabled when we haven’t had any problems with it being enabled on Wikidata.

Hey @Lucas_Werkmeister_WMDE, this issue is just now getting moved into the Suite teams backlog. I'm gathering that your patch has fallen "between the cracks" and meanwhile an important default remains wrong for now a few releases WikiBase.

On the dev side of the Suite team we have not yet submitted any patches via Gerrit to the Wikibase core, but do obviously expect to at some point. The attention hasn't been there yet. I think this may be a great first candidate for getting in there, which I am hopeful will be soon.

Change #761903 merged by Loren Johnson:

[mediawiki/extensions/Wikibase@master] Enable data value normalization by default

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