Le blog de llaumgui - Balise - cache

IPBSDK et la (mauvaise) gestion des attachements

llaumgui — Fri, 23 Sep 2005 15:51:00 +0000

Plus j'avance dans le débugage du IPBSDK et le développement de mon portail, plus je me rends compte qu'il n'y a pas de cohésion entre les différentes fonctions qui compose ce SKD. En effet, prenons l'exemple de la gestion des attachements qui fait 1 requête par attachement alors qu'il est si simple d'utiliser une fonction récupérant tout les attachements d'un message... Le pire est que cette fonction existe ! Mais les 2 ne sont pas reliées. Rassurez vous, je vous livre mon code à rajouter dans la classe MySDK.

/**

     * Get a post's attachments

     * Gestion d'une mise en cache.

     * @author Guillaume Kulakowski <guillaume_AT_llaumgui_DOT_com>

     * @author Pita <peter@randomnity.com>

     * @since 1.11

     * @param integer $postid The post id to check

     * @param boolean $override Whether to override permissions or not.

     * @return array Post's attachments

     */

     function get_post_attachments($postid,$override=FALSE) {



        $query = "";

        if($override == FALSE)

            { $query = " AND attach_approved=1"; }

            

        $this->DB->query(    "    SELECT *

                                        FROM ibf_attachments

                                        WHERE attach_pid='".$postid."'".$query);

        

        if( $this->DB->get_num_rows() == 0 )

            { return FALSE; }



        while ( $row = $this->DB->fetch_row() )

            { $return[] = $row; }

        

        $this->_cache['post_attachments'.$postid] = $return;

    }









    /**

     * Get HTML code for outputting an attachment image.

     * Utilisation d'un cache.

     * @author Guillaume Kulakowski <guillaume_AT_llaumgui_DOT_com>

     * @since 1.11

     * @param integer $postid The post id to check

     * @param integer $id The attachment id to be parsed.

     * @param boolean $override Whether to override permissions or not.

     * @return string HTML code

     * @see attachment_info(), attachment_thumb()

     */

    function attachment_image( $pid, $id,$override=FALSE ) {

        

        /*

         * On récupère tous les attachements du poste :

         */

        if ( empty($this->_cache['post_attachments'.$pid]) )

            {

                $this->get_post_attachments($pid);

                // Ancien sysèmeme, au cas où :

                if ( empty($this->_cache['post_attachments'.$pid]) )

                    {

                        $att = $this->attachment_info($id,$override);

                        if($att['attach_is_image'] == 0)

                            { return FALSE; }

                    }

            }

        else

            {

                foreach( $this->_cache['post_attachments'.$pid] as $key => $value )

                    {

                        if ( $value['attach_id'] == $id )

                            { $att = $value; }

                    }

            }

        

        /*

         * Gestion des miniatures :

         */

        if ( empty($att['attach_thumb_location']) )

            { $out = '<img src="'.$this->board_url.'/uploads/'.$att['attach_location'].'" alt="'.$att['attach_file'].'" />'; }

        else

            { $out = '<a href="'.$this->board_url.'/index.php?act=Attach&type=post&amp;id='.$att['attach_id'].'" target="_blank"><img src="'.$this->board_url.'/uploads/'.$att['attach_thumb_location'].'" class="attach" alt="'.$att['attach_file'].'" /></a>'; }

        return $out;

    }

?>

Les nouveauté d'IPB 2.1

llaumgui — Sat, 17 Sep 2005 01:28:00 +0000

Il y a quelques jours, IPS a publié la nouvelle version de son produit phare : Invision Power Board 2.1.
Cette nouvelle version a bien sûr été installée sur les 4 forums que je gère, mais la mise à jours n'a pas été si simple...

En effet, Matt, le développeur, a modifié profondément la structure des répertoires d'IPB et a renommé un grand nombre de variables principales telles que $ibforums. J'en ai donc profité pour relire, reformater et recommenter convenablement mon code plutôt que de faire de bêtes "rechercher/remplacer".

Une fois mon portail migré (IPB 2.1 + ipbSDK 1.6), je me suis livré à une petite étude des nouveautés.

L'éditeur de texte enrichi :

La 2.1 offre le choix entre un éditeur classique relooké et un éditeur enrichi en mode WYSIWYG. Même si pour le moment je teste cette fonction, qui est quelque peu bugée, selon moi, force est de constater qu'un geek tape le bbcode plus rapidement à la main qu'avec un éditeur.

L'AJAX :

L'AJAX est très en vogue actuellement et pourrait être vulgarisé comme étant du php côté serveur mais avec un dynamisme côté client.
Cependant Matt en a usé et abusé (Mon assistant par exemple). Mais je retiens 2 fonctions très pratiques : In-line Topic Editing et In-line Post Editing.

Du RSS à foison :

Comme indiqué lors de mon précédent billet, la 2.1 apporte la gestion du RSS pour les forums et les calendriers. Cerise sur le gâteau, cette gestion passe par un cache.

Mais après avoir décortiqué ce cache, je me pose des questions sur son utilité. Si dans mon esprit, un cache est fait pour faire un appel fichier en lieu et place d'un requête MySQL, ici ce n'est pas le cas puisque les informations sont stockée dans la base de donné.
En stockant les flux RSS dans un fichier sur le serveur, Matt aurait pu économiser 1 connexion serveur ainsi qu'1 requête MySQL. Mais le cache n'est là que pour avoir une requête simple à la place d'un requête musclée faisant appel à plusieurs LEFT JOIN.

Les questionnaires :

Matt a remplacé les simple sondages par des questionnaires avec plusieurs questions. Mais toujours 1 seule réponse par question ! Pourquoi recoder un truc et ne pas tenir compte des demandes ?

L'anti-bump (Merge member's concurrent posts) :

