Last modified: 2014-08-17 10:57:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68207, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66207 - VisualEditor: Fields are case sensitive in citation dialog, allowing duplicates of the same field
VisualEditor: Fields are case sensitive in citation dialog, allowing duplicat...
Status: RESOLVED INVALID
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High trivial
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-05 18:42 UTC by mvolz
Modified: 2014-08-17 10:57 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Two URL fields visible. (163.39 KB, image/png)
2014-06-05 18:42 UTC, mvolz
Details
Two fields error (149.14 KB, image/png)
2014-06-05 18:47 UTC, mvolz
Details

Description mvolz 2014-06-05 18:42:37 UTC
Created attachment 15572 [details]
Two URL fields visible.

The citation dialog field selector is case sensitive. This allows the same field to be added multiple times as long as it's a different case. This results in a "multiple <field>" error when the citation is inserted because the fields are treated as case insensitive. 

This issue could organically come up if a citation was added using wikitext and then another user inspected the citation in VE.
Comment 1 mvolz 2014-06-05 18:47:14 UTC
Created attachment 15574 [details]
Two fields error
Comment 2 mvolz 2014-08-17 10:57:07 UTC
Not a VE bug :). Turns out this was actually because "URL," which is an accepted alias of "url" by module:citation/cs1, wasn't in the templatedata as an alias. This is probably true of a LOT of the other identifiers as well. I fixed this for url on en wiki for the "big 4" but this might also be an issue with other identifiers that are commonly capitalised as well, and probably on any template that is based on this module.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links