WordPress.org

Make WordPress Core

Ticket #24346: 24346.3.diff

File 24346.3.diff, 8.6 KB (added by adamsilverstein, 22 months ago)

more rtl trinary explicit parantheses

  • wp-admin/includes/ajax-actions.php

     
    22082208                        if ( ( ( 0 != $left_handle_at && $count < $left_handle_at ) || 
    22092209                                 ( 0 != $right_handle_at && $count > ( $right_handle_at - 2 ) ) ) ) { 
    22102210                                $all_the_revisions[] = array ( 
    2211                                         'ID' => $revision->ID, 
     2211                                        'ID'        => $revision->ID, 
     2212                                        'completed' => true, //mark as completed, no need to load diff 
    22122213                                ); 
    22132214                                continue; 
    22142215                        } 
    … …  
    22162217                        if ( ( 0 != $left_handle_at && $count > ( $left_handle_at - 1 ) || 
    22172218                                 ( 0 != $left_handle_at && $count < $right_handle_at ) ) ) { 
    22182219                                $all_the_revisions[] = array ( 
    2219                                         'ID' => $revision->ID, 
     2220                                        'ID'        => $revision->ID, 
     2221                                        'completed' => true, //mark as completed, no need to load diff 
    22202222                                ); 
    22212223                                continue; 
    22222224                        } 

  • wp-admin/js/revisions.js

     
    5656                        } 
    5757                }, 
    5858 
    59                 loadDiffs: function( models ) { 
     59                loadDiffs: function( models, otherModels ) { 
    6060                        var self = this, 
    6161                                revisionsToLoad = models.where( { completed: false } ), 
    6262                                delay = 0, 
    63                                 totalChanges; 
     63                                totalChanges, 
     64                                foundModel; 
    6465 
     66                        // in two handle mode, pass the other handle's models and copy over scopeOfChanges (tick width) from that model for positions this handle can't reach (but the other handle can reach) 
     67                        if ( 'undefined' !== typeof otherModels ) { // otherModels only passed in two handle mode 
     68                                _.each( models.where( { completed: true } ), // select just the models that are marked as not needing loading, eg. positions this handle can't reach 
     69                                        function( theModel ) { 
     70                                                foundModel = otherModels.findWhere( { ID: theModel.id } ); //find the matching position from the other handle's models 
     71                                                if ( 'undefined' !== typeof foundModel ) { 
     72                                                        theModel.set( 'scopeOfChanges', // set the displayed scope of changes for ticks this handle can't get to 
     73                                                                foundModel 
     74                                                                .get( 'scopeOfChanges' ) ); // copy the scope of changes from the other handle which can reach this spot 
     75                                                } 
     76                                        } 
     77                                ); 
     78                        } 
     79 
    6580                        // match slider to passed revision_id 
    6681                        _.each( revisionsToLoad, function( revision ) { 
    6782                                if ( revision.get( 'ID' ) == revisions.model.settings.revision_id ) 
    … …  
    6984                        }); 
    7085 
    7186                        _.each( revisionsToLoad, function( revision ) { 
    72                                         _.delay( function() { 
    73                                                 revision.fetch( { 
    74                                                         update: true, 
    75                                                         add: false, 
    76                                                         remove: false, 
    77                                                         success: function( model ) { 
    78                                                                 model.set( 'completed', true ); 
     87                                revision.fetch( { 
     88                                        update: true, 
     89                                        add: false, 
     90                                        remove: false, 
     91                                        success: function( model ) { 
     92                                                model.set( 'completed', true ); 
    7993 
    80                                                                 // stop spinner when all models are loaded 
    81                                                                 if ( 0 === models.where( { completed: false } ).length ) 
    82                                                                         self.stopModelLoadingSpinner(); 
     94                                                // stop spinner when all models are loaded 
     95                                                if ( 0 === models.where( { completed: false } ).length ) 
     96                                                        self.stopModelLoadingSpinner(); 
    8397 
    84                                                                 totalChanges = model.get( 'linesAdded' ) + model.get( 'linesDeleted' ), 
    85                                                                         scopeOfChanges = 'vsmall'; 
     98                                                totalChanges = model.get( 'linesAdded' ) + model.get( 'linesDeleted' ), 
     99                                                        scopeOfChanges = 'vsmall'; 
    86100 
    87                                                                 // Note: hard coded scope of changes 
    88                                                                 // TODO change to dynamic based on range of values 
    89                                                                 if ( totalChanges > 1 && totalChanges <= 3 ) { 
    90                                                                         scopeOfChanges = 'small'; 
    91                                                                 } else if ( totalChanges > 3 && totalChanges <= 5 ) { 
    92                                                                         scopeOfChanges = 'med'; 
    93                                                                 } else if ( totalChanges > 5 && totalChanges <= 10 ) { 
    94                                                                         scopeOfChanges = 'large'; 
    95                                                                 } else if ( totalChanges > 10 ) { 
    96                                                                         scopeOfChanges = 'vlarge'; 
    97                                                                 } 
    98                                                                 model.set( 'scopeOfChanges', scopeOfChanges ); 
    99                                                                 if ( 0 !== self.rightDiff && 
    100                                                                         model.get( 'ID' ) === self.revisions.at( self.rightDiff - 1 ).get( 'ID' ) ) { 
    101                                                                         // reload if current model refreshed 
    102                                                                         self.revisionView.render(); 
    103                                                                 } 
    104                                                                 self.tickmarkView.render(); 
    105                                                         } 
    106                                         } ); 
    107                                         }, delay ) ; 
    108                                         delay = delay + 150; // stagger model loads to avoid hammering server with requests 
    109                                 } 
    110                         ); 
     101                                                // Note: hard coded scope of changes 
     102                                                // TODO change to dynamic based on range of values 
     103                                                if ( totalChanges > 1 && totalChanges <= 3 ) { 
     104                                                        scopeOfChanges = 'small'; 
     105                                                } else if ( totalChanges > 3 && totalChanges <= 5 ) { 
     106                                                        scopeOfChanges = 'med'; 
     107                                                } else if ( totalChanges > 5 && totalChanges <= 10 ) { 
     108                                                        scopeOfChanges = 'large'; 
     109                                                } else if ( totalChanges > 10 ) { 
     110                                                        scopeOfChanges = 'vlarge'; 
     111                                                } 
     112                                                model.set( 'scopeOfChanges', scopeOfChanges ); 
     113                                                if ( 0 !== self.rightDiff && 
     114                                                        model.get( 'ID' ) === self.revisions.at( self.rightDiff - 1 ).get( 'ID' ) ) { 
     115                                                        // reload if current model refreshed 
     116                                                        self.revisionView.render(); 
     117                                                } 
     118                                                self.tickmarkView.render(); 
     119                                        } 
     120                                } ); 
     121                        } ); 
    111122                }, 
    112123 
    113124                startLeftModelLoading: function() { 
    … …  
    186197                        self.leftHandleRevisions.fetch({ 
    187198                                success: function(){ 
    188199                                        self.stopLeftModelLoading(); 
    189                                         self.loadDiffs( self.leftHandleRevisions ); 
     200                                        self.loadDiffs( self.leftHandleRevisions, self.rightHandleRevisions ); 
    190201                                        self.tickmarkView.model = self.leftHandleRevisions; 
    191202                                        self.slider.refresh({ 
    192203                                                'max': self.revisions.length 
    … …  
    216227                        self.rightHandleRevisions.fetch({ 
    217228                                success: function(){ 
    218229                                        self.stopRightModelLoading(); 
    219                                         self.loadDiffs( self.rightHandleRevisions ); 
     230                                        self.loadDiffs( self.rightHandleRevisions, self.leftHandleRevisions ); 
    220231                                        self.tickmarkView.model = self.rightHandleRevisions; 
    221232                                        self.slider.refresh({ 
    222233                                                'max': self.revisions.length 
    … …  
    242253 
    243254                disabledButtonCheck: function( val ) { 
    244255                        var maxVal = this.revisions.length - 1, 
    245                                 next = ! isRtl ? $( '#next' ) : $( '#previous' ), 
    246                                 prev = ! isRtl ? $( '#previous' ) : $( '#next' ); 
     256                                next = ( ! isRtl ? $( '#next' ) : $( '#previous' ) ), 
     257                                prev = ( ! isRtl ? $( '#previous' ) : $( '#next' ) ); 
    247258 
    248259                        // Disable "Next" button if you're on the last node 
    249260                        if ( maxVal === val ) 
    … …  
    325336                                        if ( Diff.leftModelLoading ) // left model still loading, prevent sliding left handle 
    326337                                                return false; 
    327338 
    328                                         Diff.leftDiff = isRtl ? ui.values[1] : ui.values[0]; // handles are reversed in RTL mode 
     339                                        Diff.leftDiff = ( isRtl ? ui.values[1] : ui.values[0] ); // handles are reversed in RTL mode 
    329340                                } else { 
    330341                                        // Right handler 
    331342                                        if ( Diff.rightModelLoading ) // right model still loading, prevent sliding right handle 
    332343                                                return false; 
    333344 
    334                                         Diff.rightDiff = isRtl ? ui.values[0] : ui.values[1]; // handles are reversed in RTL mode 
     345                                        Diff.rightDiff = ( isRtl ? ui.values[0] : ui.values[1] ); // handles are reversed in RTL mode 
    335346                                } 
    336347 
    337348                                Diff.revisionView.render(); 
    … …  
    362373                                        Diff.tickmarkView.render(); 
    363374                                } 
    364375 
    365                                 Diff.leftDiffStart = isRtl ? ui.values[1] : ui.values[0]; // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
     376                                Diff.leftDiffStart = ( isRtl ? ui.values[1] : ui.values[0] ); // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
    366377 
    367378                        } else { 
    368379                                // Right handler 
    … …  
    377388                                } 
    378389 
    379390                                Diff.revisionView.draggingLeft = false; 
    380                                 Diff.rightDiffStart = isRtl ? ui.values[0] : ui.values[1]; // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
     391                                Diff.rightDiffStart = ( isRtl ? ui.values[0] : ui.values[1] ); // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
    381392                        } 
    382393                }, 
    383394 
    … …  
    395406                        // and the right handle, swap out when dragging 
    396407                        if ( $( ui.handle ).hasClass( 'left-handle' ) ) { 
    397408                                // Left handler 
    398                                 if ( Diff.leftDiffStart !== isRtl ? ui.values[1] : ui.values[0] ) // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
     409                                if ( Diff.leftDiffStart !== ( isRtl ? ui.values[1] : ui.values[0] ) ) // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
    399410                                        Diff.reloadRight(); 
    400411                        } else { 
    401412                                // Right handler 
    402                                 if ( Diff.rightDiffStart !== isRtl ? ui.values[0] : ui.values[1] ) // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
     413                                if ( Diff.rightDiffStart !== ( isRtl ? ui.values[0] : ui.values[1] ) ) // in RTL mode the 'left handle' is the second in the slider, 'right' is first 
    403414                                        Diff.reloadLeft(); 
    404415                        } 
    405416                },