WordPress.org

Make WordPress Core

Ticket #23901: 23901.10.patch

File 23901.10.patch, 9.4 KB (added by ocean90, 12 months ago)
  • wp-admin/includes/ajax-actions.php

     
    22712271 
    22722272                if ( ( $compare_two_mode || -1 !== $previous_revision_id ) ) { 
    22732273                        $all_the_revisions[] = array ( 
    2274                                 'ID'                   => $revision->ID, 
    2275                                 'titleTo'              => $revision_date_author, 
    2276                                 'titleFrom'            => $revision_from_date_author, 
    2277                                 'titleTooltip'         => $revision_date_author_short, 
    2278                                 'restoreLink'          => urldecode( $restore_link ), 
    2279                                 'revision_toload'      => true, 
    2280                                 'previous_revision_id' => $previous_revision_id, 
    2281                                 'is_current_revision'  => $is_current_revision, 
     2274                                'ID'           => $revision->ID, 
     2275                                'titleTo'      => $revision_date_author, 
     2276                                'titleFrom'    => $revision_from_date_author, 
     2277                                'titleTooltip' => $revision_date_author_short, 
     2278                                'restoreLink'  => urldecode( $restore_link ), 
     2279                                'previousID'   => $previous_revision_id, 
     2280                                'isCurrent'    => $is_current_revision, 
    22822281                        ); 
    22832282                } 
    22842283                $previous_revision_id = $revision->ID; 
  • wp-admin/js/revisions.js

     
    5959                        } 
    6060                }, 
    6161 
    62                 reloadToLoadRevisions: function( models, reverse_direction ) { 
     62                loadDiffs: function( models ) { 
    6363                        var self = this, 
    64                                 revisionsToLoad = models.where( { revision_toload: true } ), 
     64                                revisionsToLoad = models.where( { completed: false } ), 
    6565                                delay = 0; 
    6666 
    6767                        // match slider to passed revision_id 
     
    7777                                                        add: false, 
    7878                                                        remove: false, 
    7979                                                        success: function( model ) { 
    80                                                                 model.set( 'revision_toload', 'false' ); 
     80                                                                model.set( 'completed', true ); 
    8181 
    8282                                                                // stop spinner when all models are loaded 
    83                                                                 if ( 0 === models.where( { revision_toload: true } ).length ) 
     83                                                                if ( 0 === models.where( { completed: false } ).length ) 
    8484                                                                        self.stopModelLoadingSpinner(); 
    8585 
    8686                                                                self.tickmarkView.render(); 
    8787 
    88                                                                 var total_changes = model.get( 'lines_added' ) + model.get( 'lines_deleted'), 
    89                                                                         scope_of_changes = 'vsmall'; 
     88                                                                var totalChanges = model.get( 'linesAdded' ) + model.get( 'linesDeleted' ), 
     89                                                                        scopeOfChanges = 'vsmall'; 
    9090 
    9191                                                                // Note: hard coded scope of changes 
    9292                                                                // TODO change to dynamic based on range of values 
    93                                                                 if  ( total_changes > 1 && total_changes <= 3 ) { 
    94                                                                         scope_of_changes = 'small'; 
    95                                                                 } else if( total_changes > 3 && total_changes <= 5 ) { 
    96                                                                         scope_of_changes = 'med'; 
    97                                                                 } else if( total_changes > 5 && total_changes <= 10 ) { 
    98                                                                         scope_of_changes = 'large'; 
    99                                                                 } else if( total_changes > 10 ) { 
    100                                                                         scope_of_changes = 'vlarge'; 
     93                                                                if ( totalChanges > 1 && totalChanges <= 3 ) { 
     94                                                                        scopeOfChanges = 'small'; 
     95                                                                } else if ( totalChanges > 3 && totalChanges <= 5 ) { 
     96                                                                        scopeOfChanges = 'med'; 
     97                                                                } else if ( totalChanges > 5 && totalChanges <= 10 ) { 
     98                                                                        scopeOfChanges = 'large'; 
     99                                                                } else if ( totalChanges > 10 ) { 
     100                                                                        scopeOfChanges = 'vlarge'; 
    101101                                                                } 
    102                                                                 model.set( 'scope_of_changes', scope_of_changes ); 
     102                                                                model.set( 'scopeOfChanges', scopeOfChanges ); 
    103103                                                                if ( 0 !== self.rightDiff && 
    104104                                                                        model.get( 'ID' ) === self.revisions.at( self.rightDiff - 1 ).get( 'ID' ) ) { 
    105105                                                                        // reload if current model refreshed 
     
    149149                reloadModelSingle: function() { 
    150150                        var self = this; 
    151151 
    152                         // TODO: Only updates the query args yet 
     152                        self.startRightModelLoading(); 
     153 
    153154                        self.revisions.reload({ 
     155                                options: { 
    154156                                'showAutosaves': self.autosaves, 
    155157                                'showSplitView': self.showSplitView 
    156                         }); 
     158                                }, 
    157159 
    158                         self.startRightModelLoading(); 
    159                         self.revisions.fetch({ // reload revision data 
    160160                                success: function() { 
    161161                                        var revisionCount = self.revisions.length; 
    162162                                        self.revisionView.model = self.revisions; 
    163163                                        self.revisionView.render(); 
    164                                         self.reloadToLoadRevisions( self.revisions ); 
     164                                        self.loadDiffs( self.revisions ); 
    165165                                        self.tickmarkView.model = self.revisions; 
    166166                                        self.tickmarkView.render(); 
    167167                                        self.slider.refresh({ 
     
    173173                                error: function() { 
    174174                                        self.stopRightModelLoading(); 
    175175                                } 
    176  
    177176                        }); 
    178177                }, 
    179178 
     
    191190                        self.leftHandleRevisions.fetch({ 
    192191                                success: function(){ 
    193192                                        self.stopLeftModelLoading(); 
    194                                         self.reloadToLoadRevisions( self.leftHandleRevisions ); 
     193                                        self.loadDiffs( self.leftHandleRevisions ); 
    195194                                        self.tickmarkView.model = self.leftHandleRevisions; 
    196195                                        self.slider.refresh({ 
    197196                                                'max': self.revisions.length 
     
    222221                        self.rightHandleRevisions.fetch({ 
    223222                                success: function(){ 
    224223                                        self.stopRightModelLoading(); 
    225                                         self.reloadToLoadRevisions( self.rightHandleRevisions ); 
     224                                        self.loadDiffs( self.rightHandleRevisions ); 
    226225                                        self.tickmarkView.model = self.rightHandleRevisions; 
    227226                                        self.slider.refresh({ 
    228227                                                'max': self.revisions.length, 
     
    250249                        }); 
    251250                        this.revisionView.render(); 
    252251 
    253                         this.reloadToLoadRevisions( this.revisions ); 
     252                        this.loadDiffs( this.revisions ); 
    254253 
    255254                        this.revisionsInteractions = new revisions.view.Interact({ 
    256255                                model: this.revisions 
     
    262261                        }); 
    263262                        this.tickmarkView.render(); 
    264263                        this.tickmarkView.resetTicks(); 
    265  
    266264                } 
    267265        }); 
    268266 
     
    633631                        this.toggleCompareTwoCheckbox(); 
    634632 
    635633                        // hide the restore button when on the last sport/current post data 
    636                         $( '#restore-revision' ).toggle( ! Diff.revisions.at( Diff.rightDiff - 1 ).get( 'is_current_revision' ) ); 
     634                        $( '#restore-revision' ).toggle( ! Diff.revisions.at( Diff.rightDiff - 1 ).get( 'isCurrent' ) ); 
    637635 
    638636                        return this; 
    639637                }, 
     
    686684         */ 
    687685        Revision = revisions.model.Revision = Backbone.Model.extend({ 
    688686                idAttribute: 'ID', 
    689                 urlRoot: ajaxurl +      '?action=revisions-data' + 
    690                         '&show_autosaves=true&show_split_view=true&nonce=' + revisions.model.settings.nonce, 
     687 
    691688                defaults: { 
    692689                        ID: 0, 
    693690                        titleTo: '', 
     
    695692                        titleFrom: '', 
    696693                        diff: '<div class="diff-loading"><div class="spinner"></div></div>', 
    697694                        restoreLink: '', 
    698                         revision_toload: false, 
    699                         lines_added: 0, 
    700                         lines_deleted: 0, 
    701                         scope_of_changes: 'none', 
    702                         previous_revision_id: 0, 
    703                         is_current_revision: false 
     695                        completed: false, 
     696                        linesAdded: 0, 
     697                        linesDeleted: 0, 
     698                        scopeOfChanges: 'none', 
     699                        previousID: 0, 
     700                        isCurrent: false 
    704701                }, 
    705702 
    706703                url: function() { 
    707704                        if ( Diff.singleRevision ) { 
    708                                 return this.urlRoot + 
     705                                return ajaxurl + 
     706                                        '?action=revisions-data' + 
     707                                        '&show_autosaves=true' + 
     708                                        '&show_split_view=true' + 
     709                                        '&nonce=' + revisions.model.settings.nonce + 
    709710                                        '&single_revision_id=' + this.id + 
    710                                         '&compare_to=' + this.get( 'previous_revision_id' ) + 
     711                                        '&compare_to=' + this.get( 'previousID' ) + 
    711712                                        '&post_id=' + revisions.model.settings.post_id; 
    712713                        } else { 
    713714                                return this.collection.url() + '&single_revision_id=' + this.id; 
     
    721722         */ 
    722723        Revisions = revisions.Revisions = Backbone.Collection.extend({ 
    723724                model: Revision, 
    724                 urlRoot: ajaxurl + '?action=revisions-data', 
    725725 
    726726                initialize: function( models, options ) { 
    727727                        this.options = _.defaults( options || {}, { 
     
    736736                }, 
    737737 
    738738                url: function() { 
    739                         return this.urlRoot + 
     739                        return ajaxurl + 
     740                                '?action=revisions-data' + 
    740741                                '&compare_to=' + this.options.compareTo + 
    741742                                '&post_id=' + this.options.post_id + 
    742743                                '&show_autosaves=' + this.options.showAutosaves + 
     
    747748                }, 
    748749 
    749750                reload: function( options ) { 
    750                         this.options = _.defaults( options || {}, this.options ); 
     751                        this.options = _.defaults( options.options || {}, this.options ); 
    751752 
    752                         // TODO 
    753                         //this.fetch(); 
     753                        this.fetch({ 
     754                                success: options.success || null, 
     755                                error: options.error || null 
     756                        }); 
    754757                } 
    755758 
    756759        } ); 
  • wp-admin/css/colors-fresh.css

     
    13871387        background-color: #f7f7f7; 
    13881388} 
    13891389 
    1390 .revision-tick.loading-true { 
     1390.revision-tick.completed-false { 
    13911391        background-color: #9999cc; 
    13921392        background: url(../images/wpspin_light.gif) no-repeat; 
    13931393        background-position: middle; 
    13941394        background-size: 1px 10px; 
    13951395} 
    13961396 
    1397 .revision-tick.loading-false { 
     1397.revision-tick.completed-true { 
    13981398        background-color: #aaa; 
    13991399} 
    14001400 
  • wp-admin/css/colors-classic.css

     
    14841484        color: #0080aa; 
    14851485} 
    14861486 
    1487 .revision-tick.loading-true { 
     1487.revision-tick.completed-false { 
    14881488        background-color: #9999cc; 
    14891489        background: url(../images/wpspin_light.gif) no-repeat; 
    14901490        background-position: middle; 
    14911491        background-size: 1px 10px; 
    14921492} 
    14931493 
    1494 .revision-tick.loading-false { 
     1494.revision-tick.completed-true { 
    14951495        background-color: #aaa; 
    14961496} 
    14971497 
  • wp-admin/revision.php

     
    163163</script> 
    164164 
    165165<script id="tmpl-revision-ticks" type="text/html"> 
    166         <div class="revision-tick loading-{{{ data.revision_toload }}} scope-of-changes-{{{ data.scope_of_changes }}}"></div> 
     166        <div class="revision-tick completed-{{{ data.completed }}} scope-of-changes-{{{ data.scopeOfChanges }}}"></div> 
    167167</script> 
    168168<?php 
    169169require_once( './admin-footer.php' );