Merge "EditPage: Restore getSummaryInput(); deprecate it and getSummaryInputOOUI()"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Fri, 1 Sep 2017 20:17:01 +0000 (20:17 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Fri, 1 Sep 2017 20:17:01 +0000 (20:17 +0000)
1  2 
RELEASE-NOTES-1.30
includes/EditPage.php

diff --combined RELEASE-NOTES-1.30
@@@ -61,11 -61,6 +61,11 @@@ section)
    just been unwatched.
  * Added $wgParserTestMediaHandlers, where mock media handlers can be passed to
    MediaHandlerFactory for parser tests.
 +* Edit summaries, block reasons, and other "comments" are now stored in a
 +  separate database table. Use the CommentFormatter class to access them.
 +** This is currently gated by $wgCommentTableSchemaMigrationStage. Most wikis
 +   can set this to MIGRATION_NEW and run maintenance/migrateComments.php as
 +   soon as any necessary extensions are updated.
  
  === External library changes in 1.30 ===
  
@@@ -157,7 -152,6 +157,7 @@@ changes to languages because of Phabric
    WikiPage::makeParserOptions() to create the ParserOptions object and only
    change options that affect the parser cache key.
  * Article::viewRedirect() is deprecated.
 +* IP::isValidBlock() was deprecated. Use the equivalent IP::isValidRange().
  * DeprecatedGlobal no longer supports passing in a direct value, it requires a
    callable factory function or a class name.
  * The $parserMemc global, wfGetParserCacheStorage(), and ParserCache::singleton()
  * MWMemcached and MemCachedClientforWiki classes (deprecated in 1.27) were removed.
    The MemcachedClient class should be used instead.
  * EditPage::isOouiEnabled() is deprecated and will always return true.
+ * EditPage::getSummaryInput() and ::getSummaryInputOOUI() are deprecated. Please
+   use ::getSummaryInputWidget() instead.
  * Parser::getRandomString() (deprecated in 1.26) was removed.
  * Parser::uniqPrefix() (deprecated in 1.26) was removed.
 -* Parser::extractTagsAndParams() now only accepts three arguments.  The fourth,
 +* Parser::extractTagsAndParams() now only accepts three arguments. The fourth,
    $uniq_prefix was deprecated in 1.26 and has now been removed.
 +* (T172514) The following tables have had their UNIQUE indexes turned into proper
 +  PRIMARY KEYs for increased maintainability: categorylinks, imagelinks, iwlinks,
 +  langlinks, log_search, module_deps, objectcache, pagelinks, query_cache, site_stats,
 +  templatelinks, text, transcache, user_former_groups, user_properties.
 +* IDatabase::nextSequenceValue() is no longer needed by any database backends
 +  (formerly it was needed by PostgreSQL and Oracle), and is now deprecated.
  
  == Compatibility ==
  MediaWiki 1.30 requires PHP 5.5.9 or later. There is experimental support for
