Last modified: 2013-10-09 17:23:51 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 T44645, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 42645 - Site link used error is confusing
Site link used error is confusing
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Wikidata bugs
: need-volunteer
: 41212 (view as bug list)
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2012-12-03 15:06 UTC by Aude
Modified: 2013-10-09 17:23 UTC (History)
7 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Aude 2012-12-03 15:06:33 UTC 
When I try to add a site link which is already used in another item, I get the following error:

"Edit not allowed: * Your edit was patched into the latest version. * Site link [[frwiki:Bière]] already used by item [[Item:Q63]]."

1) the word "patched" is completely confusing to end users
2) it sounds like something was successful, but then contradicts the "edit not allowed" and other stuff it's telling me.
3) "edit not allowed" also sounds harsh to me.  it's not a permissions issue, but the important thing it tells me is that the site link is already used in another item.  

Parsing the [[Item:Q63]] and giving me a link would also be very nice.
Comment 1 Daniel Kinzler 2012-12-03 20:08:30 UTC 
I absolutely agree, but sadly, this isn't easily done. What you see is the result of the somewhat convoluted and slightly inconsistent way error messages are passed around internally. E.g. the Status class that is used to return the errors insists on escaping all message parameters, making it impossible to provide links. Etc.

Several small fixes and some refactoring would be needed to make this work right. I think we should do that, but currently, other things are more urgent.
Comment 2 Michał Łazowik 2013-01-17 14:35:39 UTC 
Isn't that a dup of (bug 41212)? If yes, Daniel's anwser should be moved there.
Comment 3 Daniel Kinzler 2013-01-17 14:42:25 UTC 
*** Bug 41212 has been marked as a duplicate of this bug. ***
Comment 4 Lydia Pintscher 2013-10-08 17:15:08 UTC 
AFAIK this is fixed now.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links