Last modified: 2011-02-25 18:20:18 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 T29617, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 27617 - Add userid of uploader to list=filearchive
Add userid of uploader to list=filearchive
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.18.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Roan Kattouw
:
Depends on:
Blocks: 19195
  Show dependency treegraph
 
Reported: 2011-02-21 18:43 UTC by db [inactive,noenotif]
Modified: 2011-02-25 18:20 UTC (History)
4 users (show)

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


Attachments

Description db [inactive,noenotif] 2011-02-21 18:43:12 UTC
Add the user ID that uploaded the image version of a deleted image is not available with list=filearchive.

Maybe add faprop=userid.

Thanks.

See bug 19195.
Comment 1 Sam Reed (reedy) 2011-02-25 17:17:59 UTC
list=filearchive&faprop=user

    <filearchive>
      <fa name="WFM.png" user="1" />
      <fa name="WFM.png" user="1" />
    </filearchive>

Noted, that I uploaded and deleted it locally. I do notice, that although "user" gives the user id, but it doesn't the text

Just tweaked, as it read horribly
' user         - Adds user who uploaded the image version',

So, this should already be done? As user returns the userid..

But should be tweaked for userid to retun the user id, and user to return the username
Comment 2 Sam Reed (reedy) 2011-02-25 17:35:56 UTC
Brought those changes in in r82803
Comment 3 db [inactive,noenotif] 2011-02-25 18:07:52 UTC
Why not change faprop=user to user_text and add faprop=userid with user. Than backport to 1.17. That is not a breaking change, because list=filearchive was added in 1.17. Than the prop and the output are like other modules.
Comment 4 Sam Reed (reedy) 2011-02-25 18:13:53 UTC
That's what I did in r82803, well, asking for user gives both.

Will backport to 1.17 and 1.17wmf1
Comment 5 Sam Reed (reedy) 2011-02-25 18:17:42 UTC
r82805, r82806
Comment 6 db [inactive,noenotif] 2011-02-25 18:20:18 UTC
Okay.

An extra faprop is not necessary, thats right.

Thanks.

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


Navigation
Links