Petite fonction sympa qui fusionne les messages si on se répond à soit même.

Les calendriers

Et oui, je dis les car la version 2.1 permet la gestion de plusieurs calendriers et leur modération. Cependant, on ne peut afficher qu'un calendrier à la fois, il n'y a malheureusement pas de vue permettant d'afficher tous les calendriers.

Une autre faiblesse est la modération qui consiste à masquer l'évènement et à ne le faire apparaître que pour les modérateurs et d'une autre couleur. Encore faut-il que les modérateurs remarquent ces évènemlents en attente. En effet, aucun message privé ni email pour les prévenir.

Pour ceux qui veulent plus d'informations sur les nouveautés qu'apporte IPB 2.1, vous pouvez consulter le changelog complet.

IPB-SDK, les guests et le cache

llaumgui — Tue, 31 May 2005 19:44:00 +0000

Ayant recodé le système de débug pour mon site, j'ai analysé les requêtes MySQL. J'ai effectué l'analyse logué en membre et délogué comme simple invité. La logique voudrait qu'un membre demande plus de requête du fait qu'il interroge sa boite de message privé, récupère ses infos, etc. Hors sous IPB-SDK c'est l'inverse.

Le listing des requêtes faite ressortir une requête identique qui se répète plusieurs fois (5 fois dans mon cas) :

SELECT m.*, me.signature, me.avatar_size, me.avatar_location, me.avatar_type, me.vdirs,

me.location, me.msnname, me.interests, me.yahoo, me.website, me.aim_name, me.icq_number,

g.*, cf.*

FROM ibf_members m

LEFT JOIN ibf_groups g ON (m.mgroup=g.g_id)

LEFT JOIN ibf_pfields_content cf ON (cf.member_id=m.id)

LEFT JOIN ibf_member_extra me ON (me.id=m.id)

WHERE m.id='0'

Le constat est rapide. Il n'y a pas de membre à l'id = 0. La requête retourne rien donc alimente le cache avec rien. De ce fait une interrogation du cache ne retourne rien et ré-effectue la requête. La fonction concernée par ce problème et la fonction get_advinfo() que j'ai redéfinie pour qu'elle tienne compte des invités.

/**

* Grabs detailed information on a member.

* Modification : Gestion des guest.

* @author Guillaume Kulakowski <guillaume_AT_llaumgui_DOT_com>

* @author Pita (conversion to 2.0) <peter@randomnity.com>

* @author Cow <khlo@global-centre.com>

* @param integer $memberid

* @return array Member Information, or FALSE on failure

* @see get_info(),get_avatar(),get_raw_sig(),get_photo(),get_member_pips(),get_member_icon(),get_num_new_posts(),get_skin_id()

*/

