BlogoText / blogotext

A little more than a lightweight SQLite Blog-Engine.
Other
137 stars 30 forks source link

Freeze pour la version 3.7 #178

Closed BoboTiG closed 7 years ago

BoboTiG commented 7 years ago

Hop, la 3.7 est freezée !

Rapportez toute erreur éventuelle sur cette issue.


Quelques commits ne rentrant pas en compte dans le processus de nettoyage :

BoboTiG commented 7 years ago

Bug : admin > Flux RSS, il y a une erreur JS qui empêche de marquer tous les flux visibles comme lu. Je laisse de côté, en attendant si @remrem tu veux t'y coller, pas de soucis.

remrem commented 7 years ago

ça ne sera pas avant ce W.E. pour ma part :/

BoboTiG commented 7 years ago

Il manque une constante URL_BACKUP pour l'export des données. Actuellement, on utilise DIR_BACKUP mais les liens sont erronés. Chaque dossier de backup se trouvera dans son vhost pour la v4, en attendant je zappe et utilise URL_ROOT.

BoboTiG commented 7 years ago

Bon, je n'aurai pas le temps d'en faire plus avant la semaine prochaine. Ce qu'il reste à faire :

@remrem tu peux repasser sur la partie admin, je passerai sur la partie public la semaine pro.

remrem commented 7 years ago

@BoboTiG ok

remrem commented 7 years ago

première passe sur admin/feed

BoboTiG commented 7 years ago

Dirty fix, ça me convient bien dans l'immédiat :)

Ça règle en partie le problème. Il reste ce scénario qui bloque : quand tu sélectionnes sur "Tous les flux" et que là tu cliques sur l'icône pour tout marquer comme lu => erreur JS.

remrem commented 7 years ago

Je commit tout à l'heure

remrem commented 7 years ago

181

BoboTiG commented 7 years ago

J'ai supprimé les fonctions inutilisées, dis moi si ça te convient, sinon je revert. Pour info, j'ai utilisé cette commande :

egrep -rhio 'function .+\(' *.php admin inc | awk -F'[( ]' '{print "echo -n " $2 " && grep -rin " $2 " *.php admin inc | grep -v function | wc -l"}' | bash | egrep '[^0-9]0$'
remrem commented 7 years ago

hormis pour la function str2() qui est utilisée dans l'addon calendar, ça me semble bon ;)

BoboTiG commented 7 years ago

Je mettrai à jour l'addon. D'ailleurs, je n'ai absolument pas vérifié les addons, à ne pas oublier.

remrem commented 7 years ago

188 ajout d'un script pour l'update

Procédure de mise à jour :

  1. sauvegarder BDD + config (une copie en laissant les originaux)
  2. supprimer les dossiers
    • addons/
    • admin/
    • cache/
    • inc/
  3. claquer la version 3.7-dev de BT
  4. rendez-vous sur http://example.com/admin/install.php et suivez la procédure (! ce n'est pas l'install normale :) )
  5. faire le tour des parties public et admin afin de vérifier qu'il n'y ai pas de message d'erreur ou de contenu disparu

Pour info, durant la procédure, BT doit vous demander de spécifier un utilisateur, c'est normal étant donné que le process de hashage des mots de passes change avec la 3.7 (sans rétrocompatibilité).

En cas d'échec :

  1. réinstaller BT master, remettre vos sauvegardes (BDD + config)
  2. venir nous le signaler sur github en étant le plus précis possible dans la description des problèmes rencontrés.

upd 2016-12-28 22:48

BoboTiG commented 7 years ago

Ça me semble pas mal du tout :)

Je corrige 2-3 choses et devrait être pas mal.

B4rb3rouss commented 7 years ago

Dans le thème par défaut, on a dans les liens de navigation :

Après la publication d'un article, le lien "Partager" (à côté de "voir en ligne") ne fonctionne pas semble-t-il. Rien n'est proposé actuellement, ça affiche juste l'article.

C'est tout ce que j'ai vu pour l'instant.

Question à réfléchir : y a -t-il des changements à annoncer pour la mise à jour d'une instance du blog (thème...) ?

remrem commented 7 years ago

Pour le template "à propos" et l'intitulé du calendar, à voir avec @BoboTiG, mais bien vu ;)

Pour le changelogs, yep, pas mal de choses, on avait discuté avec @BoboTiG de mettre en place un changelog.txt (ou dans ce style), je crois que je vais m'y atteler demain (vu que la 3.7 est repoussée).

upd

y a -t-il des changements à annoncer pour la mise à jour d'une instance du blog (thème...) ?

En principe les thèmes customs ne devrait pas être impactés, de mémoire il n'y que l'ajout d'un nouveau tag + les tags des addons. Pour la procédure de mise à jour, elle est différente que pour les versions précédentes, voir mon commentaire ci dessus

remrem commented 7 years ago

Après la publication d'un article, le lien "Partager" (à côté de "voir en ligne") ne fonctionne pas semble-t-il. Rien n'est proposé actuellement, ça affiche juste l'article.

La fonction de partage, il me semble que c'est pour les articles non publiés (invisible ou date de publication > date actuelle) ça ajoute un paramètre share (&share) à la fin de l'url qui permet de donner l’accès à un article. Il y a un autre comportement attendu ou différent dans les versions précédentes de BT ?

B4rb3rouss commented 7 years ago

Ok pour le partage, j'avais pensé à un truc du type "partager sur facebook" et autres bêtises du genre. Mais si c'est pour faire lire le brouillon c'est top. Il faut changer peut-être l'intitulé dans ce cas ?

