[Solved] Stranezze di Simpletagging

[blast2007]

Ciao,
premesso che appena installato un nuovo modulo riesco sempre a imbattermi in un problema al terzo click...  vorrei segnalarvi questa anomalia del modulo simpletagging.

In breve:
Simpletagging è installato (aggiunte le istruzioni nei template) e funzionante.
Le prove effettuate sono riferite alle pagine demo di CMSMS.

Edito 'Home', aggiungo un  carattere alla fine del contenuto e clicco su invia (quindi il campo tags è vuoto).
Edito 'How CMSMS Works' aggiungo un carattere alla fine del contenuto e clicco su invia (quindi il campo tags è vuoto).

Fatto cio' le pagine diventano 'correlate' al 100% in quanto condividono un tag vuoto. Questa mi sembra una stranezza.

Spulciando in mysql, vedo che vengono aggiornate le tabella 'cms_content_props' (viene aggiunto prop_name di tipo 'Tags' vuoto ad ogni pagina modificata) e la tabella 'cms_module_simpletagging' (vengono aggiunti gli ID e i tags delle pagine, in questo caso vuoti).

Pensate che questo sia un comportamento "normale" del modulo?
Ciao
blast

[alby]

Spulciando in mysql, vedo che vengono aggiornate le tabella 'cms_content_props' (viene aggiunto prop_name di tipo 'Tags' vuoto ad ogni pagina modificata) e la tabella 'cms_module_simpletagging' (vengono aggiunti gli ID e i tags delle pagine, in questo caso vuoti).

In 'cms_content_props' è normale perchè deriva dai tag trovati in 'cms_content'
Probabilmente è un errore (perchè non era stato previsto) l'inserimento del tag vuoto in 'cms_module_simpletagging' (che secondo me ha poco senso). Dovrebbe esserci un controllo preventivo se !empty() allora INSERT ...

Alby

[blast2007]

Bene, ho modificato la funzione addTags dentro a simpletagging.module.php

function addTags($pageid, $tags)
{
$db =& $this->GetDb();

$alltags = explode(',', $tags);
foreach ($alltags as $tag)
{
$tag = trim($tag);
if (!empty($tag))
{
$db->Execute("INSERT INTO ".cms_db_prefix()."module_simpletagging (page_id, tag) VALUES (?,?)", array($pageid,$tag));
}
}

}

(le modifiche sono in grassetto)

e ora funziona.

Ciao e grazie
blast

[alby]

e ora funziona.

ottimo.
Metti un post nel bug tracker di quel modulo così potrebbe correggerlo

Alby