Changeset 1866

Show
Ignore:
Timestamp:
04/11/08 19:19:30 (5 months ago)
Author:
bchoate
Message:

Changes to store binary state to junk_status column. BugId:79280

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/release-34/build/mt-dists/default.mk

    r1731 r1866  
    22 
    33PRODUCT_VERSION = 4.15 
    4 SCHEMA_VERSION = 4.0052 
     4SCHEMA_VERSION = 4.0053 
    55API_VERSION = 4.15 
    66 

  • branches/release-34/lib/MT/App/CMS.pm

    r1858 r1866  
    755755                        { 
    756756                            created_on  => [ $ts, undef ], 
    757                             junk_status => [ 0,   1 ] 
     757                            junk_status => MT::Comment::NOT_JUNK(), 
    758758                        }, 
    759759                        { range_incl => { created_on => 1 }, unique => 1 } 
    …… 
    821821                handler => sub { 
    822822                    my ( $terms, $args ) = @_; 
    823                     $terms->{junk_status} = [ 0, 1 ]; 
     823                    require MT::TBPing; 
     824                    $terms->{junk_status} = MT::TBPing::NOT_JUNK(); 
    824825                }, 
    825826            }, 
    …… 
    831832                    require MT::Entry; 
    832833                    my $app = MT->instance; 
    833                     $terms->{junk_status} = [ 0, 1 ]; 
     834                    require MT::TBPing; 
    834835                    require MT::Trackback; 
     836                    $terms->{junk_status} = MT::TBPing::NOT_JUNK(); 
    835837                    $args->{join} = MT::Trackback->join_on( 
    836838                        undef, 
    …… 
    863865                handler => sub { 
    864866                    my ( $terms, $args ) = @_; 
    865                     $terms->{junk_status} = [ 0, 1 ]; 
     867                    require MT::TBPing; 
     868                    $terms->{junk_status} = MT::TBPing::NOT_JUNK(); 
    866869                    $terms->{visible} = 0; 
    867870                }, 
    …… 
    872875                handler => sub { 
    873876                    my ( $terms, $args ) = @_; 
    874                     $terms->{junk_status} = -1; 
     877                    require MT::TBPing; 
     878                    $terms->{junk_status} = MT::TBPing::NOT_JUNK(); 
    875879                }, 
    876880            }, 
    …… 
    884888                    $terms->{created_on} = [ $ts, undef ]; 
    885889                    $args->{range_incl}{created_on} = 1; 
    886                     $terms->{junk_status} = [ 0, 1 ]; 
     890                    $terms->{junk_status} = MT::TBPing::NOT_JUNK(); 
    887891                }, 
    888892            }, 
    …… 
    894898                handler => sub { 
    895899                    my ( $terms, $args ) = @_; 
    896                     $terms->{junk_status} = [ 0, 1 ]; 
     900                    require MT::Comment; 
     901                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
    897902                }, 
    898903            }, 
    …… 
    903908                    my ( $terms, $args ) = @_; 
    904909                    require MT::Entry; 
     910                    require MT::Comment; 
    905911                    my $app = MT->instance; 
    906                     $terms->{junk_status} = [ 0, 1 ]; 
     912                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
    907913                    # This join syntax employs a hack that allows us 
    908914                    # to do joins on abitrary columns. Typically, 
    …… 
    932938                handler => sub { 
    933939                    my ( $terms, $args ) = @_; 
    934                     $terms->{junk_status} = [ 0, 1 ]; 
     940                    require MT::Comment; 
     941                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
    935942                    $terms->{visible} = 0; 
    936943                }, 
    …… 
    941948                handler => sub { 
    942949                    my ( $terms, $args ) = @_; 
    943                     $terms->{junk_status} = -1; 
     950                    require MT::Comment; 
     951                    $terms->{junk_status} = MT::Comment::JUNK(); 
    944952                }, 
    945953            }, 
    …… 
    952960                }, 
    953961            }, 
    954             #            my_comments => { 
    955             #                label   => 'My comments', 
    956             #                order   => 600, 
    957             #                handler => sub { 
    958             #                    my ( $terms, $args ) = @_; 
    959             #                    $terms->{commenter_id} = $app->user->id; 
    960             #                }, 
    961             #            }, 
    962962            last_7_days => { 
    963963                label   => 'Comments in the last 7 days', 
    …… 
    965965                handler => sub { 
    966966                    my ( $terms, $args ) = @_; 
     967                    require MT::Comment; 
    967968                    my $ts = time - 7 * 24 * 60 * 60; 
    968969                    $ts = epoch2ts( MT->app->blog, $ts ); 
    969970                    $terms->{created_on} = [ $ts, undef ]; 
    970971                    $args->{range_incl}{created_on} = 1; 
    971                     $terms->{junk_status} = [ 0, 1 ]; 
    972                 }, 
    973             }, 
    974             #           last_24_hours => { 
    975             #               label   => 'All comments in the last 24 hours', 
    976             #               order   => 800, 
    977             #               handler => sub { 
    978             #                   my ( $terms, $args ) = @_; 
    979             #                   my $ts = time - 24 * 60 * 60; 
    980             #                   $ts = epoch2ts( MT->app->blog, $ts ); 
    981             #                   $terms->{created_on} = [ $ts, undef ]; 
    982             #                   $args->{range_incl}{created_on} = 1; 
    983             #                   $terms->{junk_status} = [ 0, 1 ]; 
    984             #               }, 
    985             #           }, 
     972                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
     973                }, 
     974            }, 
    986975            _comments_by_user => { 
    987976                label => sub { 
    …… 
    10191008                handler => sub { 
    10201009                    my ( $terms, $args ) = @_; 
     1010                    require MT::Comment; 
    10211011                    my $entry_id = int( MT->app->param('filter_val') ); 
    10221012                    $terms->{entry_id} = $entry_id; 
    1023                     $terms->{junk_status} = [ 0, 1 ]; 
     1013                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
    10241014                }, 
    10251015            }, 
    …… 
    10571047                handler => sub { 
    10581048                    my ( $terms, $args ) = @_; 
     1049                    require MT::Comment; 
    10591050                    my $val = $app->param('filter_val'); 
    10601051                    my ( $from, $to ) = split /-/, $val; 
    …… 
    10631054                    $to = undef unless $to =~ m/^\d{8}$/; 
    10641055                    $to .= '235959'; 
    1065                     $terms->{junk_status} = [ 0, 1 ]; 
     1056                    $terms->{junk_status} = MT::Comment::NOT_JUNK(); 
    10661057                    $terms->{created_on} = 
    10671058                      [ MT::Object::ts2db($from), MT::Object::ts2db($to) ]; 
    …… 
    33093300        my $tb_count_iter = 
    33103301          MT::TBPing->count_group_by( 
    3311             { blog_id => $blog_id, junk_status => [ 0, 1 ] }, 
     3302            { blog_id => $blog_id, junk_status => MT::TBPing::NOT_JUNK() }, 
    33123303            { group   => ['tb_id'] } ); 
    33133304        while ( my ( $count, $tb_id ) = $tb_count_iter->() ) { 
    …… 
    33903381                    { 
    33913382                        tb_id       => $tb->id, 
    3392                         junk_status => [ 0, 1 ] 
     3383                        junk_status => MT::TBPing::NOT_JUNK(), 
    33933384                    } 
    33943385                  ); 

  • branches/release-34/lib/MT/App/Trackback.pm

    r1823 r1866  
    306306        $ping->source_url($url); 
    307307        $ping->ip( $app->remote_ip || '' ); 
    308         $ping->junk_status(0); 
    309308        $ping->visible(1); 
    310309    } 
    …… 
    594593        ); 
    595594    } 
    596     $arg{not} = { junk_status => 1 }; 
    597595    my $iter = MT::TBPing->load_iter( 
    598596      { 
    599597        tb_id       => $tb->id, 
    600         junk_status => -1, 
     598        junk_status => MT::TBPing::NOT_JUNK(), 
    601599        visible     => 1 
    602600      }, 

  • branches/release-34/lib/MT/CMS/Blog.pm

    r1823 r1866  
    17771777            ? $comment_count->{$blog_id} 
    17781778            : $comment_count->{$blog_id} = MT::Comment->count( 
    1779                 { blog_id => $blog_id, junk_status => [ 0, 1 ] } 
     1779                { blog_id => $blog_id, junk_status => MT::Comment::NOT_JUNK() } 
    17801780            ) 
    17811781          ) 
    …… 
    17841784            $ping_count ? $ping_count->{$blog_id} : $ping_count->{$blog_id} = 
    17851785              MT::TBPing->count( 
    1786                 { blog_id => $blog_id, junk_status => [ 0, 1 ] } 
     1786                { blog_id => $blog_id, junk_status => MT::TBPing::NOT_JUNK() } 
    17871787              ) 
    17881788        ) || 0; 

  • branches/release-34/lib/MT/CMS/Comment.pm

    r1823 r1866  
    273273        if ( $filter_col eq 'status' ) { 
    274274            if ( $val eq 'approved' ) { 
    275                 $terms{junk_status} = 1; 
     275                $terms{visible} = 1; 
    276276            } 
    277277            elsif ( $val eq 'pending' ) { 
    278                 $terms{junk_status} = 0; 
     278                $terms{visible} = 0; 
    279279            } 
    280280            elsif ( $val eq 'junk' ) { 
    281                 $terms{junk_status} = -1; 
     281                $terms{junk_status} = MT::Comment::JUNK(); 
    282282            } 
    283283            else { 
    284                 $terms{junk_status} = [ 0, 1 ]; 
     284                $terms{junk_status} = MT::Comment::NOT_JUNK(); 
    285285            } 
    286286        } 
    …… 
    934934    my $class = $app->model($type); 
    935935    my $arg   = {}; 
    936     $arg->{junk_status} = -1; 
     936    require MT::Comment; 
     937    $arg->{junk_status} = MT::Comment::JUNK(); 
    937938    $arg->{blog_id} = $blog_id if $blog_id; 
    938939    $class->remove($arg); 
    …… 
    14461447    elsif ( $status eq 'moderate' ) { 
    14471448        $obj->moderate; 
    1448         $obj->junk_status(0); 
    14491449    } 
    14501450    elsif ( $status eq 'junk' ) { 

  • branches/release-34/lib/MT/CMS/Search.pm

    r1823 r1866  
    412412        } 
    413413        if ( $class->has_column('junk_status') ) { 
     414            require MT::Comment; 
    414415            if ($is_junk) { 
    415                 $terms{junk_status} = -1; 
     416                $terms{junk_status} = MT::Comment::JUNK(); 
    416417            } 
    417418            else { 
    418                 $terms{junk_status} = [ 0, 1 ]; 
     419                $terms{junk_status} = MT::Comment::NOT_JUNK(); 
    419420            } 
    420421        } 

  • branches/release-34/lib/MT/CMS/TrackBack.pm

    r1823 r1866  
    152152    ## entries. 
    153153    my %arg; 
     154    require MT::TBPing; 
    154155    if ( ( $app->param('tab') || '' ) eq 'junk' ) { 
    155156        $app->param( 'filter',     'junk_status' ); 
    156         $app->param( 'filter_val', '-1' ); 
     157        $app->param( 'filter_val', MT::TBPing::JUNK() ); 
    157158        $param{filter_special} = 1; 
    158159        $param{filter_phrase}  = $app->translate('Junk TrackBacks'); 
    159160    } 
    160161    else { 
    161         $terms{'junk_status'} = [ 0, 1 ]; 
     162        $terms{'junk_status'} = MT::TBPing::NOT_JUNK(); 
    162163    } 
    163164 
    …… 
    473474    elsif ( $status eq 'moderate' ) { 
    474475        $obj->moderate; 
    475         $obj->junk_status(0); 
    476476    } 
    477477    elsif ( $status eq 'junk' ) { 

  • branches/release-34/lib/MT/Comment.pm

    r1578 r1866  
    1010use base qw( MT::Object MT::Scorable ); 
    1111use MT::Util qw( weaken ); 
     12 
     13sub JUNK ()     { -1 } 
     14sub NOT_JUNK () { 1 } 
    1215 
    1316__PACKAGE__->install_properties({ 
    …… 
    5558    }, 
    5659    defaults => { 
    57         junk_status => 0, 
     60        junk_status => NOT_JUNK, 
    5861        last_moved_on => '20000101000000', 
    5962    }, 
    …… 
    6265    primary_key => 'id', 
    6366}); 
    64  
    65 sub JUNK ()     { -1 } 
    66 sub NOT_JUNK () { 1 } 
    6767 
    6868my %blocklists = (); 

  • branches/release-34/lib/MT/TBPing.pm

    r1578 r1866  
    1010use base qw( MT::Object MT::Scorable ); 
    1111 
    12 use constant JUNK => -1; 
    13 use constant NOT_JUNK => 1; 
     12sub JUNK()      { -1 } 
     13sub NOT_JUNK () {  1 } 
    1414 
    1515__PACKAGE__->install_properties({ 
    …… 
    5252    }, 
    5353    defaults => { 
    54         junk_status => 0, 
     54        junk_status => NOT_JUNK, 
    5555        last_moved_on => '20000101000000', 
    5656    }, 

  • branches/release-34/lib/MT/Upgrade.pm

    r1806 r1866  
    463463        }, 
    464464        'core_init_comment_junk_status' => { 
    465             on_field => 'MT::Comment->junk_status', 
     465            version_limit => 4.0053, 
    466466            priority => 3.1, 
    467467            updater => { 
    468468                type => 'comment', 
    469                 condition => sub { !defined $_[0]->junk_status }, 
     469                condition => sub { !$_[0]->junk_status }, 
    470470                code => sub { $_[0]->junk_status(1) }, 
    471471                label => 'Assigning junk status for comments...', 
    472472                sql => 'update mt_comment set comment_junk_status = 1 
    473                         where comment_junk_status is null', 
     473                        where comment_junk_status is null 
     474                           or comment_junk_status=0', 
    474475            } 
    475476        }, 
    …… 
    487488        }, 
    488489        'core_init_tbping_junk_status' => { 
    489             on_field => 'MT::TBPing->junk_status', 
     490            version_limit => 4.0053, 
    490491            priority => 3.1, 
    491492            updater => { 
    492493                type => 'tbping', 
    493                 condition => sub { !defined $_[0]->junk_status }, 
     494                condition => sub { !$_[0]->junk_status }, 
    494495                code => sub { $_[0]->junk_status(1) }, 
    495496                label => 'Assigning junk status for TrackBacks...', 
    496497                sql => 'update mt_tbping set tbping_junk_status = 1 
    497                         where tbping_junk_status is null', 
     498                        where tbping_junk_status is null 
     499                          or tbping_junk_status=0', 
    498500            } 
    499501        },