Plugin 'Update_history'

[pbasmo]

Bonjour à toute l'équipe,

Cela faisait longtemps que je n'avais plus fait une petite halte dans le forum. Mais voila c'est réparé
J'espère que tout l'équipe se porte au mieux et j'imagine que certains profitent encore des vacances.

Il y avait dans la CPG 1.4, un plugin que j'avais trouvé intéressant à l'époque, il s'agissait de 'Update_History' qui permettait d'indiquer aux internautes les dernières mises à jour des albums au cours d'une période donnée.
Depuis l'arrivée de CPG 1.5, ce plugin n'avait pas été mis à jour. Mais voila .... c'est réparé !

Il ne s'agit pas d'une réécriture mais d'une simple adaptation pour la version CPG 1.5
Toutes les lignes modifiée portent la mention "for CPG 1.5".
Je l'ai testé, et aussitôt intégré dans mon site.

En revanche ce que je n'ai pas touché c'est le numéro de la version du plugin. Vous avez plus l'habitude que moi sur ce sujet.

Je joins en PJ l'archive du plugin.

Bonne soirée à tous

[François Keller]

Merci pbasmo pour cette adaptation que je n'ai pas eu le temps de faire.
je vais jeter un oeil dès que possible pour y ajouter le N° de version et placer le plugin sur le SVN

[pbasmo]

re bonjour,

De rien et il n'y a pas péril en la demeure, en ces temps de vacances il y a mieux à faire.
Toutefois, je me suis rendu compte que le fichiers de langue french.php avait été enregistré en UTF-8 (avec BOM).
L'effet est que cela empêchait l'affichage du captcha!
Donc il faudra réenregistrer les fichiers de langue sous le bon format UTF-8 sans BOM.
J'ai fait la modification et le captcha est revenu.
Désolé pour cette mauvaise manipulation de ma part.
Bonne soirée
Pierre

[pbasmo]

Bonsoir,

Je vous renvoie ici le plugin pour la version CPG 1.5.x car j'y ai apporté quelques corrections :

- j'ai réactivé la fonction de confirmation de suppression de la table associée lors de la désinstallation du plugin. Je l'avais désactivée car il y avait une erreur de fonctionnement. Quand j'ai réalisé mon plugin 'geodata' (et en lisant la documentation concernant le dev) j'ai réussi à faire fonctionner la fonction 'cleanup' adéquate. Donc je l'ai réintroduite dans votre plugin.

- par ce fait j'ai modifié le script 'basic.sql' en y ajoutant le mot clé IGNORE dans les requêtes INSERT (pour éviter l'erreur si la table est conservée après une précédente désinstallation);

- les fichiers de langues sont au bon format UTF-8 sans BOM (pour corriger ma boulette !);

- j'ai corrigé une erreur qui donnait une mauvaise information sur la date des dernières mises à jour des albums et un lien incorrect sur les albums. C'est dans la fonction 'update_history' du script 'codebase.php' où j'ai remonté la ligne $i++ à l'intérieur de la 1ère boucle foreach.

Bonne continuation

Pierre

[François Keller]

super boulot, merci encore...
heureusement que je n'ai pas eu le temps de placer le plugin sur le SVN

[pbasmo]

de rien, c'est avec plaisir !
Mais c'est à moi de vous dire SUPER BOULOT car s'essayer au développement d'un plugin m'a permis de voir le travail extraordinaire que vous avez tous fourni pour réaliser ce logiciel ! Rien n'est complètement figé ! Prévoir des portes d'entrées pour ajouter des fonctions perso sans toucher aux programmes originaux, c'est fabuleux ! Encore faut-il trouver les bonnes clés d'entrées (plugin hooks).
Tant qu'on n'a pas mis les mains dans le camboui, on ne peut pas s'imaginer ce que cela représente. Maintenant j'ai vraiment pris la mesure !
Donc je vous retourne l'expression : SUPER BOULOT !

[François Keller]

je viens de me rendre compte qu'il y avait déjà eu un portage de ce plugin par daxa http://coppermine-gallery.com/forum/index.php/topic,70073.0.html
je n'ai pas été voir si il y avait des différences avec ton travail. Si tu en as le courage et le temps, ça serait inintéressant de regarder ça... 

[pbasmo]

Bonjour,

Je viens de regarder le plugin (V1.1) transposé par daxad et je l'ai installé sur mon site de test.
Il y a une erreur au démarrage du site qui est celle-ci (bouton START):