function get_advinfo($memberid = '') {



if (!$memberid)

  { $memberid = $GLOBALS['ibforums']->member['id']; }



// Gestion des guests non présentes de base ! :

if ( empty($memberid) || $memberid == 0 )

  { return false; }

else if ($cache = $this->get_cache('get_advinfo', $memberid))

  { return $cache; }

else

  {

   $this->DB->query ("SELECT m.*, me.signature, me.avatar_size, me.avatar_location, me.avatar_type, me.vdirs,

me.location, me.msnname, me.interests, me.yahoo,

me.website, me.aim_name, me.icq_number,

g.*, cf.*

FROM ibf_members m

LEFT JOIN ibf_groups g ON (m.mgroup=g.g_id)

LEFT JOIN ibf_pfields_content cf ON (cf.member_id=m.id)

LEFT JOIN ibf_member_extra me ON (me.id=m.id)

WHERE m.id='" . intval($memberid) . "'");

   if ($this->DB->get_num_rows())

    {

     $info = $this->DB->fetch_row();

     $this->save_cache('get_advinfo', $memberid, $info);

    return $info;

    }

   else

    { return FALSE; }

  }

}

Dans le cas d'XPerience-Fr, cette fonction étant utilisée plusieurs fois, j'économise ainsi 5 requêtes MySQL sur la page d'accueil. Vous pouvez en gagner plus ou moins selon votre code.

Cache ipbSDK vs cache IPB

llaumgui — Sat, 28 May 2005 11:19:00 +0000

Afin d'afficher 2 blocs d'informations supplémentaires sur ma page d'accueil, je me suis mis à étudier les requêtes MySQL engendrées par le ipbSDK afin d'en réduire leur nombre et de pouvoir en mettre 2 de plus pour mes 2 blocs :). Et la surprise :

SELECT * FROM ibf_cache_store WHERE cs_key IN ( 'banfilters', 'settings', 'group_cache', 'systemvars', 'skin_id_cache', 'forum_cache', 'moderators', 'stats', 'languages' )

SELECT cs_value FROM ibf_cache_store WHERE cs_key = "stats"

A quoi sert de charger les informations issue du cache IPB (1ère requête), si la fonction get_board_stats() ne les utilise pas ???!!! La réponse est malheureusement simple, le cache ipbSDK ($this->_CACHE) et celui d'IPB ($GLOBALS'ibforums'->cache) sont différents... J'ai d'abord pensé charger le cache IPB dans celui du SDK:

$this->_cache = $GLOBALS['ibforums']->cache

Mais quel intêret de 2 cache identique ? Et de plus je trouvais que leur vocations étaient différentes car le cache du SDK pouvais contenir des informations sur le membres et non pas uniquement sur le forum... J'ai donc modifié la fonction get_board_stats() pour qu'elle interroge non pas le cache ipbSDK mais celui d'IPB. Bilan, 1 requêtes de gagnée !!!

/**

* Obtenir les statistiques de mon forums.

* Modification : Liaison avec le cache IPB au lieu de celui du SDK.

* @author Guillaume Kulakowski <guillaume_AT_llaumgui_DOT_com>

* @author Pita (conversion to 2.0) <peter_AT_randomnity.com>

* @author Cow <khlo_AT_global-centre.com>

* @return array Board Statistics

*/

function get_board_stats() {

    // 1 : Je cherche dans le cache IPB :

    if ( !empty($GLOBALS['ibforums']->cache['stats']) && is_array($GLOBALS['ibforums']->cache['stats']) )

        { return $GLOBALS['ibforums']->cache['stats']; }        

    else

        {

            $this->DB->query ('SELECT cs_value FROM ibf_cache_store WHERE cs_key = "stats"');

            $row                                     = $this->DB->fetch_row();

            $stats                                    = unserialize(stripslashes($row['cs_value']));

            $GLOBALS['ibforums']->cache['stats']     =  $stats;

            return $stats;

        }

}

A noter que la modification utilise ma classe MySQL et que je n'ai pas modifié la fonction du SDK mais que je l'ai redéfinie.