Statistiques

[nienfba]

Ajout des du code de traking statistiques dans la configuration et suppression du templates index. Insertion du code dans le tempate de pied de page.

[camillemonchicourt]

Oui plus propre en effet ! Merci !

[camillemonchicourt]

Merci de faire les PR plutôt dans la branche 'DEVELOP' si possible.

[nienfba]

ok je passe sur la branche develop pour la suite.

[camillemonchicourt]

:+1:

[camillemonchicourt]

En fait, on capte pas tout dans cette PR.

Tu as viré le JS de GA dans index.tpl.html, nickel.

Mais il faut le remettre dans bottom.inc.tpl.html du coup, non ?

Il faut qu'on ajoute la clé GA en paramètre dans config.php aussi pour pas la mettre en dur.

Et je vois pas bien à quoi correspond le paramètre tracking_stats car il est déclaré nul part ?

[romthi38]

L'application en développement ne marche plus car la fonction "if" n'est pas reconnue par Smarty.

[nienfba]

ok donc en effet viré le JS de GA dans index.tpl.html

Rajout de ce code dans le fichier bottom.tpl.html

{* Tacking statistiques *} {if(isset(config::get('tracking_stats')) && config::get('tracking_stats') != '')} {config::get('tracking_stats')} {/if}

Je fais un PR pour

1. pour corriger le bogue rajouter le {/if} à la dernière ligne de bottom.tpl.html ... sorry ;)
2. Le paramètre "tracking_stats" est défini dans le fichier config.inc.php. Le commit n'était pas passé tout à l'heure.

[camillemonchicourt]

OK cool pour le petit correctif, on l'avait repéré entre temps. Ça fait un bon exercice ! :-)

OK du coup on met tout le JS de tracking dans le paramètre tracking_stats. Je pensai que c'était juste le code GA. Mais en effet comme ça, ça permet d'utiliser potentiellement un autre outil de stats moins Googlien ! :-)

[nienfba]

Oui moi j'utilise Piwik qui est vraiment très bon ;) Mais faut avoir un serveur et un peu de ressources si le site est consulté. Donc on laisse la porte ouverte au libre ;)

[camillemonchicourt]

:+1: