[Bug 38675] Review Wikibase API error handling

2013-10-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Lydia Pintscher  changed:

   What|Removed |Added

 Status|RESOLVED|VERIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2013-10-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Lydia Pintscher  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2013-10-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #11 from Addshore  ---
As far as I can tell both points mentioned in the first comment on this big are
resolved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2013-10-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Lydia Pintscher  changed:

   What|Removed |Added

 CC||lydia.pintscher@wikimedia.d
   ||e

--- Comment #10 from Lydia Pintscher  ---
Adam: What's the status here? Can we close this or is something still needed?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2013-07-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Addshore  changed:

   What|Removed |Added

 CC||addshorew...@gmail.com
 Blocks||4

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-11-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

db  changed:

   What|Removed |Added

 CC||duplicate...@googlemail.com

--- Comment #9 from db  2012-11-28 14:27:15 UTC ---
(In reply to comment #7)
> https://gerrit.wikimedia.org/r/23298

Status Merged

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #8 from jeb...@gmail.com 2012-11-01 12:29:37 UTC ---
*** Bug 41599 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-09-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #7 from jeb...@gmail.com 2012-09-10 12:41:00 UTC ---
Marking API error messages with a comment "#Do not translate"

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

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-09-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #6 from Raimond Spekking  2012-09-09 
12:50:15 UTC ---
(In reply to comment #5)
> For later, please note the patchset https://gerrit.wikimedia.org/r/#/c/23154/1
> 
> We probably don't need to mark the ordinary messages, but we should probably
> mark the qqq descriptions. Or we should verify which one of them the
> i18n-people want us to mark.
> 
> Also see
> https://gerrit.wikimedia.org/r/#/c/23078/
> https://gerrit.wikimedia.org/r/#/c/21330/

If you have messages to be untranslated, please add a PHP comment like '# Do
not translate' to it line in the English message section.

The i18n team (most of the time Siebrand or me) will add the keys to our config
file
https://gerrit.wikimedia.org/r/gitweb?p=translatewiki.git;a=blob;f=MediaWiki/mediawiki-defines.txt#l2055
once the patches are merged.

Ignored Messages will be deleted from the qqq section (beides all existing
translations) during the daily export by our scripts.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-09-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #5 from jeb...@gmail.com 2012-09-08 11:46:14 UTC ---
For later, please note the patchset https://gerrit.wikimedia.org/r/#/c/23154/1

We probably don't need to mark the ordinary messages, but we should probably
mark the qqq descriptions. Or we should verify which one of them the
i18n-people want us to mark.

Also see
https://gerrit.wikimedia.org/r/#/c/23078/
https://gerrit.wikimedia.org/r/#/c/21330/

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #4 from jeb...@gmail.com 2012-08-24 07:57:00 UTC ---
Can't find my notes on this but basically this is what needs to be fixed if we
want a generalized approach that solves this in general

* Change the calls to dieUsage to dieUsageMsg wheree applicable, or
* Change the calls to wfMsg in dieUsage

* Add the necessary arguments to the result in dieUsage where such arguments
are needed for later processing, and
* Make sure it is safe to report those arguments

* Copy the now localized messages from the i18n-file to getPossibleErrors, or
* Copy them to the actual call to dieUsage, or
* Copy them to the actual call to dieUsageMsg (more trouble with args)

* Investigate if the rest of the API is sufficiently well-behaved that we can
localize in the user interface

* Figure out which API error message we want to localize that is now in the
details section, or (we could easily end up with a 3-figure number)
* If we simply don't want to localize them at all, or
* If we simply don't want to show error messages in the details section at all

* Figure out which API error message we want to map to simplified UI messages

* Implement the API calls to get the localized messages
** Inject a spinner into the details section
** A request to meta=allmessages (session-failure will give a call
http://wikidata-test-repo.wikimedia.de/w/api.php?action=query&meta=allmessages&ammessages=wikibase-api-session-failure&amlang=de)
** Store the error message in the ordinary message structure in the browser
** Insert the args supplied from the error message (the dieUsage/dieUsageMsg)
into the localized message
** Remove the spinner from the details section
** Insert the finalized localized message into the details section

I don't think that this has an easy fix.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

jeb...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 CC||jeb...@gmail.com

--- Comment #3 from jeb...@gmail.com 2012-08-24 07:24:23 UTC ---
I really don't think we should assign this one without further discussion, this
is a rather big task.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Aude  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

Aude  changed:

   What|Removed |Added

 CC||aude.w...@gmail.com

--- Comment #2 from Aude  2012-08-23 10:03:44 UTC ---
Talked with Siebrand and we should be localizing the error messages when they
are used in the UI and do so in the JavaScript, in our case.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

denny vrandecic  changed:

   What|Removed |Added

   Priority|High|Normal

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

--- Comment #1 from denny vrandecic  2012-08-02 
11:04:38 UTC ---
Ask on wikitech-l if there are respective plans for core.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 38675] Review Wikibase API error handling

2012-08-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=38675

denny vrandecic  changed:

   What|Removed |Added

   Priority|Unprioritized   |High
 CC||denny.vrandecic@wikimedia.d
   ||e

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l