Make WordPress Core

Changeset 14401


Ignore:
Timestamp:
05/03/2010 08:19:13 PM (15 years ago)
Author:
dd32
Message:

WP_Query whitespace cleanup, some loop cleanup as well

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-includes/query.php

    r14341 r14401  
    12051205        );
    12061206
    1207         foreach ($keys as $key) {
     1207        foreach ( $keys as $key ) {
    12081208            if ( !isset($array[$key]))
    12091209                $array[$key] = '';
     
    12281228     * @param string|array $query
    12291229     */
    1230     function parse_query($query) {
     1230    function parse_query($query ) {
    12311231        if ( !empty($query) || !isset($this->query) ) {
    12321232            $this->init();
     
    13021302
    13031303            if ( $qv['day'] ) {
    1304                 if (! $this->is_date) {
     1304                if ( ! $this->is_date ) {
    13051305                    $this->is_day = true;
    13061306                    $this->is_date = true;
     
    13091309
    13101310            if ( $qv['monthnum'] ) {
    1311                 if (! $this->is_date) {
     1311                if ( ! $this->is_date ) {
    13121312                    $this->is_month = true;
    13131313                    $this->is_date = true;
     
    13161316
    13171317            if ( $qv['year'] ) {
    1318                 if (! $this->is_date) {
     1318                if ( ! $this->is_date ) {
    13191319                    $this->is_year = true;
    13201320                    $this->is_date = true;
     
    13241324            if ( $qv['m'] ) {
    13251325                $this->is_date = true;
    1326                 if (strlen($qv['m']) > 9) {
     1326                if ( strlen($qv['m']) > 9 ) {
    13271327                    $this->is_time = true;
    1328                 } else if (strlen($qv['m']) > 7) {
     1328                } else if ( strlen($qv['m']) > 7 ) {
    13291329                    $this->is_day = true;
    1330                 } else if (strlen($qv['m']) > 5) {
     1330                } else if ( strlen($qv['m']) > 5 ) {
    13311331                    $this->is_month = true;
    13321332                } else {
     
    13351335            }
    13361336
    1337             if ('' != $qv['w']) {
     1337            if ( '' != $qv['w'] ) {
    13381338                $this->is_date = true;
    13391339            }
     
    13421342                $this->is_category = false;
    13431343            } else {
    1344                 if (strpos($qv['cat'], '-') !== false) {
     1344                if ( strpos($qv['cat'], '-') !== false ) {
    13451345                    $this->is_category = false;
    13461346                } else {
     
    14551455
    14561456        // if we're previewing inside the write screen
    1457         if ('' != $qv['preview'])
     1457        if ( '' != $qv['preview'] )
    14581458            $this->is_preview = true;
    14591459
     
    15111511        }
    15121512
    1513         if ( !empty($qv['post_type']) ) {
    1514             if(is_array($qv['post_type']))
     1513        if ( !empty($qv['post_type']) ) {
     1514            if ( is_array($qv['post_type']) )
    15151515                $qv['post_type'] = array_map('sanitize_user', $qv['post_type'], array(true));
    15161516            else
     
    15271527        // Done correcting is_* for page_on_front and page_for_posts
    15281528
    1529         if ('404' == $qv['error'])
     1529        if ( '404' == $qv['error'] )
    15301530            $this->set_404();
    15311531
     
    15461546        $this->is_404 = true;
    15471547
    1548         $this->is_feed = $is_feed;
     1548        //$this->is_feed = $is_feed;
    15491549    }
    15501550
     
    15591559     */
    15601560    function get($query_var) {
    1561         if (isset($this->query_vars[$query_var])) {
     1561        if ( isset($this->query_vars[$query_var]) )
    15621562            return $this->query_vars[$query_var];
    1563         }
    15641563
    15651564        return '';
     
    16401639            $q['posts_per_page'] = $q['posts_per_archive_page'];
    16411640        if ( !isset($q['nopaging']) ) {
    1642             if ($q['posts_per_page'] == -1) {
     1641            if ( $q['posts_per_page'] == -1 ) {
    16431642                $q['nopaging'] = true;
    16441643            } else {
     
    16651664        }
    16661665
    1667         if (isset($q['page'])) {
     1666        if ( isset($q['page']) ) {
    16681667            $q['page'] = trim($q['page'], '/');
    16691668            $q['page'] = absint($q['page']);
     
    16801679            $q['m'] = '' . preg_replace('|[^0-9]|', '', $q['m']);
    16811680            $where .= " AND YEAR($wpdb->posts.post_date)=" . substr($q['m'], 0, 4);
    1682             if (strlen($q['m'])>5)
     1681            if ( strlen($q['m']) > 5 )
    16831682                $where .= " AND MONTH($wpdb->posts.post_date)=" . substr($q['m'], 4, 2);
    1684             if (strlen($q['m'])>7)
     1683            if ( strlen($q['m']) > 7 )
    16851684                $where .= " AND DAYOFMONTH($wpdb->posts.post_date)=" . substr($q['m'], 6, 2);
    1686             if (strlen($q['m'])>9)
     1685            if ( strlen($q['m']) > 9 )
    16871686                $where .= " AND HOUR($wpdb->posts.post_date)=" . substr($q['m'], 8, 2);
    1688             if (strlen($q['m'])>11)
     1687            if ( strlen($q['m']) > 11 )
    16891688                $where .= " AND MINUTE($wpdb->posts.post_date)=" . substr($q['m'], 10, 2);
    1690             if (strlen($q['m'])>13)
     1689            if ( strlen($q['m']) > 13 )
    16911690                $where .= " AND SECOND($wpdb->posts.post_date)=" . substr($q['m'], 12, 2);
    16921691        }
     
    17171716                    continue;
    17181717
    1719                 if ( ! $ptype_obj->hierarchical || strpos($q[ $ptype_obj->query_var ], '/') === false) {
     1718                if ( ! $ptype_obj->hierarchical || strpos($q[ $ptype_obj->query_var ], '/') === false ) {
    17201719                    // Non-hierarchical post_types & parent-level-hierarchical post_types can directly use 'name'
    17211720                    $q['name'] = $q[ $ptype_obj->query_var ];
     
    17741773                }
    17751774            }
    1776         } elseif ('' != $q['attachment']) {
     1775        } elseif ( '' != $q['attachment'] ) {
    17771776            $q['attachment'] = str_replace('%2F', '/', urlencode(urldecode($q['attachment'])));
    17781777            $attach_paths = '/' . trim($q['attachment'], '/');
     
    18251824            $n = !empty($q['exact']) ? '' : '%';
    18261825            $searchand = '';
    1827             foreach( (array) $q['search_terms'] as $term) {
     1826            foreach( (array) $q['search_terms'] as $term ) {
    18281827                $term = addslashes_gpc($term);
    18291828                $search .= "{$searchand}(($wpdb->posts.post_title LIKE '{$n}{$term}{$n}') OR ($wpdb->posts.post_content LIKE '{$n}{$term}{$n}'))";
     
    19691968        $intersections = array('category__and' => 'category', 'tag__and' => 'post_tag', 'tag_slug__and' => 'post_tag', 'tag__in' => 'post_tag', 'tag_slug__in' => 'post_tag');
    19701969        $tagin = array('tag__in', 'tag_slug__in'); // These are used to make some exceptions below
    1971         foreach ($intersections as $item => $taxonomy) {
     1970        foreach ( $intersections as $item => $taxonomy ) {
    19721971            if ( empty($q[$item]) ) continue;
    19731972            if ( in_array($item, $tagin) && empty($q['cat']) ) continue; // We should already have what we need if categories aren't being used
     
    20452044
    20462045        if ( empty($q['author']) || ($q['author'] == '0') ) {
    2047             $whichauthor='';
     2046            $whichauthor = '';
    20482047        } else {
    2049             $q['author'] = ''.urldecode($q['author']).'';
     2048            $q['author'] = (string)urldecode($q['author']);
    20502049            $q['author'] = addslashes_gpc($q['author']);
    2051             if (strpos($q['author'], '-') !== false) {
     2050            if ( strpos($q['author'], '-') !== false ) {
    20522051                $eq = '!=';
    20532052                $andor = 'AND';
    20542053                $q['author'] = explode('-', $q['author']);
    2055                 $q['author'] = '' . absint($q['author'][1]);
     2054                $q['author'] = (string)absint($q['author'][1]);
    20562055            } else {
    20572056                $eq = '=';
     
    20592058            }
    20602059            $author_array = preg_split('/[,\s]+/', $q['author']);
    2061             $whichauthor .= " AND ($wpdb->posts.post_author ".$eq.' '.absint($author_array[0]);
    2062             for ($i = 1; $i < (count($author_array)); $i = $i + 1) {
    2063                 $whichauthor .= ' '.$andor." $wpdb->posts.post_author ".$eq.' '.absint($author_array[$i]);
    2064             }
    2065             $whichauthor .= ')';
     2060            $_author_array = array();
     2061            foreach ( $author_array as $key => $_author )
     2062                $_author_array[] = "$wpdb->posts.post_author " . $eq . ' ' . absint($_author);
     2063            $whichauthor .= ' AND (' . implode(" $andor ", $_author_array) . ')';
     2064            unset($author_array, $_author_array);
    20662065        }
    20672066
    20682067        // Author stuff for nice URLs
    20692068
    2070         if ('' != $q['author_name']) {
    2071             if (strpos($q['author_name'], '/') !== false) {
    2072                 $q['author_name'] = explode('/',$q['author_name']);
    2073                 if ($q['author_name'][count($q['author_name'])-1]) {
    2074                     $q['author_name'] = $q['author_name'][count($q['author_name'])-1];#no trailing slash
     2069        if ( '' != $q['author_name'] ) {
     2070            if ( strpos($q['author_name'], '/') !== false ) {
     2071                $q['author_name'] = explode('/', $q['author_name']);
     2072                if ( $q['author_name'][ count($q['author_name'])-1 ] ) {
     2073                    $q['author_name'] = $q['author_name'][count($q['author_name'])-1]; #no trailing slash
    20752074                } else {
    2076                     $q['author_name'] = $q['author_name'][count($q['author_name'])-2];#there was a trailling slash
     2075                    $q['author_name'] = $q['author_name'][count($q['author_name'])-2]; #there was a trailling slash
    20772076                }
    20782077            }
     
    20812080            if ( $q['author'] )
    20822081                $q['author'] = $q['author']->ID;
    2083             $whichauthor .= " AND ($wpdb->posts.post_author = ".absint($q['author']).')';
     2082            $whichauthor .= " AND ($wpdb->posts.post_author = " . absint($q['author']) . ')';
    20842083        }
    20852084
     
    20892088            $whichmimetype = wp_post_mime_type_where($q['post_mime_type']);
    20902089
    2091         $where .= $search.$whichcat.$whichauthor.$whichmimetype;
     2090        $where .= $search . $whichcat . $whichauthor . $whichmimetype;
    20922091
    20932092        if ( empty($q['order']) || ((strtoupper($q['order']) != 'ASC') && (strtoupper($q['order']) != 'DESC')) )
     
    20962095        // Order by
    20972096        if ( empty($q['orderby']) ) {
    2098             $q['orderby'] = "$wpdb->posts.post_date ".$q['order'];
     2097            $q['orderby'] = "$wpdb->posts.post_date " . $q['order'];
    20992098        } elseif ( 'none' == $q['orderby'] ) {
    21002099            $q['orderby'] = '';
     
    21172116                    continue;
    21182117
    2119                 switch ($orderby) {
     2118                switch ( $orderby ) {
    21202119                    case 'menu_order':
    21212120                        break;
     
    21752174            $where .= " AND $wpdb->posts.post_type = 'attachment'";
    21762175            $post_type_object = get_post_type_object ( 'attachment' );
    2177         } elseif ($this->is_page) {
     2176        } elseif ( $this->is_page ) {
    21782177            $where .= " AND $wpdb->posts.post_type = 'page'";
    21792178            $post_type_object = get_post_type_object ( 'page' );
     
    22992298                $pgstrt = '';
    23002299                $pgstrt = ($page - 1) * $q['posts_per_page'] . ', ';
    2301                 $limits = 'LIMIT '.$pgstrt.$q['posts_per_page'];
     2300                $limits = 'LIMIT ' . $pgstrt . $q['posts_per_page'];
    23022301            } else { // we're ignoring $page and using 'offset'
    23032302                $q['offset'] = absint($q['offset']);
     
    23342333            $post_ids = array();
    23352334
    2336             foreach ($this->comments as $comment)
     2335            foreach ( $this->comments as $comment )
    23372336                $post_ids[] = (int) $comment->comment_post_ID;
    23382337
     
    24222421                    if  ( $post_status_obj->protected ) {
    24232422                        // User must have edit permissions on the draft to preview.
    2424                         if (! current_user_can($edit_cap, $this->posts[0]->ID)) {
     2423                        if ( ! current_user_can($edit_cap, $this->posts[0]->ID) ) {
    24252424                            $this->posts = array();
    24262425                        } else {
    24272426                            $this->is_preview = true;
    2428                             if ('future' != $status)
     2427                            if ( 'future' != $status )
    24292428                                $this->posts[0]->post_date = current_time('mysql');
    24302429                        }
     
    24982497
    24992498        // Sanitize before caching so it'll only get done once
    2500         for ($i = 0; $i < $this->post_count; $i++) {
     2499        for ( $i = 0; $i < $this->post_count; $i++ ) {
    25012500            $this->posts[$i] = sanitize_post($this->posts[$i], 'raw');
    25022501        }
     
    25052504            update_post_caches($this->posts, $post_type);
    25062505
    2507         if ($this->post_count > 0) {
     2506        if ( $this->post_count > 0 ) {
    25082507            $this->post = $this->posts[0];
    25092508        }
     
    25622561     */
    25632562    function have_posts() {
    2564         if ($this->current_post + 1 < $this->post_count) {
     2563        if ( $this->current_post + 1 < $this->post_count ) {
    25652564            return true;
    2566         } elseif ($this->current_post + 1 == $this->post_count && $this->post_count > 0) {
     2565        } elseif ( $this->current_post + 1 == $this->post_count && $this->post_count > 0 ) {
    25672566            do_action_ref_array('loop_end', array(&$this));
    25682567            // Do some cleaning up after the loop
     
    25822581    function rewind_posts() {
    25832582        $this->current_post = -1;
    2584         if ($this->post_count > 0) {
     2583        if ( $this->post_count > 0 ) {
    25852584            $this->post = $this->posts[0];
    25862585        }
     
    26152614        $comment = $this->next_comment();
    26162615
    2617         if ($this->current_comment == 0) {
     2616        if ( $this->current_comment == 0 ) {
    26182617            do_action('comment_loop_start');
    26192618        }
     
    26312630     */
    26322631    function have_comments() {
    2633         if ($this->current_comment + 1 < $this->comment_count) {
     2632        if ( $this->current_comment + 1 < $this->comment_count ) {
    26342633            return true;
    2635         } elseif ($this->current_comment + 1 == $this->comment_count) {
     2634        } elseif ( $this->current_comment + 1 == $this->comment_count ) {
    26362635            $this->rewind_comments();
    26372636        }
     
    26482647    function rewind_comments() {
    26492648        $this->current_comment = -1;
    2650         if ($this->comment_count > 0) {
     2649        if ( $this->comment_count > 0 ) {
    26512650            $this->comment = $this->comments[0];
    26522651        }
     
    27392738        $this->get_queried_object();
    27402739
    2741         if (isset($this->queried_object_id)) {
     2740        if ( isset($this->queried_object_id) ) {
    27422741            return $this->queried_object_id;
    27432742        }
     
    27582757     */
    27592758    function WP_Query($query = '') {
    2760         if (! empty($query)) {
     2759        if ( ! empty($query) ) {
    27612760            $this->query($query);
    27622761        }
Note: See TracChangeset for help on using the changeset viewer.