X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=blobdiff_plain;f=includes%2Fpage%2FWikiPage.php;h=f760cd1601c65776f7283c31876e4bc625b5e721;hp=071cee7eb3af4b8517e3dd85f831f46d4ec3ffb6;hb=5eff86b740f75ade6c35e8069253fe076c0867bb;hpb=9101cc6ef9d7ecacb3745d655ba6fd927d1b554b diff --git a/includes/page/WikiPage.php b/includes/page/WikiPage.php index 071cee7eb3..f760cd1601 100644 --- a/includes/page/WikiPage.php +++ b/includes/page/WikiPage.php @@ -316,7 +316,10 @@ class WikiPage implements Page, IDBAccessObject { protected function pageData( $dbr, $conditions, $options = [] ) { $fields = self::selectFields(); - Hooks::run( 'ArticlePageDataBefore', [ &$this, &$fields ] ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + + Hooks::run( 'ArticlePageDataBefore', [ &$wikiPage, &$fields ] ); $row = $dbr->selectRow( 'page', $fields, $conditions, __METHOD__, $options ); @@ -1121,7 +1124,10 @@ class WikiPage implements Page, IDBAccessObject { * @return bool */ public function doPurge( $flags = self::PURGE_ALL ) { - if ( !Hooks::run( 'ArticlePurge', [ &$this ] ) ) { + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + + if ( !Hooks::run( 'ArticlePurge', [ &$wikiPage ] ) ) { return false; } @@ -1498,68 +1504,6 @@ class WikiPage implements Page, IDBAccessObject { return $flags; } - /** - * Change an existing article or create a new article. Updates RC and all necessary caches, - * optionally via the deferred update array. - * - * @param string $text New text - * @param string $summary Edit summary - * @param int $flags Bitfield: - * EDIT_NEW - * Article is known or assumed to be non-existent, create a new one - * EDIT_UPDATE - * Article is known or assumed to be pre-existing, update it - * EDIT_MINOR - * Mark this edit minor, if the user is allowed to do so - * EDIT_SUPPRESS_RC - * Do not log the change in recentchanges - * EDIT_FORCE_BOT - * Mark the edit a "bot" edit regardless of user rights - * EDIT_AUTOSUMMARY - * Fill in blank summaries with generated text where possible - * EDIT_INTERNAL - * Signal that the page retrieve/save cycle happened entirely in this request. - * - * If neither EDIT_NEW nor EDIT_UPDATE is specified, the status of the - * article will be detected. If EDIT_UPDATE is specified and the article - * doesn't exist, the function will return an edit-gone-missing error. If - * EDIT_NEW is specified and the article does exist, an edit-already-exists - * error will be returned. These two conditions are also possible with - * auto-detection due to MediaWiki's performance-optimised locking strategy. - * - * @param bool|int $baseRevId The revision ID this edit was based off, if any. - * This is not the parent revision ID, rather the revision ID for older - * content used as the source for a rollback, for example. - * @param User $user The user doing the edit - * - * @throws MWException - * @return Status Possible errors: - * edit-hook-aborted: The ArticleSave hook aborted the edit but didn't - * set the fatal flag of $status - * edit-gone-missing: In update mode, but the article didn't exist. - * edit-conflict: In update mode, the article changed unexpectedly. - * edit-no-change: Warning that the text was the same as before. - * edit-already-exists: In creation mode, but the article already exists. - * - * Extensions may define additional errors. - * - * $return->value will contain an associative array with members as follows: - * new: Boolean indicating if the function attempted to create a new article. - * revision: The revision object for the inserted revision, or null. - * - * Compatibility note: this function previously returned a boolean value - * indicating success/failure - * - * @deprecated since 1.21: use doEditContent() instead. - */ - public function doEdit( $text, $summary, $flags = 0, $baseRevId = false, $user = null ) { - wfDeprecated( __METHOD__, '1.21' ); - - $content = ContentHandler::makeContent( $text, $this->getTitle() ); - - return $this->doEditContent( $content, $summary, $flags, $baseRevId, $user ); - } - /** * Change an existing article or create a new article. Updates RC and all necessary caches, * optionally via the deferred update array. @@ -1598,6 +1542,7 @@ class WikiPage implements Page, IDBAccessObject { * @param array|null $tags Change tags to apply to this edit * Callers are responsible for permission checks * (with ChangeTags::canAddTagsAccompanyingChange) + * @param Int $undidRevId Id of revision that was undone or 0 * * @throws MWException * @return Status Possible errors: @@ -1619,7 +1564,7 @@ class WikiPage implements Page, IDBAccessObject { */ public function doEditContent( Content $content, $summary, $flags = 0, $baseRevId = false, - User $user = null, $serialFormat = null, $tags = [] + User $user = null, $serialFormat = null, $tags = [], $undidRevId = 0 ) { global $wgUser, $wgUseAutomaticEditSummaries; @@ -1648,14 +1593,15 @@ class WikiPage implements Page, IDBAccessObject { $user = $user ?: $wgUser; $flags = $this->checkFlags( $flags ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + // Trigger pre-save hook (using provided edit summary) $hookStatus = Status::newGood( [] ); - $hook_args = [ &$this, &$user, &$content, &$summary, + $hook_args = [ &$wikiPage, &$user, &$content, &$summary, $flags & EDIT_MINOR, null, null, &$flags, &$hookStatus ]; // Check if the hook rejected the attempted save - if ( !Hooks::run( 'PageContentSave', $hook_args ) - || !ContentHandler::runLegacyHooks( 'ArticleSave', $hook_args, '1.21' ) - ) { + if ( !Hooks::run( 'PageContentSave', $hook_args ) ) { if ( $hookStatus->isOK() ) { // Hook returned false but didn't call fatal(); use generic message $hookStatus->fatal( 'edit-hook-aborted' ); @@ -1698,7 +1644,8 @@ class WikiPage implements Page, IDBAccessObject { 'oldId' => $this->getLatest(), 'oldIsRedirect' => $this->isRedirect(), 'oldCountable' => $this->isCountable(), - 'tags' => ( $tags !== null ) ? (array)$tags : [] + 'tags' => ( $tags !== null ) ? (array)$tags : [], + 'undidRevId' => $undidRevId ]; // Actually create the revision and create/update the page @@ -1876,10 +1823,12 @@ class WikiPage implements Page, IDBAccessObject { 'oldrevision' => $meta['oldRevision'] ] ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; // Trigger post-save hook - $params = [ &$this, &$user, $content, $summary, $flags & EDIT_MINOR, - null, null, &$flags, $revision, &$status, $meta['baseRevId'] ]; - ContentHandler::runLegacyHooks( 'ArticleSaveComplete', $params ); + $params = [ &$wikiPage, &$user, $content, $summary, $flags & EDIT_MINOR, + null, null, &$flags, $revision, &$status, $meta['baseRevId'], + $meta['undidRevId'] ]; Hooks::run( 'PageContentSaveComplete', $params ); } ), @@ -1996,16 +1945,15 @@ class WikiPage implements Page, IDBAccessObject { ) { // Update links, etc. $this->doEditUpdates( $revision, $user, [ 'created' => true ] ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; // Trigger post-create hook - $params = [ &$this, &$user, $content, $summary, + $params = [ &$wikiPage, &$user, $content, $summary, $flags & EDIT_MINOR, null, null, &$flags, $revision ]; - ContentHandler::runLegacyHooks( 'ArticleInsertComplete', $params, '1.21' ); Hooks::run( 'PageContentInsertComplete', $params ); // Trigger post-save hook $params = array_merge( $params, [ &$status, $meta['baseRevId'] ] ); - ContentHandler::runLegacyHooks( 'ArticleSaveComplete', $params, '1.21' ); Hooks::run( 'PageContentSaveComplete', $params ); - } ), DeferredUpdates::PRESEND @@ -2274,9 +2222,12 @@ class WikiPage implements Page, IDBAccessObject { } } - Hooks::run( 'ArticleEditUpdates', [ &$this, &$editInfo, $options['changed'] ] ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; - if ( Hooks::run( 'ArticleEditUpdatesDeleteFromRecentchanges', [ &$this ] ) ) { + Hooks::run( 'ArticleEditUpdates', [ &$wikiPage, &$editInfo, $options['changed'] ] ); + + if ( Hooks::run( 'ArticleEditUpdatesDeleteFromRecentchanges', [ &$wikiPage ] ) ) { // Flush old entries from the `recentchanges` table if ( mt_rand( 0, 9 ) == 0 ) { JobQueueGroup::singleton()->lazyPush( RecentChangesUpdateJob::newPurgeJob() ); @@ -2320,9 +2271,12 @@ class WikiPage implements Page, IDBAccessObject { if ( !$recipient ) { wfDebug( __METHOD__ . ": invalid username\n" ); } else { + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + // Allow extensions to prevent user notification // when a new message is added to their talk page - if ( Hooks::run( 'ArticleEditUpdateNewTalk', [ &$this, $recipient ] ) ) { + if ( Hooks::run( 'ArticleEditUpdateNewTalk', [ &$wikiPage, $recipient ] ) ) { if ( User::isIP( $shortTitle ) ) { // An anonymous user $recipient->setNewtalk( true, $revision ); @@ -2451,7 +2405,10 @@ class WikiPage implements Page, IDBAccessObject { $nullRevision = null; if ( $id ) { // Protection of existing page - if ( !Hooks::run( 'ArticleProtect', [ &$this, &$user, $limit, $reason ] ) ) { + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + + if ( !Hooks::run( 'ArticleProtect', [ &$wikiPage, &$user, $limit, $reason ] ) ) { return Status::newGood(); } @@ -2538,9 +2495,12 @@ class WikiPage implements Page, IDBAccessObject { __METHOD__ ); + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + Hooks::run( 'NewRevisionFromEditComplete', [ $this, $nullRevision, $latest, $user ] ); - Hooks::run( 'ArticleProtectComplete', [ &$this, &$user, $limit, $reason ] ); + Hooks::run( 'ArticleProtectComplete', [ &$wikiPage, &$user, $limit, $reason ] ); } else { // Protection of non-existing page (also known as "title protection") // Cascade protection is meaningless in this case $cascade = false; @@ -2820,9 +2780,12 @@ class WikiPage implements Page, IDBAccessObject { return $status; } + // Avoid PHP 7.1 warning of passing $this by reference + $wikiPage = $this; + $user = is_null( $user ) ? $wgUser : $user; if ( !Hooks::run( 'ArticleDelete', - [ &$this, &$user, &$reason, &$error, &$status, $suppress ] + [ &$wikiPage, &$user, &$reason, &$error, &$status, $suppress ] ) ) { if ( $status->isOK() ) { // Hook aborted but didn't set a fatal status