A propos de countModules()

[jisse03]

Re : A propos de countModules()

Tu peux remplacer la ligne 491 par:

$query = $dbo->getQuery(true); 

Je posterai un patch correct sur la forge. Tu as raison, il y a eu régression du code.

[ced1870]

Re : A propos de countModules()

salut
tout ça m'intéresse... je lis avec attention la solution qui sera retenue
Ced

[ced1870]

Re : A propos de countModules()

PS : il serait intéressant de voir de quelle manière on pourrait overrider la fonction countmodule en php pour y ajouter des tests de contenu de module...

[Mushr00m]

Re : A propos de countModules()

Oui j'avais testé cette ligne de code quand je l'avais vu sur le patch mais ça ne fonctionne pas et me sort cette erreur :

Warning: mysql_fetch_row() expects parameter 1 to be resource, boolean given in /libraries/joomla/database/database/mysql.php on line 683
Warning: mysql_free_result() expects parameter 1 to be resource, boolean given in /libraries/joomla/database/database/mysql.php on line 726 

[Mushr00m]

Re : A propos de countModules()

PS : il serait intéressant de voir de quelle manière on pourrait overrider la fonction countmodule en php pour y ajouter des tests de contenu de module...

Pour le test de contenu regarde mon post #13 ici même. Mais je ne sais pas si c'est assez propre. En tout cas ça fonctionne.

Comme je on ne peut pas overrider ce type de fonction avec un template, je pensais la placer dans un helper du template comme je dis dans le post plus haut mais je ne sais pas si ça ce fait de faire un helper dans un template.

[jisse03]

Re : A propos de countModules()

Oui j'avais testé cette ligne de code quand je l'avais vu sur le patch mais ça ne fonctionne pas et me sort cette erreur :

Warning: mysql_fetch_row() expects parameter 1 to be resource, boolean given in /libraries/joomla/database/database/mysql.php on line 683
Warning: mysql_free_result() expects parameter 1 to be resource, boolean given in /libraries/joomla/database/database/mysql.php on line 726 

Ce n'est pas une erreur mais un warning

Il y a en fait une modif à faire, je patche ça sur une machine d edev et je soumets le patch sur joomlacode

[ced1870]

Re : A propos de countModules()

Pour le test de contenu regarde mon post #13 ici même. Mais je ne sais pas si c'est assez propre. En tout cas ça fonctionne.

Comme je on ne peut pas overrider ce type de fonction avec un template, je pensais la placer dans un helper du template comme je dis dans le post plus haut mais je ne sais pas si ça ce fait de faire un helper dans un template.

oui j'ai vu ton post 13, je ne sais pas si c'est la meilleure solution mais si ça marche c'est déjà pas mal !
dans mon idée je pensais à une méthode que l'on pourrait utiliser ailleurs que seulement dans le template, dans un module par exemple pour faire un countmodule-1 s'il n'a pas de contenu.
Maintenant si c'est pour le mettre dans ton template alors oui (mais il faut remplacer le countmodule standard par ce code)
Ced

[jisse03]

Re : A propos de countModules()

Pour le problème countMenuChildren, voilà le patch qui fonctionne:
Mode d'emploi, dans /libraries/joomla/document/html/html.php, en ligne 479, remplacer toute la fonction countMenuChildren par celle là

    public function countMenuChildren()
    {
        static $children;

        if (!isset($children))
        {
            $dbo = JFactory::getDbo();
            $app = JFactory::getApplication();
            $menu = $app->getMenu();
            $active = $menu->getActive();
            if ($active)
            {
                $query = 'SELECT COUNT (*) FROM #__menu WHERE parent_id= ' . $active->id .
                    ' AND published = 1' ;
                $dbo->setQuery( $query );
                $children = $dbo->loadResult();
            }
            else
            {
                $children = 0;
            }
        }

        return $children;
    } 

et maintenant, ça renvoie le bon résultat.

Et donc, le code suivant dans le template:

<?php $mydoc = JFacrory::getDocument() ;
$children = $mydoc->countMenuChildren() ; *

if ( $this->countModules( 'left' ) && $children  ): ?> 
    <div id="left"> 
        <jdoc:include type="modules" name="left"   /> 
    </div> 
<?php endif; ?>

devrait passer correctement.

[EDIT]:Ce patch sera soumis sur joomlacode pour inclusion, il s'agit d'un vrai bug qui a été découvert

[Mushr00m]

Re : A propos de countModules()

Cool pour le patch si rapide ! et content d'avoir trouvé avec vous un bug réparable ;-)

oui j'ai vu ton post 13, je ne sais pas si c'est la meilleure solution mais si ça marche c'est déjà pas mal !
dans mon idée je pensais à une méthode que l'on pourrait utiliser ailleurs que seulement dans le template, dans un module par exemple pour faire un countmodule-1 s'il n'a pas de contenu.
Maintenant si c'est pour le mettre dans ton template alors oui (mais il faut remplacer le countmodule standard par ce code)
Ced

