Last modified: 2011-02-21 23:33:21 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 T17348, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 15348 - prop=categories returns timestamps that are not in ISO 8601 format
prop=categories returns timestamps that are not in ISO 8601 format
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.14.x
All All
: Normal normal (vote)
: ---
Assigned To: jeremyb
http://commons.wikimedia.org/w/api.ph...
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-28 16:06 UTC by jeremyb
Modified: 2011-02-21 23:33 UTC (History)
3 users (show)

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


Attachments
format as ISO 8601 cl_timestamp before adding it to the array (patch against trunk) (484 bytes, patch)
2008-08-28 16:06 UTC, jeremyb
Details

Description jeremyb 2008-08-28 16:06:10 UTC
Created attachment 5223 [details]
format as ISO 8601 cl_timestamp before adding it to the array (patch against trunk)

ENV: bug tested on commons which says it's running 1.14 alpha (r40071)

prop=categories returns timestamps that are not in ISO 8601 format

I combed through `gfind -name .svn -type d -prune -o -iname \*api\* -type f -exec grep -in -25 "timestamp" {} +` and the only place I could find that returns a timestamp not in ISO 8601 format is fixed in the attached patch.

Of course, there could be cases that don't use the token 'timestamp' in or near the assignment or I could have missed one.  (I'm not terribly well versed in MW code)

The patch has not yet been tested.
Comment 1 Roan Kattouw 2008-08-28 16:27:39 UTC
Applied in r40161 and r40162.
Comment 2 jeremyb 2008-08-28 16:48:25 UTC
WFM after checkin and sync.

Thanks.
Comment 3 Sam Reed (reedy) 2011-02-21 23:33:00 UTC
do we use verified?
Comment 4 Krinkle 2011-02-21 23:33:21 UTC
Marking as resolved.

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


Navigation
Links