WordPress.org
Get WordPress

Plugin Directory

Changeset 900447


Ignore:
Timestamp:
04/22/2014 08:18:36 PM (12 years ago)
Author:
mobius5150
Message:

Tagging version 0.7 from GitHub and updating stable tag.

Location:
vip-scanner/trunk
Files:
6 added
3 deleted
16 edited

Legend:

Unmodified
Added
Removed

  • vip-scanner/trunk/css/vip-scanner.css

    r794835 r900447  
    125125    font-weight: 600;
    126126}
     127
     128/* Analyzer */
     129#analysis ul ul {
     130    margin-left: 30px;
     131}

  • vip-scanner/trunk/readme.md

    r883100 r900447  
    3030Changelog
    3131---------
     32
     33__0.7__
     34
     35* Modified analyzer to use PHP tokens rather than regular expressions
     36* New checks, including white/blacklist checking for file types and names
     37* Added basic async scanning as an admin bar node
     38* WP CLI scan commands now support paths in addition to theme slugs
     39* WP CLI `scan_type` argument is now optional
    3240
    3341__0.6__

  • vip-scanner/trunk/readme.txt

    r883100 r900447  
    44Requires at least: 3.4
    55Tested up to: 3.9
    6 Stable tag: 0.6
     6Stable tag: 0.7
    77
    88Scan all sorts of themes and files and things.
    … …  
    3535
    3636== Changelog ==
     37
     38= 0.7 =
     39
     40* Modified analyzer to use PHP tokens rather than regular expressions
     41* New checks, including white/blacklist checking for file types and names
     42* Added basic async scanning as an admin bar node
     43* WP CLI scan commands now support paths in addition to theme slugs
     44* WP CLI `scan_type` argument is now optional
    3745
    3846= 0.6 =

  • vip-scanner/trunk/vip-scanner.php

    r883100 r900447  
    55Description: Easy to use UI for the VIP Scanner.
    66Author: Automattic (Original code by Pross, Otto42, and Thorsten Ott)
    7 Version: 0.6
     7Version: 0.7
    88
    99License: GPLv2
    … …  
    1515
    1616class VIP_Scanner_UI {
    17     const key = 'vip-scanner';
     17    const   key      = 'vip-scanner';
     18    private $version = null;
    1819
    1920    public $default_review;
    … …  
    7374        }
    7475        return self::$instance;
     76    }
     77
     78    function get_version() {
     79        if ( is_null( $this->version ) ) {
     80            // Load plugin version from plugin data
     81            $plugin_data = get_plugin_data( __FILE__ );
     82            $this->version = $plugin_data['Version'];
     83        }
     84
     85        return $this->version;
    7586    }
    7687
    … …  
    135146        $scanner = VIP_Scanner::get_instance()->run_theme_review( $theme, $review );
    136147
    137         $transient_key = 'vip_scanner_' . md5( $theme . $review );
     148        $transient_key = 'vip_scanner_' . $this->get_version() . '_' . md5( $theme . $review );
    138149        if ( $scanner !== get_transient( $transient_key ) )
    139150            @set_transient( $transient_key, $scanner );
    … …  
    337348
    338349        $results .= $title = apply_filters( 'vip_scanner_export_title', "$theme - $review", $review ) . PHP_EOL;
    339         $results .= str_repeat( '=', strlen( $title ) ) . PHP_EOL . PHP_EOL;
     350        $title_len = strlen( $title );
     351        $results .= str_repeat( '=', $title_len ) . PHP_EOL;
     352
     353        $version_str = ' ' . sprintf( __( 'VIP Scanner %s', 'theme-check' ), $this->get_version() ) . ' ';
     354        $side_spacing = ( $title_len - strlen( $version_str ) ) / 2.;
     355        $results .= str_repeat( '=', ceil( $side_spacing ) ) . $version_str . str_repeat( '=', floor( $side_spacing ) ) . PHP_EOL;
     356        $results .= str_repeat( '=', $title_len ) . PHP_EOL . PHP_EOL;
    340357
    341358        $form_results = apply_filters( 'vip_scanner_form_results', '', $review );
    … …  
    429446
    430447    function get_cached_theme_review( $theme, $review ) {
    431         $transient_key = 'vip_scanner_' . md5( $theme . $review );
     448        $transient_key = 'vip_scanner_' . $this->get_version() . '_' . md5( $theme . $review );
    432449
    433450        if ( false === $scanner = get_transient( $transient_key ) ) {
    … …  
    496513            // redirect with error message
    497514            if ( !$zip )
    498                 break;
     515                return;
    499516
    500517            $mail = wp_mail(

  • vip-scanner/trunk/vip-scanner/analyzers/CustomResourceAnalyzer.php

    r883100 r900447  
    1212    protected $resource_types = array(
    1313        array(
    14             'func_name' => 'apply_filters',
     14            'func_name' => array( 'apply_filters' ),
    1515            'plural'    => 'filters',
    1616            'singular'  => 'filter',
    … …  
    1818       
    1919        array(
    20             'func_name' => 'do_action',
     20            'func_name' => array( 'do_action' ),
    2121            'plural'    => 'actions',
    2222            'singular'  => 'action',
    … …  
    2424       
    2525        array(
    26             'func_name' => '->add_cap',
     26            'func_name' => array( '->add_cap' ),
    2727            'plural'    => 'capabilities',
    2828            'singular'  => 'capability',
    … …  
    3131       
    3232        array(
    33             'func_name' => 'add_role',
     33            'func_name' => array( 'add_role' ),
    3434            'plural'    => 'roles',
    3535            'singular'  => 'role',
    … …  
    3737       
    3838        array(
    39             'func_name' => 'add_shortcode',
     39            'func_name' => array( 'add_shortcode' ),
    4040            'plural'    => 'shortcodes',
    4141            'singular'  => 'shortcode',
    … …  
    4343       
    4444        array(
    45             'func_name' => 'register_post_type',
     45            'func_name' => array( 'register_post_type' ),
    4646            'plural'    => 'custom post types',
    4747            'singular'  => 'custom post type',
    … …  
    4949       
    5050        array(
    51             'func_name' => 'register_taxonomy',
     51            'func_name' => array( 'register_taxonomy' ),
    5252            'plural'    => 'taxonomies',
    5353            'singular'  => 'taxonomy',
    … …  
    8989            if ( !array_key_exists( $filepath, $file_metas ) ) {
    9090                // This is not a file we can handle
    91                 var_dump( "Not scanning file: {$file->get_filepath()}: " . $file_metas[$filepath]->get_file()->get_filepath());
    9291                continue;
    9392            }
    … …  
    108107     */
    109108    public function scan_file( $file, $file_renderer ) {
    110         $file_functions = $file->get_code_elements( 'functions' );
    111        
     109        $function_calls = $file->get_code_elements( 'function_calls' );
     110
    112111        foreach ( $this->resource_types as $resource ) {
    113             $regexes = array();
    114                
    115             if ( is_array( $resource['func_name'] ) ) {
    116                 foreach ( $resource['func_name'] as $func_name ) {
    117                     $regexes[] = "/{$func_name}\s*\(\s*(?<name>([a-zA-Z0-9_'\".$-]|\s*)+)/ix";
    118                 }
    119             } else {
    120                 $regexes[] = "/{$resource['func_name']}\s*\(\s*(?<name>([a-zA-Z0-9_'\".$-]|\s*)+)/ix";
    121             }
    122                
    123             if ( isset( $resource['regexes'] ) ) {
    124                 $regexes = array_merge( $regexes, $resource['regexes'] );
    125             }
    126            
    127             foreach ( $regexes as $regex ) {
    128                 foreach ( $file_functions as $functions ) {
    129                     // Scan the functions in the file
    130                     $phpelements = $file->get_code_elements( 'php' );
    131                     $code_blocks_to_scan = array_merge( $functions, $phpelements[''] );
     112            foreach ( $resource['func_name'] as $function_name ) {
     113                foreach ( $function_calls as $call_path => $functions ) {
     114                    // check and see if this function was called
     115                    if ( array_key_exists( $function_name, $functions ) ) {
     116                        if ( isset( $functions[$function_name]['args'] ) ) {
     117                            $calls = array( $functions[$function_name] );
     118                        } else {
     119                            $calls = $functions[$function_name];
     120                        }
    132121
    133                     foreach ( $code_blocks_to_scan as $code_block ) {
    134                         preg_match_all( $regex, $code_block['contents'], $matches, PREG_OFFSET_CAPTURE );
    135                         foreach ( $matches['name'] as $match ) {
    136                             $match = str_replace( $this->remove_chars, '', $match );
    137                             $child_renderer = $this->create_child_renderer_from_match( $match, $code_block['contents'], $resource, $file, $code_block['line'] );
     122                        foreach( $calls as $call ) {
     123                            $child_renderer = new ResourceRenderer( str_replace( $this->remove_chars, '', $call['args'][0] ) );
     124                            $child_renderer->set_resource_type( $resource['singular'], $resource['plural'] );
     125                            $child_renderer->add_attribute( 'file', $file->get_filename() );
     126                            $child_renderer->add_attribute( 'args', $call['args'] );
     127
    138128                            $file_renderer->add_child( $child_renderer );
    139129                            $this->renderers[$resource['plural']]->add_child( $child_renderer );
    … …  
    144134        }
    145135    }
    146    
    147     public function create_child_renderer_from_match( $match, $contents, $resource, $file, $line_offset = 0 ) {
    148         $child_renderer = new ResourceRenderer( $match[0] );
    149         $child_renderer->set_resource_type( $resource['singular'], $resource['plural'] );
    150         $child_renderer->add_attribute( 'line', $file->compute_line_number( $contents, $match[1], $line_offset ) );
    151         $child_renderer->add_attribute( 'file', $file->get_filename() );
    152         return $child_renderer;
    153     }
    154136}

  • vip-scanner/trunk/vip-scanner/analyzers/PHPAnalyzer.php

    r883100 r900447  
    88        'functions'  => 'FunctionRenderer',
    99    );
    10    
     10
     11    protected $check_hierarchy = array(
     12        'php' => array(
     13            'namespaces' => array(
     14                'classes' => array(
     15                    'functions' => array(),
     16                    'members'   => array(),
     17                ),
     18            ),
     19
     20            'classes' => array(
     21                'functions' => array(),
     22                'members'   => array(),
     23            ),
     24
     25            'functions' => array(),
     26            'members'   => array(),
     27        ),
     28    );
     29
    1130    function __construct() {
    1231        $this->renderers = array(
    … …  
    3150                continue;
    3251            }
    33            
     52
    3453            $file_meta = new FileRenderer( $file );
    3554            $this->add_renderers( $file, $file_meta );
    … …  
    4867    public function add_renderers( $file, &$renderer, $path = '', $hierarchy = null ) {
    4968        if ( is_null( $hierarchy ) ) {
    50             $hierarchy = $file->get_check_hierarchy();
     69            $hierarchy = $this->check_hierarchy;
    5170        }
    5271
    5372        foreach ( $hierarchy as $level => $hierarchy_children ) {
    5473            $code_elements = $file->get_code_elements( $level, $path );
    55            
    5674            if ( empty( $code_elements ) ) {
    5775                $this->add_renderers( $file, $renderer, $path, $hierarchy_children );
    … …  
    5977            } else {
    6078                foreach ( $code_elements as $child_path => $child_element ) {
    61                    
    6279                    if ( array_key_exists( $level, $this->hierarchy_metas ) ) {
    6380                        $child_meta = new $this->hierarchy_metas[$level]( $child_path );

  • vip-scanner/trunk/vip-scanner/checks/VCMergeConflictCheck.php

    r883100 r900447  
    33class VCMergeConflictCheck extends BaseCheck {
    44    protected $checks = array(
     5        // Check for the signs of a merge conflict IE:
     6        // <<<<<< YOUR_SIDE
     7        // /*some stuff*/
     8        // ======
     9        // /*some other stuff*/
     10        // >>>>>> THEIR_SIDE
    511        'merge-conflict' => '/(<{4,}\W+(?<your_side>\w+)[\s\S]*>{4,}\W+(?<their_side>\w+))/im',
     12
     13        // Matches the filename of a conflict file created by SVN. Eg: test.php.mine, test.php.r10, test.php.r100
    614        'conflict_file'  => '/mine|r[0-9]+/im'
    715    );

  • vip-scanner/trunk/vip-scanner/checks/VIPRestrictedPatternsCheck.php

    r883100 r900447  
    88        $checks = array(
    99            "/(\\\$isIE)+/msiU" => array( "level" => "Warning", "note" => 'Using $isIE conflicts with full page caching' ),
     10            "/WordPress VIP/msiU" => array( "level" => "Warning", "note" => 'Please use "WordPress.com VIP" rather than "WordPress VIP"' ),
    1011            "/(kses)+/msiU" => array ( "level" => "Warning", "note" => "Working with kses" ),
    1112            "/(\\\$wpdb->|mysql_)+.+(ALTER)+\s+/msiU" => array( "level" => "Blocker", "note" => "Possible database table alteration" ),

  • vip-scanner/trunk/vip-scanner/class-analyzed-php-file.php

    r883100 r900447  
    55    protected $filecontents = '';
    66    protected $processed_file_contents = '';
    7    
    8     protected $hierarchy_elements = array(
    9         'namespaces' => array(),
    10         'classes'    => array(),
    11         'functions'  => array(),
    12         'php'        => array(),
    13     );
    14    
    15     protected $comments_regex = <<<EOT
    16         (\/\*(?:(?!\*\/)[\s\S])*\*\/)           # match a multiline comment
    17 EOT
    18         ;
    197
    20     protected $single_comment_regex = <<<EOT
    21         (\/\/.*$)           # match a single line comment
    22 EOT;
    23    
    24     protected $heredox_regex = <<<EOT
    25         <<<(?<herestart>\S+)((?!\1)[\s\S])*\3;  # match a heredoc
    26 EOT
    27         ;
    28 
    29     protected $strip_inline_php_regex = '\?>((?!<\?php)[\s\S])*<\?php';
    30    
    31     protected $hierarchy_regexes = array(
    32         'php'        => array( 'regex' => <<<EOT
    33             (                               # start of bracket 1
    34                 <\?php                          # php opening tag
    35                     (?<contents>((?!\?>)[\s\S])*)   # match anything except a php closing tag
    36                 (\?>)?                          # match a closing tag
    37             )                               # end of bracket 1
    38 EOT
    39             ),
    40 
    41         'namespaces' => array( 'regex' => <<<EOT
    42             namespace\s+(?<name>(\\\\?\w+)+);    # Match the name of the namespace
    43             (?<contents>((?!namespace)[\s\S])*)  # match the contents of the namespace
    44 EOT
    45             , ),
    46 
    47         'classes'    => array( 'regex' => <<<EOT
    48             ((?<abstract>abstract)\s+)? # optionally match an abstract class
    49             class\s+(?<name>\w+)\s+ # match the classname
    50             (extends\s+(?<parentclass>\w+)\s*)? # optionally match a parentclass
    51             (?<contents>
    52                 (                       # start of bracket 7
    53                     {                   # match an opening curly bracket
    54                         (?:
    55                             [^{}]++     # one or more non curly brackets
    56                               |
    57                             (?7)        # recurse to bracket 7
    58                         )*
    59                     }                   # match a closing curly bracket
    60                 )                       # end of bracket 7
    61             )
    62 EOT
    63            , ),
    64 
    65         'functions'  => array( 'regex' => <<<EOT
    66             \s*(                                    # match function modifiers (visibility, static, abstract)
    67                     \s*(?<visibility>private|protected|public)\s+
    68                         |
    69                     \s*(?<static>static)\s+
    70                         |
    71                     \s*(?<abstract>abstract)\s+
    72                 ){0,3}
    73             \s*function\s+(?<name>[a-zA-Z0-9_]+\s*) # match the function definition & name
    74             \((?<args>(\s|\w|[$,_='"])+)?\)\s*      # match the function arguments
    75             (?<contents>
    76                 (?(abstract);|                          # match either the semicolon of an abstract function or a closure
    77                     (                                   # start of bracket 1
    78                         {                               # match an opening curly bracket
    79                             (?:
    80                                 [^{}]++                 # one or more non curly brackets
    81                                   |
    82                                 (?8)                    # recurse to bracket 1
    83                             )*
    84                         }                               # match a closing curly bracket
    85                     )                                   # end of bracket 1
    86                 )
    87             )
    88 EOT
    89             , ),
    90 
    91         'members'    => array( 'regex' => '', ),
    92     );
    93    
    94     protected $check_hierarchy = array(
    95         'php' => array(
    96             'namespaces' => array(
    97                 'classes' => array(
    98                     'functions' => array(),
    99                     'members'   => array(),
    100                 ),
    101             ),
    102 
    103             'classes' => array(
    104                 'functions' => array(),
    105                 'members'   => array(),
    106             ),
    107 
    108             'functions' => array(),
    109             'members'   => array(),
    110         ),
    111     );
    112    
    1138    /**
    1149     * Analyzes this file.
    … …  
    12318            return;
    12419        }
    125        
    126         // Strip strings and comments from the file. Preserve line numbers
    127         $stripped = $this->strip_strings_and_comments( $this->filecontents );
    128        
    129         // Do the php check hierarchy
    130         $this->processed_file_contents = $this->do_check_hierarchy( '', $this->check_hierarchy, $stripped, 0 );
     20
     21        // Parse the tokens
     22        require_once( 'class-token-parser.php' );
     23        $parser = new TokenParser();
     24        $items = $parser->parse_contents( $this->filecontents );
     25
     26        // Parse the items
     27        $this->hierarchy_elements = array();
     28        $this->parse_token_results( $items );
    13129    }
    13230
    13331    protected function get_strings_and_comments_regexes() {
    134         return array(
    135             $this->single_comment_regex,
    136             $this->comments_regex,
    137             $this->heredox_regex,
    138             $this->strip_inline_php_regex,
    139         );
     32        return array();
    14033    }
    14134
     35    private function parse_token_results( $items ) {
     36        foreach ( $items as $item ) {
     37            $type = '';
     38            switch ( $item['type'] ) {
     39                case 'class':
     40                    $type = 'classes';
     41                    break;
     42
     43                case 'const':
     44                    $type = 'constants';
     45                    break;
     46
     47                default:
     48                    $type = $item['type'] . 's';
     49            }
     50
     51            if ( !isset( $this->hierarchy_elements[$type] ) ) {
     52                $this->hierarchy_elements[$type] = array();
     53            }
     54
     55            if ( !isset( $this->hierarchy_elements[$type][$item['path']] ) ) {
     56                $this->hierarchy_elements[$type][$item['path']] = array();
     57            }
     58
     59            // There's a chance for duplicate items that are significant. Ie: two calls to one function within a block of code.
     60            if ( isset( $this->hierarchy_elements[$type][$item['path']][$item['name']] ) ) {
     61                if ( isset( $this->hierarchy_elements[$type][$item['path']][$item['name']][0] ) ) {
     62                    $this->hierarchy_elements[$type][$item['path']][$item['name']][] = $item;
     63                } else {
     64                    $this->hierarchy_elements[$type][$item['path']][$item['name']] = array(
     65                        $this->hierarchy_elements[$type][$item['path']][$item['name']],
     66                        $item,
     67                    );
     68                }
     69            } else {
     70                $this->hierarchy_elements[$type][$item['path']][$item['name']] = $item;
     71            }
     72
     73            if ( !empty( $item['children'] ) ) {
     74                $this->parse_token_results( $item['children'] );
     75            }
     76        }
     77    }
    14278}

  • vip-scanner/trunk/vip-scanner/class-analyzer-renderer.php

    r883100 r900447  
    186186                foreach ( $this->attributes as $slug => $attribute ) {
    187187                    if ( ! in_array( $slug, $skip_attributes ) && ! empty( $attribute ) ) {
    188                         $output .= sprintf( '<li><strong>%s</strong>: %s</li>', esc_html( $slug ), esc_html( $attribute ) );
     188                        $output .= $this->display_html_attribute( $slug, $attribute, $args );
    189189                    }
    190190                }
    … …  
    193193                foreach ( $this->attributes as $slug => $attribute ) {
    194194                    if ( ! in_array( $slug, $skip_attributes ) && ! empty( $attribute ) ) {
    195                         $output .= sprintf( "%s> %s: %s\n", str_repeat( $this->spacing_char, $args['level'] ), $slug, $attribute );
     195                        if ( is_string( $attribute ) ) {
     196                            $output .= sprintf( "%s> %s: %s\n", str_repeat( $this->spacing_char, $args['level'] ), $slug, $attribute );
     197                        } elseif ( is_numeric( $attribute ) ) {
     198                            $output .= sprintf( "%s> %s: %s\n", str_repeat( $this->spacing_char, $args['level'] ), $slug, number_format( $attribute ) );
     199                        } elseif ( is_array( $attribute ) ) {
     200                            $output .= sprintf( "%s> %s: %s\n", str_repeat( $this->spacing_char, $args['level'] ), $slug, implode( ', ', $attribute ) );
     201                        }
    196202                    }
    197203                }
    … …  
    199205        }
    200206        return $output;
     207    }
     208
     209    function display_html_attribute( $slug, $attribute, $args ) {
     210        $fstring = '<li><strong>%s</strong>: %s</li>';
     211        if ( is_string( $attribute ) ) {
     212            if ( substr_count( $attribute, "\n" ) ) {
     213                return sprintf( $fstring, esc_html( $slug ), '<pre>' . esc_html( $attribute ) . '</pre>' );
     214            } else {
     215                return sprintf( $fstring, esc_html( $slug ), esc_html( $attribute ) );
     216            }
     217        } elseif ( is_numeric( $attribute ) ) {
     218            return sprintf( $fstring, esc_html( $slug ), number_format( $attribute ) );
     219        } elseif ( is_bool( $attribute ) ) {
     220            return sprintf( $fstring, esc_html( $slug ), $attribute ? __( 'true', 'vip-scanner' ) : __( 'false', 'vip-scanner' ) );
     221        } elseif ( is_array( $attribute ) ) {
     222            $output = '';
     223            foreach ( $attribute as $key => $value ) {
     224                $output .= $this->display_html_attribute( $key, $value, $args );
     225            }
     226            return sprintf( $fstring, esc_html( $slug ), "<ul>$output</ul>" );
     227        }
    201228    }
    202229   
    … …  
    421448     */
    422449    protected function skip_attributes() {
    423         return array( 'contents', 'name' );
     450        return array( 'contents', 'name', 'children' );
    424451    }
    425452   

  • vip-scanner/trunk/vip-scanner/class-base-check.php

    r883100 r900447  
    214214        $scanner = $this->get_scanner();
    215215
    216         if ( ! $scanner instanceof ThemeScanner )
     216        if ( !is_subclass_of( $scanner, 'DirectoryScanner' ) )
    217217            return null;
    218218

  • vip-scanner/trunk/vip-scanner/class-base-scanner.php

    r883100 r900447  
    1313    public $renderers = array();
    1414    public $stats = array();
     15    //recognized extensions
    1516    public $known_extensions = array(
    1617        'php' => array( 'php', 'php5', 'inc' ),
    1718        'css' => 'css',
    1819        'js' => 'js',
     20        'gif' => 'gif',
     21        'jpg' => array( 'jpg', 'jpeg' ),
     22        'png' => 'png',
     23        'svg' => 'svg',
     24        'txt' => 'txt',
     25    );
     26    //these extensions are not allowed and will produce blocking errors
     27    public $known_bad_extensions = array(
     28        'gz',
     29        'zip',
     30        'tar',
     31        'orig',
     32        'rej',
     33        'bak',
     34        'log',
     35        'git',
     36        'asp',
     37        'py',
     38        'cfm',
     39        'htaccess',
     40        'swf'
     41    );
     42    //these patterns are not allowed anywhere in any filename and will produce blocking errors
     43    public $known_bad_file_patterns = array(
     44        '\.php\..*',
     45        '^\.DS_Store$',
     46        '^Thumbs.db$',
     47        '^WS_FTP.*',
    1948    );
    2049
    … …  
    6493    }
    6594
     95    public function is_bad_file_type( $filename ) {
     96        return in_array( $this->get_file_type( $filename ), $this->known_bad_extensions );
     97    }
     98
     99    public function has_bad_file_pattern( $filename ) {
     100        foreach ( $this->known_bad_file_patterns as $pattern ) {
     101            $pattern = '/' . $pattern . '/i';
     102            if ( 1 === preg_match( $pattern, $filename ) ) {
     103                return true;
     104            }
     105        }
     106    }
     107
     108    public function check_filename( $filename, $type ) {
     109        if ( $this->has_bad_file_pattern( basename( $filename ) ) ) {
     110            $this->add_error(
     111                'badfile-error',
     112                'bad file in theme',
     113                'Blocker',
     114                basename( $filename )
     115            );
     116            return false;
     117        }
     118
     119        if ( $this->is_bad_file_type( $filename ) ) {
     120            $this->add_error(
     121                'filetype-error',
     122                'File type ' . $type . ' not permitted',
     123                'Blocker',
     124                basename( $filename )
     125            );
     126            return false;
     127        }
     128
     129        if ( !$this->is_known_file_type( $filename ) ) {
     130            $this->add_error(
     131                'filetype-error',
     132                'File type ' . $type . ' detected',
     133                'Warning',
     134                basename( $filename )
     135            );
     136            return false;
     137        }
     138
     139        return true;
     140    }
     141
    66142    public function get_file_count() {
    67143        $count = 0;
    … …  
    81157            $file_type = $this->get_file_type( $filename );
    82158
    83             // If we only want to scan files of a certain type
    84             //if ( ! $this->is_known_file_type( $filename ) )
    85             //  continue;
     159            $this->check_filename( $filename, $file_type);
    86160
    87161            if( !isset( $grouped_files[$file_type] ) )
    … …  
    112186            $this->run_scanners( 'analyzers' );
    113187        }
    114        
     188
    115189        return $pass;
    116190    }
    … …  
    137211                $check_file = '';
    138212            }
     213
     214            if ( ! apply_filters( 'vip_scanner_run_check', true, $check ) ) {
     215                $this->add_error( 'skipped-check', sprintf( __( 'The "%s" check was skipped.', 'vip-scanner' ), $check ), BaseScanner::LEVEL_WARNING );
     216                continue;
     217            }
     218
    139219            $check_exists = $this->load_check( $check, $check_file, $type );
    140220

  • vip-scanner/trunk/vip-scanner/class-function-renderer.php

    r883100 r900447  
    2222
    2323        $args = $this->get_attribute( 'args' );
    24         $header_items[] = sprintf(
    25             'function %s%s',
    26             $this->stylize_text( esc_html( $this->name() ), array( 'bold' => true, 'classes' => array( 'renderer-function-name' ) ) ),
    27             $args === ';' ? ';' : '(' . esc_html( $args ) . ')'
    28         );
     24        if ( is_array( $args ) ) {
     25            $header_items[] = sprintf(
     26                'function %s%s',
     27                $this->stylize_text( esc_html( $this->name() ), array( 'bold' => true, 'classes' => array( 'renderer-function-name' ) ) ),
     28                $args === ';' ? ';' : '(' . esc_html( implode( ', ', $args ) ) . ')'
     29            );
     30        } else {
     31            $header_items[] = sprintf(
     32                'function %s%s',
     33                $this->stylize_text( esc_html( $this->name() ), array( 'bold' => true, 'classes' => array( 'renderer-function-name' ) ) ),
     34                $args === ';' ? ';' : '(' . esc_html( $args ) . ')'
     35            );
     36        }
    2937
    3038        return implode( ' ', $header_items );

  • vip-scanner/trunk/vip-scanner/class-theme-scanner.php

    r883100 r900447  
    77            return $this->add_error( 'wp-load', sprintf( '%s requires WordPress to be loaded.', get_class( $this ) ), 'blocker' );
    88
    9         // Get Theme Path
    10         $path = sprintf( '%s/%s', get_theme_root(), $theme );
     9        // decide whether to interpret theme as a path by checking if the path exists
     10        $potential_file_path = realpath( $theme );
     11        if ( $potential_file_path ) {
     12            $path = $potential_file_path;
     13        } else {
     14            $path = sprintf( '%s/%s', get_theme_root(), $theme );
     15        }
    1116
    1217        // Call Parent Constructor

  • vip-scanner/trunk/vip-scanner/class-wp-cli.php

    r883100 r900447  
    1515     *
    1616     * @subcommand scan-theme
    17      * @synopsis --theme=<theme-name> --scan_type=<scan-type> [--format=<format>] [--summary=<summary>]
     17     * @synopsis --theme=<theme-name> [--scan_type=<scan-type>] [--format=<format>] [--summary=<summary>]
    1818     */
    1919    public function scan_theme( $args, $assoc_args ) {
    … …  
    7777                    $lines = array();
    7878
    79                     // Not all errors have lines
    80                     if ( isset( $error['lines'] ) )
    81                         $lines = $error['lines'];
     79                    // Not all errors have lines -- assign a null line if we lack lines entirely
     80                    $lines =  ( isset( $error['lines'] ) ) ? $error['lines'] : array( '' );
    8281
    8382                    // In JSON output, group the lines together

  • vip-scanner/trunk/vip-scanner/vip-scanner.php

    r883100 r900447  
    2828require_once( VIP_SCANNER_DIR . '/class-function-renderer.php' );
    2929require_once( VIP_SCANNER_DIR . '/class-base-analyzer.php' );
     30
     31if ( is_admin() ) {
     32    require_once( VIP_SCANNER_DIR . '/class-async-directory-scanner.php' );
     33    require_once( VIP_SCANNER_DIR . '/vip-scanner-async.php' );
     34}
    3035
    3136class VIP_Scanner {
    … …  
    6368            return false;
    6469
     70        do_action( 'vip_scanner_pre_theme_review', $theme, $review_type );
     71
    6572        $scanner = new ThemeScanner( $theme, $review );
    6673        $scanner->scan( $scanners );
     74
     75        do_action( 'vip_scanner_post_theme_review', $theme, $review_type, $scanner );
    6776        return $scanner;
    6877    }
Note: See TracChangeset for help on using the changeset viewer.