WordPress.org
Get WordPress

Make WordPress Core

Ticket #23901: 23901.10.patch

File 23901.10.patch, 9.4 KB (added by ocean90, 12 years 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' );