Page MenuHomePhabricator

Accepting lowercase Sense IDs as values in statements
Closed, ResolvedPublic

Description

Problem:
When linking to the Sense the letters in the Sense ID have to be capitalized to be accepted as values. They should also be accepted in lowercase for convenience. This works fine for linking to Forms.

Example:
"L1-S1" works as input when linking to a Sense in a statement but "l1-s1" doesn't.

BDD
GIVEN a value input field for a statement of datatype Sense
AND I enter the Sense ID in lowercase
THEN the input is accepted

Acceptance criteria:

  • Lowercase Sense IDs are accepted.

Event Timeline

Restricted Application added a project: Wikidata. · View Herald TranscriptSep 1 2018, 6:00 PM
Lydia_Pintscher triaged this task as High priority.Sep 1 2018, 6:00 PM

When I try this out locally, lowercase form IDs like l1-f1 don’t work either. Could this be because of CirrusSearch on Wikidata?

Change 457441 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/Wikibase@master] Make EntityIdSearchHelper case-insensitive

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

Change 457441 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Make EntityIdSearchHelper case-insensitive

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