ApiClearHasMsg: Clean up after I341b395e
authorBrad Jorsch <bjorsch@wikimedia.org>
Tue, 3 Jan 2017 19:23:32 +0000 (14:23 -0500)
committerBrad Jorsch <bjorsch@wikimedia.org>
Tue, 3 Jan 2017 19:23:32 +0000 (14:23 -0500)
Since I341b395e, $this->getRequest()->wasPosted() should never be false.
So don't bother testing for it.

Change-Id: Iccc60424ed5442d950f4968297fdd18001b3a02a

includes/api/ApiClearHasMsg.php

index 99242a8..a5474b5 100644 (file)
 class ApiClearHasMsg extends ApiBase {
        public function execute() {
                $user = $this->getUser();
-               if ( $this->getRequest()->wasPosted() ) {
-                       $user->setNewtalk( false );
-               } else {
-                       DeferredUpdates::addCallableUpdate( function () use ( $user ) {
-                               $user->setNewtalk( false );
-                       } );
-               }
+               $user->setNewtalk( false );
                $this->getResult()->addValue( null, $this->getModuleName(), 'success' );
        }