Changeset 2824

Show
Ignore:
Timestamp:
07/21/08 20:27:29 (4 months ago)
Author:
mpaschal
Message:

Rename variables and clarify logic here to better understand Jay's patch
BugzID: 80010

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/release-41/lib/MT/CMS/Template.pm

    r2823 r2824  
    17021702    require MT::Util; 
    17031703 
    1704     my @blog_ids; 
     1704    my $user = $app->user; 
     1705    my @blogs_not_refreshed; 
    17051706    my $refreshed; 
     1707    my $can_refresh_system = $user->is_superuser() ? 1 : 0; 
    17061708    foreach my $blog_id (@id) { 
    17071709        my $blog; 
    …… 
    17101712            next unless $blog; 
    17111713        } 
    1712         if ( !$app->user->is_superuser() ) { 
     1714 
     1715        if (!$can_refresh_system) {  # system refreshers can refresh all blogs 
    17131716            my $perms = MT::Permission->load( 
    1714                 { blog_id => $blog_id, author_id => $app->user->id } ); 
    1715             if ( 
    1716                 !$perms 
    1717                 || (   !$perms->can_edit_templates() 
    1718                     && !$perms->can_administer_blog() ) 
    1719               ) 
    1720             { 
    1721                 push @blog_ids, $blog->id; 
     1717                { blog_id => $blog_id, author_id => $user->id } ); 
     1718            my $can_refresh_blog = !$perms                       ? 0 
     1719                                 : $perms->can_edit_templates()  ? 1 
     1720                                 : $perms->can_administer_blog() ? 1 
     1721                                 :                                 0 
     1722                                 ; 
     1723            if (!$can_refresh_blog) { 
     1724                push @blogs_not_refreshed, $blog->id; 
    17221725                next; 
    17231726            } 
    …… 
    18941897        $refreshed = 1; 
    18951898    } 
    1896     if (@blog_ids) { 
    1897         $app->add_return_arg( 'error_id' => join( ',', @blog_ids ) ); 
     1899    if (@blogs_not_refreshed) { 
     1900        $app->add_return_arg( 'error_id' => join( ',', @blogs_not_refreshed ) ); 
    18981901    } 
    18991902    $app->add_return_arg( 'refreshed' => 1 ) if $refreshed;