remrem commented 7 years ago

Yep, l'intitulé actuel peux prété à confusion, je te rejoins pour le changer.
Pour les médias sociaux, un addon serait plus adapté ;)

B4rb3rouss commented 7 years ago

+1 pour l'addon, j'ai commencé un truc inspiré de pluxml, mais j'attends la sortie de la 3.7 pour m'y replonger.

remrem commented 7 years ago

Pour les addons, tu peux t'y mettre, la 3.7 est freezée, donc plus de bouleversements, juste du debug et de du peaufinage ;)

B4rb3rouss commented 7 years ago

Question bête : la 3.7 reste en freeze jusqu'à une date limite ?

remrem commented 7 years ago

@B4rb3rouss , yep, elle est même dépassée, malheureusement @BoboTiG a certains impératifs à gérer IRL, donc la 3.7 est en stand by. Tu as pu la tester ? (branch dev)

B4rb3rouss commented 7 years ago

yep, j'ai pas mal testé, importé mon ancien blogotext et n'ai rien remarqué de particulier. J'hésite à passer en prod' mais rien ne presse après tout :) . Et puisque je vois tes commits sur la 4.0 ça donne encore plus envie.

J'aurais un peu de temps bientôt pour me replonger dans les plugins, ça sera peut-être l'occasion de reprendre la 3.7.

remrem commented 7 years ago

Cool, merci pour le retour. Pour passe en prod, si tu n'es pas pressé, vaux mieux attendre la release 3.7. Yep, la 4.0 promets, mais la 3.7 bouscule pas mal de choses ;)

B4rb3rouss commented 7 years ago

J'ai refait une migration ce matin et j'ai remarqué que lors de l'export du blog pour l'importer ensuite, les paramètres "Nom de l'auteur, du site, description..." n'étaient pas exportés. Rien d'étonnant lorsqu'on voit le code mais il faudra penser à l'expliquer dans les notes de mises à jour.

Sinon, fonctionnent très bien :

remrem commented 7 years ago

@B4rb3rouss , je jette un coup d'oeil, mais pour la migration, pas besoin d'exporter/importer. La procédure https://github.com/BoboTiG/blogotext/issues/178#issuecomment-268914694

upd Bien vu :D Le mécanisme pour l'export de la config du blog existe bien ("maintenance" / "exporter" / "Exporter la base de données et des fichiers dans une archive zip"), mais il n'y a pas de mécanisme pour l'importation :/ Faudrait prévoir ça pour la v4 ;)

B4rb3rouss commented 7 years ago

Non en effet, pas besoin d'export/import pour la migration, mais c'était par habitude.

B4rb3rouss commented 7 years ago

D'autres remarques suite à des tests approfondis, dans le fichier list.html

remrem commented 7 years ago

Si tu veux un coup de main pour les correctifs, n'hésite pas...

B4rb3rouss commented 7 years ago

pour le meta name=* je veux bien. Sinon c'est bon pour includes.js et je m'occupe des avatars en soirée.

remrem commented 7 years ago

@B4rb3rouss ok, je m'occupe de la <meta />

B4rb3rouss commented 7 years ago

C'est bon pour moi, j'ai fait les 2 PR.

Pour <meta /> je te laisse la main car ne suis pas sûr de moi. Il faudrait échapper tous les éléments xml du texte qui ne sont pas valides.

B4rb3rouss commented 7 years ago

Nouveau "bug" lorsqu'on renomme le dossier admin : il persiste à vouloir trouver les libs dans /admin :

Warning: require_once(/htdocs/yeuxdelibad.net/Ignore/php/blogotext/admin/inc/lang/fr_fr.php): failed to open stream: No such file or directory in /htdocs/yeuxdelibad.net/Ignore/php/blogotext/inc/util.php on line 263

Fatal error: require_once(): Failed opening required '/htdocs/yeuxdelibad.net/Ignore/php/blogotext/admin/inc/lang/fr_fr.php' (include_path='.:/pear/lib:/var/www/pear/lib') in /htdocs/yeuxdelibad.net/Ignore/php/blogotext/inc/util.php on line 263
remrem commented 7 years ago

héhé, my bad pour le coup du path admin, j'ai vu ça en bossant sur la v4, mais j'ai oublié de fix la 3.7 :s Je PR un fix tout à l'heure...

remrem commented 7 years ago

pour l'admin path, le #210 doit fixer le problème ;)

B4rb3rouss commented 7 years ago

Vu, merci !

Rien de tel qu d'installer la 3.7-dev pendant quelques jours pour chasser les bugs. Il va avoir du boulot @BoboTiG à son retour ^^

B4rb3rouss commented 7 years ago

@remrem : testé. Ça semble fonctionner comme prévu.

remrem commented 7 years ago

Retour au freeze de la 3.7, je teste de mon côté la stabilité, je cherche les quelques bugs et on la pousse en prod. Tous retours, tests sont les biens venus ;)

B4rb3rouss commented 7 years ago

J'utilise en prod la 3.7 depuis un moment pour l'instant, et mis à part les flèches de navigation dans le lecteur de flux, rien à redire.

remrem commented 7 years ago

ok, je vois ça...

B4rb3rouss commented 7 years ago

J'ai mal dû m'exprimer, ça a été réglé dans un précédent commit.

remrem commented 7 years ago

oki ;) derniers tests et je pousse en prod ;)

remrem commented 7 years ago

Je revois le dossier addon du repos blogotext, je vais juste mettre un dossier basique avec une note dedans.