Code Select Expand


While executing query 'SELECT a.aid, a.title, a.description, a.thumb, a.keyword, category, visibility, filepath, filename, url_prefix, pwidth,
pheight, a.owner  FROM cpg148_albums as a  LEFT JOIN cpg148_pictures as p ON a.thumb=p.pid  WHERE a.category= AND a.aid NOT IN
(16)  ORDER BY a.pos, a.aid LIMIT 0,1' in index.php on line 771

mySQL error: 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 'AND a.aid NOT IN (16)  ORDER BY a.pos, a.aid LIMIT 0,1' at line 1


On peut voir que dans la requête il manque la valeur du champ a.category.

Mais l'erreur ne se produit pas quand on clique sur le bouton ALBUMS ou quand on affiche une catégorie particulière.

En explorant le codebase du script je me suis rendu compte que la variable $cat est réinitialisée à vide (même nom que dans le script index):

Code Select Expand


if ($superCage->get->keyExists('cat')) {
    $cat = $superCage->get->getInt('cat');
} else {
    $cat = '';
}


J'ai donc juste renommé la variable $cat en $cat1 dans la fonction 'update_history' de codebase.php et tout est nickel.
Ce qui donne le code :

Code Select Expand


if ($superCage->get->keyExists('cat')) {
    $cat1 = $superCage->get->getInt('cat');
} else {
    $cat1 = '';
}

$nb=intval($param['number']);
$end = time();
$counter = array();
$info = array();
$FORBIDDEN_SET_UPD = ($FORBIDDEN_SET) ? 'AND p.aid NOT IN (' . implode(', ', $FORBIDDEN_SET_DATA) . ')' : '';
$CAT_FILTER = ($cat1) ? "AND a.category = $cat1" : '';


Sinon le plugin donne aussi les infos de mise à jour quand on clique sur une catégorie particulière. C'est une bonne idée.
Finalement j'aurais du regarder le forum anglais avant, je n'y pense pas toujours 

Bonne soirée
Pierre

[pbasmo]

je me réponds à moi-même 
j'ai oublié de préciser qu'il faudrait ajouter dans le script basic.sql, le mot clé IGNORE si on conserve la table après une désinstallation.

[François Keller]

merci d'avoir regardé les différences, il y aurait donc lieu de faire un mix des deux versions pour avoir un plugin complêt ?

[pbasmo]

un mix ? pas nécessairement. Le plugin modifié par Daxad fait très bien l'affaire finalement et en plus il affiche les infos de mises à jour dans l'affichage des catégories, mais juste pour les albums appartenant à la catégorie affichée.
En revanche dans sa version il faudrait modifier le script basic.sql pour y ajouter le mot clé IGNORE et le script schema.sql pour enlever l'instruction 'type=myisam'

[François Keller]

ok, je vais regarder ça dès que j'ai un moment...

[pbasmo]

Bonjour François,

Je reviens un peu sur ce que j'ai dit à propos du plugin adapté par Daxad : il fait sauter les captchas.
Donc j'ai repris dans notre plugin le petit + concernant les mises à jours dans l'affichage des catégories (le mix dont tu parlais) et pour ma part je vais conserver le notre.
Je ferai une archive que je posterai ici pour en terminer avec ce sujet 

Pierre

[François Keller]

ok, j'attends donc ton archive pour la mettre sur le SVN

[pbasmo]

Bonsoir François,

Désolé d'avoir été un peu long pour venir poster le plugin 'update_history'.
Mais avant je voudrais corriger mon dernier message, car finalement le plugin adapté par Daxad fonctionne (j'espère ne pas avoir été médisant !): les captchas sont revenues après avoir enregistré les fichiers de langue au bon format (utf-8 sans bom - ce n'est pas la 1ère fois que je me fais attraper avec ce format !).
On se retrouve donc avec 2 versions identiques du plugin.
Je les poste ici toutes les 2. A toi de voir ce qu'il convient de faire pour la suite (peut-être garder ici ta version et poster l'autre dans le forum anglais)

Pierre

[François Keller]

Pas de soucis, je vais simplement mettre à jour le plugin existant en lui donnant un nouveau numéro de version et en modifiant le changelog afin d'y ajouter les changements réalisés. Comme les deux plugins sont identiques, peu importe finalement lequel est sur le SVN

Merci encore pour le boulot réalisé

P.S. Comme cette semaine je n'ai pas de connexion très fiable, ça attendra la semaine prochaine