Last modified: 2014-11-04 17:48:30 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 T74685, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 72685 - ApiBounceHandler improvements
ApiBounceHandler improvements
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
BounceHandler (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 69019
  Show dependency treegraph
 
Reported: 2014-10-29 17:23 UTC by Sam Reed (reedy)
Modified: 2014-11-04 17:48 UTC (History)
4 users (show)

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


Attachments

Description Sam Reed (reedy) 2014-10-29 17:23:54 UTC
The module should probably be marked internal.

It should probably also use proper api parameter handling rather than:
$email = $this->getMain()->getVal( 'email' );


Might be worth adding a description too...
Comment 1 Gerrit Notification Bot 2014-11-01 16:20:34 UTC
Change 170523 had a related patch set uploaded by 01tonythomas:
Marked the BounceHandler API as internal

https://gerrit.wikimedia.org/r/170523
Comment 2 Gerrit Notification Bot 2014-11-03 17:55:59 UTC
Change 170523 merged by jenkins-bot:
Marked the BounceHandler API as internal

https://gerrit.wikimedia.org/r/170523
Comment 3 Gerrit Notification Bot 2014-11-04 08:19:32 UTC
Change 170889 had a related patch set uploaded by 01tonythomas:
Add API parameter handling details for the bounehandler API

https://gerrit.wikimedia.org/r/170889
Comment 4 Gerrit Notification Bot 2014-11-04 17:40:01 UTC
Change 170889 merged by jenkins-bot:
Add API parameter handling details for the bounehandler API

https://gerrit.wikimedia.org/r/170889
Comment 5 Sam Reed (reedy) 2014-11-04 17:48:30 UTC
Think this is done now...

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


Navigation
Links