diff --combined includes/EditPage.php
@@@ -2658,7 -2658,8 +2658,7 @@@ class EditPage 
                $wgOut->addHTML( Html::openElement(
                        'form',
                        [
 -                              // Keep mw-editform-ooui class for backwards-compatibility temporarily
 -                              'class' => 'mw-editform mw-editform-ooui',
 +                              'class' => 'mw-editform',
                                'id' => self::EDITFORM_ID,
                                'name' => self::EDITFORM_ID,
                                'method' => 'post',
  
                if ( $this->wasDeletedSinceLastEdit() && 'save' == $this->formtype ) {
                        $username = $this->lastDelete->user_name;
 -                      $comment = $this->lastDelete->log_comment;
 +                      $comment = CommentStore::newKey( 'log_comment' )->getComment( $this->lastDelete )->text;
  
                        // It is better to not parse the comment at all than to have templates expanded in the middle
                        // TODO: can the checkLabel be moved outside of the div so that wrapWikiMsg could be used?
                ];
        }
  
+       /**
+        * Standard summary input and label (wgSummary), abstracted so EditPage
+        * subclasses may reorganize the form.
+        * Note that you do not need to worry about the label's for=, it will be
+        * inferred by the id given to the input. You can remove them both by
+        * passing [ 'id' => false ] to $userInputAttrs.
+        *
+        * @deprecated since 1.30 Use getSummaryInputWidget() instead
+        * @param string $summary The value of the summary input
+        * @param string $labelText The html to place inside the label
+        * @param array $inputAttrs Array of attrs to use on the input
+        * @param array $spanLabelAttrs Array of attrs to use on the span inside the label
+        * @return array An array in the format [ $label, $input ]
+        */
+       public function getSummaryInput( $summary = "", $labelText = null,
+               $inputAttrs = null, $spanLabelAttrs = null
+       ) {
+               wfDeprecated( __METHOD__, '1.30' );
+               $inputAttrs = $this->getSummaryInputAttributes( $inputAttrs );
+               $inputAttrs += Linker::tooltipAndAccesskeyAttribs( 'summary' );
+               $spanLabelAttrs = ( is_array( $spanLabelAttrs ) ? $spanLabelAttrs : [] ) + [
+                       'class' => $this->missingSummary ? 'mw-summarymissed' : 'mw-summary',
+                       'id' => "wpSummaryLabel"
+               ];
+               $label = null;
+               if ( $labelText ) {
+                       $label = Xml::tags(
+                               'label',
+                               $inputAttrs['id'] ? [ 'for' => $inputAttrs['id'] ] : null,
+                               $labelText
+                       );
+                       $label = Xml::tags( 'span', $spanLabelAttrs, $label );
+               }
+               $input = Html::input( 'wpSummary', $summary, 'text', $inputAttrs );
+               return [ $label, $input ];
+       }
        /**
         * Builds a standard summary input with a label.
         *
+        * @deprecated since 1.30 Use getSummaryInputWidget() instead
         * @param string $summary The value of the summary input
         * @param string $labelText The html to place inside the label
         * @param array $inputAttrs Array of attrs to use on the input
         * @return OOUI\FieldLayout OOUI FieldLayout with Label and Input
         */
        function getSummaryInputOOUI( $summary = "", $labelText = null, $inputAttrs = null ) {
+               wfDeprecated( __METHOD__, '1.30' );
+               $this->getSummaryInputWidget( $summary, $labelText, $inputAttrs );
+       }
+       /**
+        * Builds a standard summary input with a label.
+        *
+        * @param string $summary The value of the summary input
+        * @param string $labelText The html to place inside the label
+        * @param array $inputAttrs Array of attrs to use on the input
+        *
+        * @return OOUI\FieldLayout OOUI FieldLayout with Label and Input
+        */
+       function getSummaryInputWidget( $summary = "", $labelText = null, $inputAttrs = null ) {
                $inputAttrs = OOUI\Element::configFromHtmlAttributes(
                        $this->getSummaryInputAttributes( $inputAttrs )
                );
                }
  
                $labelText = $this->context->msg( $isSubjectPreview ? 'subject' : 'summary' )->parse();
-               $wgOut->addHTML( $this->getSummaryInputOOUI(
+               $wgOut->addHTML( $this->getSummaryInputWidget(
                                $summary,
                                $labelText,
                                [ 'class' => $summaryClass ]
                $wgOut->addHTML( Html::hidden( 'wpStarttime', $this->starttime ) );
                $wgOut->addHTML( Html::hidden( 'wpEdittime', $this->edittime ) );
                $wgOut->addHTML( Html::hidden( 'editRevId', $this->editRevId ) );
 -              $wgOut->addHTML( Html::hidden( 'wpScrolltop', $this->scrolltop ) );
 +              $wgOut->addHTML( Html::hidden( 'wpScrolltop', $this->scrolltop, [ 'id' => 'wpScrolltop' ] ) );
  
                if ( !$this->checkUnicodeCompliantBrowser() ) {
                        $wgOut->addHTML( Html::hidden( 'safemode', '1' ) );
         */
        protected function getLastDelete() {
                $dbr = wfGetDB( DB_REPLICA );
 +              $commentQuery = CommentStore::newKey( 'log_comment' )->getJoin();
                $data = $dbr->selectRow(
 -                      [ 'logging', 'user' ],
 +                      [ 'logging', 'user' ] + $commentQuery['tables'],
                        [
                                'log_type',
                                'log_action',
                                'log_user',
                                'log_namespace',
                                'log_title',
 -                              'log_comment',
                                'log_params',
                                'log_deleted',
                                'user_name'
 -                      ], [
 +                      ] + $commentQuery['fields'], [
                                'log_namespace' => $this->mTitle->getNamespace(),
                                'log_title' => $this->mTitle->getDBkey(),
                                'log_type' => 'delete',
                                'user_id=log_user'
                        ],
                        __METHOD__,
 -                      [ 'LIMIT' => 1, 'ORDER BY' => 'log_timestamp DESC' ]
 +                      [ 'LIMIT' => 1, 'ORDER BY' => 'log_timestamp DESC' ],
 +                      [
 +                              'user' => [ 'JOIN', 'user_id=log_user' ],
 +                      ] + $commentQuery['joins']
                );
                // Quick paranoid permission checks...
                if ( is_object( $data ) ) {
                        }
  
                        if ( $data->log_deleted & LogPage::DELETED_COMMENT ) {
 -                              $data->log_comment = $this->context->msg( 'rev-deleted-comment' )->escaped();
 +                              $data->log_comment_text = $this->context->msg( 'rev-deleted-comment' )->escaped();
 +                              $data->log_comment_data = null;
                        }
                }
  
         *   where bool indicates the checked status of the checkbox
         * @return array
         */
 -      protected function getCheckboxesDefinition( $checked ) {
 +      public function getCheckboxesDefinition( $checked ) {
                global $wgUser;
                $checkboxes = [];