Je pense que ca reste une bonne solution de vérifier avec le Buffer mais en fait il y a pas trop d'autre solution que de la parcourir sinon c'est impossible pour le système de le savoir car le render n'est pas encore lancé. Après dans le chrome pour ne pas afficher un module vide il suffit de faire :

if (!empty ($module->content)) {.......} 

[Mushr00m]

Re : A propos de countModules()

Pour le problème countMenuChildren, voilà le patch qui fonctionne:
Mode d'emploi, dans /libraries/joomla/document/html/html.php, en ligne 479, remplacer toute la fonction countMenuChildren par celle là

    public function countMenuChildren()
    {
        static $children;

        if (!isset($children))
        {
            $dbo = JFactory::getDbo();
            $app = JFactory::getApplication();
            $menu = $app->getMenu();
            $active = $menu->getActive();
            if ($active)
            {
                $query = 'COUNT (*) FROM #__menu WHERE parent_id= ' . $active->id) .    <=== ICI
                    ' AND published = 1' ;
                $dbo->setQuery( $query );
                $children = $dbo->loadResult();
            }
            else
            {
                $children = 0;
            }
        }

        return $children;
    } 

et maintenant, ça renvoie le bon résultat.

Et donc, le code suivant dans le template:

<?php $mydoc = JFacrory::getDocument() ;
$children = $mydoc->countMenuChildren() ; *

if ( $this->countModules( 'left' ) && $children  ): ?> 
    <div id="left"> 
        <jdoc:include type="modules" name="left"   /> 
    </div> 
<?php endif; ?>

devrait passer correctement.

[EDIT]:Ce patch sera soumis sur joomlacode pour inclusion, il s'agit d'un vrai bug qui a été découvert

Par contre fait attention a ton code tu as une parenthèse fermante sans l'ouvrante à la ligne ou j'ai mis la flèche comme ça (voir au dessus) : <=== ICI

Et aussi ne pas oublier le SELECT devant le COUNT(*) sinon ca ne retournera rien ;-)

[Mushr00m]

Re : A propos de countModules()

Plutôt que de long discours voici ton code corrigé et testé :

public function countMenuChildren() {
    static $children;
    if (!isset($children)) {
        $dbo = JFactory::getDbo();
        $app = JFactory::getApplication();
        $menu = $app->getMenu();
        $active = $menu->getActive();
        if ($active) {
            $query = 'SELECT COUNT(*) FROM #__menu WHERE parent_id = '.$active->id.' AND published = 1';
            $dbo->setQuery($query);
            $children = $dbo->loadResult();
        }
        else {
            $children = 0;
        }
    }

    return $children;
} 

Et le même en gardant l'aspect des commandes SQL de Joomla :

    public function countMenuChildren() {
        static $children;
        if (!isset($children)) {
            $dbo = JFactory::getDbo();
            $app = JFactory::getApplication();
            $menu = $app->getMenu();
            $active = $menu->getActive();
            if ($active) {
                $query = $dbo->getQuery(true); 
                $query->select('COUNT(*)');
                $query->from('#__menu');
                $query->where('parent_id = ' . $active->id);
                $query->where('published = 1');
                $dbo->setQuery($query);
                $children = $dbo->loadResult();
            }
            else {
                $children = 0;
            }
        }

        return $children;
    } 

NB pour l'utilisation de countMenuChildren : Attention cependant après des tests si vous avez un menu de niveau 1 qui est un alias d'un menu de niveau 2, alors logiquement cette commande renvoie 0 enfant, car il regarde sur le menu pointé par l'alias...

[jisse03]

Re : A propos de countModules()

Par contre fait attention a ton code tu as une parenthèse fermante sans l'ouvrante à la ligne ou j'ai mis la flèche comme ça (voir au dessus) : <=== ICI

Désolé, erreur en recopie

[jisse03]

Re : A propos de countModules()

Et le même en gardant l'aspect des commandes SQL de Joomla :

    public function countMenuChildren() {
        static $children;
        if (!isset($children)) {
            $dbo = JFactory::getDbo();
            $app = JFactory::getApplication();
            $menu = $app->getMenu();
            $active = $menu->getActive();
            if ($active) {
                $query = $dbo->getQuery(true); 
                $query->select('COUNT(*)');
                $query->from('#__menu');
                $query->where('parent_id = ' . $active->id);
                $query->where('published = 1');
                $dbo->setQuery($query);
                $children = $dbo->loadResult();
            }
            else {
                $children = 0;
            }
        }

        return $children;
    } 

Là, $query = $dbo->getQuery(true) retourne un warning mysql, tout au moins avec MySQL 5.5. D'ou le retour à une approche plus classique pour la requête.

[Mushr00m]

Re : A propos de countModules()

Oui elle te retourne un warning avec le niveau de rapport d'erreur à "développement" et j'ai du rajouter aussi "$dbo->setQuery($query);" pour que elle fonctionne bien.