Tag: plugin

Durante el tiempo que no hubo actividad alguna en este blog y por algunos trabajos que me encomendaron relacionados al desarrollo de plugins y modificación del código de WordPress, he visto algo que se repite casi en todos los plugins que he visto hasta el momento: usan varias consultas pequeñas cuando sólo una puede hacer el trabajo (no tengo idea de porqué hacen las cosas de ese modo).

El problema del código mostrado es que se hacen consultas innecesarias a la base de datos cuando en este caso un simple JOIN serviría para el mismo propósito.

Como ejemplo voy a poner dos plugins que acompañaron a este blog desde hace más de un año y por los que el número de consultas SQL aunmentaba en 55 para generar la página principa. Éstos son:

• Real Fast Latest Comments: En este caso el plugin realiza una consulta para obtener las entradas que recibieron comentarios recientemente y luego itera para mostrar los datos devueltos:

  php:

  function rflc_show_comments($comment_limit = 5, $show_trackbacks = false) {
          global $wpdb;

          if(!$show_trackbacks) {
                  $activity = $wpdb->get_results("SELECT $wpdb->comments.comment_date, $wpdb->comments.comment_author,
                                                  $wpdb->comments.comment_ID, $wpdb->posts.post_title,
                                                  $wpdb->posts.ID FROM $wpdb->comments INNER JOIN
                                                  $wpdb->posts ON $wpdb->posts.ID = $wpdb->comments.comment_post_ID WHERE
                                                  $wpdb->comments.comment_approved = '1' AND $wpdb->comments.comment_type
                                                  NOT LIKE '%back%' ORDER BY $wpdb->comments.comment_date DESC
                                                  LIMIT $comment_limit");
          } else {
                  $activity = $wpdb->get_results("SELECT $wpdb->comments.comment_date, $wpdb->comments.comment_author,
                                                  $wpdb->comments.comment_ID, $wpdb->posts.post_title,
                                                  $wpdb->posts.ID FROM $wpdb->comments INNER JOIN
                                                  $wpdb->posts ON $wpdb->posts.ID = $wpdb->comments.comment_post_ID WHERE
                                                  $wpdb->comments.comment_approved = '1'
                                                  ORDER BY $wpdb->comments.comment_date DESC
                                                  LIMIT $comment_limit");
          }
         
          if($activity) {
                  echo '<ul>';
                  foreach($activity as $comment) {

                          echo '<li>'.wp_specialchars($comment->comment_author).' en: <a href="'. get_permalink($comment->ID) .'#comment-'. $comment->comment_ID .'">'. wp_specialchars($comment->post_title) .'</a></li>' . "\n";

                  }
                  echo '</ul>';
          }
  }

  El problema en este caso es que para mostrar el enlace (permalink) de una entrada, el plugin invoca en cada iteración a la función get_permalink, la misma que hace una nueva consulta (o varias dependiendo de la estructura de permalinks) para recuperar la entrada y formatear el enlace, esto pasa siempre y cuando el argumento pasado no sea un objeto o la entrada ya se encuentre en la caché de objetos.

• Related Posts: El problema es el mismo que se comenta en el anterior, a continuación muestro la parte relevante:

  php:

  $sql = "SELECT ID, post_title, post_content,"
           . "MATCH (post_name, post_content) "
           . "AGAINST ('$terms') AS score "
           . "FROM $wpdb->posts WHERE "
           . "MATCH (post_name, post_content) "
           . "AGAINST ('$terms') "
           . "AND post_date <= '$now' "
           . "AND (post_status IN ( 'publish',  'static' ) && ID != '$post->ID') ";
  if ($show_pass_post=='false') { $sql .= "AND post_password ='' "; }
  $sql .= "ORDER BY score DESC LIMIT $limit";
  $results = $wpdb->get_results($sql);
  $output = '';
  if ($results) {
          foreach ($results as $result) {
                  $title = stripslashes(apply_filters('the_title', $result->post_title));

                  $permalink = get_permalink($result->ID);

                  $post_content = strip_tags($result->post_content);
                  $post_content = stripslashes($post_content);
                  $output .= $before_title .'<a href="'. $permalink .'" rel="bookmark" title="Permanent Link: ' . $title . '">' . $title . '</a>' . $after_title;

                  if ($show_excerpt=='true') {
                          $words=split(" ",$post_content);
                          $post_strip = join(" ", array_slice($words,0,$len));
                          $output .= $before_post . $post_strip . $after_post;
                  }
          }
          echo $output;
  }

Para evitar este comportamiento pero con algunas posibles consecuencias no deseadas**, se debe recuperar también el campo post_name en la primera consulta de ambos casos y a continuación invocar a la función get_permalink con un objeto como parámetro -- get_permalink($comment); y get_permalink($result); respectivamente.

**: Al usar la función get_permalink con un objeto como parámetro, éste se almacena en el caché de objetos y cualquier función que haga uso de get_post puede mostrar entradas "incompletas". Una forma de evitar esto es recuperar todos los campos de la tabla posts o quitar la entrada almacenada en cache luego de invocar a la función get_permalink.