[RÉGLÉ] Adaptation module joomla 1.5 pour joomla 3

[cat1910]

Chers tous,
j'essaie d'adapter un module développé pour Joomla 1.5 pour Joomla 3, mais je ne suis pas développeuse et j'essaie de me dépatouiller comme je peux.
Le module lit un fichier xml déposé sur le serveur et créé automatiquement des articles joomla dans une catégorie donnée à partir des infos de ce fichier xml.
J'ai réussi à adapter le fichier xml du module, c'est le plus facile.

mais apparemment la requête sql n'est pas juste, j'obtiens l'erreur suivante :
Fatal error: Call to undefined function from() in .../modules/mod_newsxmlreader/mod_newsxmlreader.php on line 202

La requête en ligne 202 est (bidouillée par mes soins):

$db =& JFactory::getDBO();
$query->select * from($db->quoteName('#__content'));
$query->where($db->nameQuote('metadesc').' = '.$db->Quote($updated).' AND '.$db->nameQuote('state').' > '.$db->Quote('-1'));
$db->setQuery($query);
$upd = $db->loadAssocList();

L'original qui fonctionne pour joomla 1.5 :
$db =& JFactory::getDBO();
$query = 'SELECT *'.' FROM '.$db->nameQuote('#__content').' WHERE '.$db->nameQuote('metadesc').' = '.$db->Quote($updated).' AND '.$db->nameQuote('state').' >-1';
$db->setQuery($query);
$upd = $db->loadAssocList();

Une bonne âme développeuse pourrait me donner un coup de main ?
Merci d'avance !
Catherine

[pmleconte]

Bonjour Catherine,

Bienvenue dans le monde impitoyable des développeurs. Impitoyable car PHP ne laisse pas passer les erreurs.....ou presque.

Regardes sur le lien https://docs.joomla.org/Selecting_da...g_JDatabase/fr

Tu devrais y voir que :
$db= JFactory::getDBO(); fonctionne sans le &

Tu y verras aussi la syntaxe du $query->select et de ses amis qui sont des fonctions donc avec des parenthèses, ....

Tu peux essayer

$db = JFactory::getDbo();
$query = $db->getQuery(true);
$query->select('*')
->from($db->quoteName('#__content'))
->where($db->quoteName('metadesc').' = '.$db->quoteName($updated).' AND '.$db->nameQuote('state').' > -1'));
$db->setQuery($query);
$upd = $db->loadAssocList(); 

Pascal

[cat1910]

Merci Pascal de te pencher sur mon problème, je sais bien que le code ne souffre pas d'à peu près :-) j'essaie de comprendre ce que j'écris mais je n'ai pas encore l'oeil bien affuté!
J'ai testé ta proposition, ça avance, j'ai maintenant l'erreur suivante:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '>-1' at line 3

[pmleconte]

Comme écrit précédemment, pas d'approximation, même pour moi

Ce n'est pas $db->nameQuote mais $db->quoteName

Pascal

[cat1910]

C'est encore mieux... :-)
mais maintenant j'ai cette erreur: 1054 - Unknown column '20180409T081741Z' in 'where clause'
qui pour le coup je suppose dépend plus de la programmation même du module que d'erreurs de syntaxe :-(

[pmleconte]

Je pense qu'il s'agit du contenu de la variable $updated.

Pascal

[jfque]

Ce n'est pas $db->quoteName($updated), c'est $db->quote($updated).

quoteName est la fonction qui met des guillemets ` autour des noms de colonnes MySQL, tandis que quote met des guillemets ' autour des variables.

[cat1910]

Merci, ça arrange encore un peu plus :-) ... maintenant j'ai le message suivant...
1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '* FROM `#__content`,`#__content` WHERE `metadesc` = '20180411T054257Z' AND `' at line 1

[pmleconte]

Bonjour Catherine,

Peux-tu nous remettre ta commande SQL complète car il doit encore manquer un petit quelque chose ?

Pascal

[cat1910]

Voilà le morceau de code un peu plus large, peut être qu'avec le fichier intégral (il fait 250 lignes), vous auriez une meilleure vision du pb potentiel ?

$mainframe = JFactory::getApplication();
$new='index.xml';

$path=$param;
$objDOM = new DOMDocument();
$objDOM->load($path.$new);
$updated = $objDOM->getElementsByTagName("ThisRevisionCreated")->item(0)->nodeValue;
$driver = $objDOM->getElementsByTagName("NewsComponent");

$db = JFactory::getDbo();
$query = $db->getQuery(true);
$query->select('*')
->from($db->quoteName('#__content'))
->where($db->quoteName('metadesc').' = '.$db->quote($updated).' AND '.$db->quoteName('state').' > -1');
$db->setQuery($query);
$upd = $db->loadAssocList(); 

Merci de votre aide, c'est vraiment sympa!

[jfque]

Vous êtes sûre du message d'erreur, parce que ce n'est pas normal d'avoir FROM `#__content`,`#__content` WHERE. Si c'est vraiment ça le message, l'erreur vient de là mais pourquoi ... ?

[cat1910]

oui, c'est le message qui apparait !

[jfque]

Et c'est le seul endroit où il y a cette requête ? Pouvez-vous ajouter

echo $query->dump(); exit;

après $db->setQuery($query); puis recharger la page et copier la requête qui s'affiche pour voir la tête qu'elle a vraiment, parce qu'avec le code, je ne vois rien qui cloche.

[cat1910]

Bonjour,

j'ai remis la requête sous la forme ou elle était écrite à l'origine pour Joomla 1.5, avec les corrections apportées par Pascal et vous même.
Ca donne ceci:

//$path=$param.$categorias[$cat];
$path=$param;
//$objDOM = new DOMDocument();
$objDOM = new DOMDocument();
$objDOM->load($path.$new);
$updated = $objDOM->getElementsByTagName("ThisRevisionCreated")->item(0)->nodeValue;
$driver = $objDOM->getElementsByTagName("NewsComponent");

$db = JFactory::getDbo();
$query = $db->getQuery(true);
//$query->select('*')
//->from($db->quoteName('#__content'))
//->where($db->quoteName('metadesc').' = '.$db->quote($updated).' AND '.$db->quoteName('state').' > -1');

$query = 'SELECT *'.' FROM '.$db->quoteName('#__content').' WHERE '.$db->quoteName('metadesc').' = '.$db->quote($updated).' AND '.$db->quoteName('state').' >-1';

$db->setQuery($query);
$upd = $db->loadAssocList();


if(empty($upd)){
    $titulos = $objDOM->getElementsByTagName("HeadLine");
    $cx=-1;
    $k=0;
    $z=-1;
    $y=0;
    foreach($titulos as $valor){
        $z++;
        $titu = $titulos->item($z)->nodeValue;

        $query = 'SELECT *'.' FROM '.$db->quoteName('#__content').' WHERE '.$db->quoteName('title').' = '.$db->quote($titu).' AND '.$db->quoteName('state').' >-1';
        $db->setQuery($query);
        $upd = $db->loadAssocList();
        if(!empty($upd)){
        //hay q hacer update
            //echo 'Actualisé ->'. $titu .'<br>';
            $id = $upd[0]['id'];
            getIndexXml($driver,$updated,$path,$z,2,$id, $id_cat);
        }else{
        //hay q agregar
            //echo 'Agrégé ->'. $titu .'<br>';
            getIndexXml($driver,$updated,$path,$z,1,0, $id_cat);
        }
    }
    //echo "Mises a jour";
}else{
    //echo 'Categorie a jour';
} 

J'ai aussi corrigé plus haut dans le code (et à différents endroits avec des variables $datenow :

'created' => $datenow->toMysql(), 

en

'created' => $datenow->Format( $dateFormat ), 

Je n'ai plus de message d'erreur, et j'attends le prochain fichier xml déposé sur le serveur automatiquement pour voir si le module fait son boulot!

[jfque]

Attention, dans Joomla! 3, c'est maintenant $datenow->format($dateFormat) (avec un 'f' minuscule). L'ancien code était 'toFormat'.