[SPIP Zone] spip_loader.php et création des dossiers plugins/auto et lib

Jacques3 · Septembre 4, 2018, 11:13

Bonjour,
A la suite de à et de différentes discussions : il semblerait que la création en auto des dossiers plugins/auto et lib ne concerne vraiment que les gens qui utilisent spip_loader.php Comme par ailleurs il semble y avoir consensus sur le fait qu’il n’est plus utile de bloquer la création de ces dossiers, on ajouterait les lignes ci-dessous à spip_loader.php.

b_b · Septembre 14, 2018, 1:05

Hop,

Le 04/09/2018 à 13:13, Jacques a écrit :

Bonjour,
A la suite de à Avoir les répertoires plugins/auto par défaut dans la dist (#3610) · Tickets · spip / spip · GitLab et de différentes discussions : il semblerait que la création en auto des dossiers plugins/auto et lib ne concerne vraiment que les gens qui utilisent spip_loader.php
Comme par ailleurs il semble y avoir consensus sur le fait qu'il n'est plus utile de bloquer la création de ces dossiers, on ajouterait les lignes ci-dessous à spip_loader.php.

Pour répondre au commit dont nous n'avons pas reçu la notification par email :

Tout d'abord merci pour le commit

Comme le disait marcimat sur IRC, ça pourrait être amélioré :

1) une fonction spécifique "creer_repertoires_plugins" ou je ne sais quel nom
2) l’appeler donc après le tester_repertoire peut être ou ailleurs, si le test des répertoires à réussi ?

J'attends qu'on améliore ça avant de mettre à jour le loader sur spip.net pour la diffusion de la modif.

++
b_b

Jacques3 · Septembre 30, 2018, 5:17

Bonsoir,
Avec https://core.spip.net/projects/spip-loader/repository/revisions/111806 j'ai essayé d'améliorer.
Merci de vos retours

Jacques

Le 14/09/2018 à 15:05, Bruno Bergot a écrit :

Hop,

Le 04/09/2018 à 13:13, Jacques a écrit :

Bonjour,
A la suite de à Avoir les répertoires plugins/auto par défaut dans la dist (#3610) · Tickets · spip / spip · GitLab et de différentes discussions : il semblerait que la création en auto des dossiers plugins/auto et lib ne concerne vraiment que les gens qui utilisent spip_loader.php
Comme par ailleurs il semble y avoir consensus sur le fait qu'il n'est plus utile de bloquer la création de ces dossiers, on ajouterait les lignes ci-dessous à spip_loader.php.

Pour répondre au commit dont nous n'avons pas reçu la notification par email :

Connexion · GitLab

Tout d'abord merci pour le commit

Comme le disait marcimat sur IRC, ça pourrait être amélioré :

1) une fonction spécifique "creer_repertoires_plugins" ou je ne sais quel nom
2) l’appeler donc après le tester_repertoire peut être ou ailleurs, si le test des répertoires à réussi ?

J'attends qu'on améliore ça avant de mettre à jour le loader sur spip.net pour la diffusion de la modif.

++
b_b

JLuc · Septembre 30, 2018, 8:05

Le 30/09/2018 à 19:17, Jacques a écrit :

Bonsoir,
Avec https://core.spip.net/projects/spip-loader/repository/revisions/111806 j'ai essayé d'améliorer.
Merci de vos retours

Il manque pas le plus important, dans ce commit ?
JL

Jacques3 · Septembre 30, 2018, 8:08

En fait celui-là c'est la version que j'avais écrasée...
Le vrai c'est celui là : https://core.spip.net/projects/spip-loader/repository/revisions/111805

Jacques

Le 30/09/2018 à 22:05, JLuc a écrit :

Le 30/09/2018 à 19:17, Jacques a écrit :

Bonsoir,
Avec https://core.spip.net/projects/spip-loader/repository/revisions/111806 j'ai essayé d'améliorer.
Merci de vos retours

Il manque pas le plus important, dans ce commit ?
JL

----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Franck · Octobre 1, 2018, 9:28

Hello
J'ai fait une série de test chez ovh, donc, pour info, la création des dossiers "lib" et "plugins/auto" avec les droits 0705 fonctionne parfaitement, concernant l'installation d'un nouveau spip, j'ai pas encore eu le temps de faire des tests quand il y avait déjà un spip en place, n'y quand il y avait déjà un spip en place avec "plugins/spip" voir aussi "lib"
En faite, il l'unique problème que j'ai eu, c'est en php 7.2 avec le moteur "php" en mode "dev" (mais bon spip lui-même n'est pas encore compatible php 7.2, donc...)
Les tests que j'ai fait sont :
Environnement : stable
Version PHP 5.4
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.4
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.4
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.4
Moteur:php
Mode: production

Environnement : stable
Version PHP 5.5
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.5
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.5
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.5
Moteur:php
Mode: production

Environnement : stable
Version PHP 5.6
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.6
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.6
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.6
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.0
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 7.0
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.0
Moteur:php
Mode: développement

Environnement : stable
Version PHP 7.0
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.1
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 7.1
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.1
Moteur:php
Mode: développement

Environnement : stable
Version PHP 7.1
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:phpcgi
Mode: développement
Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Après avoir fait un clique sur "Commencer l'installation"
Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Warning: Cannot modify header information - headers already sent by (output started at /.../pclzip.php:28) in /.../spip_loader.php on line 1228
Pour info, il y a quand même création des dossiers "lib" et "plugins/auto" mais spip_loader ne fonctionne pas !

Environnement : stable
Version PHP 7.2
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:php
Mode: développement
Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Après avoir fait un clique sur "Commencer l'installation"
Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Spip_loader semble quand même ok !

Franck

-----Message d'origine-----
De : Jacques <jacques@jack31.net>
Envoyé : dimanche 30 septembre 2018 22:09
À : spip-zone@rezo.net
Objet : Re: [SPIP Zone] spip_loader.php et création des dossiers plugins/auto et lib

En fait celui-là c'est la version que j'avais écrasée...
Le vrai c'est celui là :
https://core.spip.net/projects/spip-loader/repository/revisions/111805

Jacques

Le 30/09/2018 à 22:05, JLuc a écrit :

Le 30/09/2018 à 19:17, Jacques a écrit :

Bonsoir,
Avec
https://core.spip.net/projects/spip-loader/repository/revisions/111806
j'ai essayé d'améliorer.
Merci de vos retours

Il manque pas le plus important, dans ce commit ?
JL

----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

8de2996781608e1d92f4 · Octobre 2, 2018, 8:54

Le code ajouté mériterait d’être nettoyé et vérifié, j’ai filé un coup de main a jacques rapidement avant de partir dimanche =).

j’ai pas habitude d’utiliser le loader et je ne sais pas quels sont les tenants et aboutissants d’une telle modif.

Le seul truc pertinent, que j’ai noté, c’est de faire renvoyer la valeur du chmod par la fonction tester_repertoire (de mémoire).
Ensuite il faut veillez à ce que la création ne supprime pas les contenus des répertoires si ils existent (avec tout plein de plugin en dedans ; ça serait dommage de tout supprimer lors d’un maj :')
Sinon je ne sais pas si la fonction creer_repertoire_plugin doit être dans la fonction déballer ou ou milieu du script, en tout cas il faut gérer correctement le retour de cette fonction.
pour afficher d’éventuelles erreurs, confirmations etc…

Bonne soirée

Le mar. 2 oct. 2018 à 01:14, Franck <spip.franck@lien-d-amis.net> a écrit :

Hello
J’ai fait une série de test chez ovh, donc, pour info, la création des dossiers « lib » et « plugins/auto » avec les droits 0705 fonctionne parfaitement, concernant l’installation d’un nouveau spip, j’ai pas encore eu le temps de faire des tests quand il y avait déjà un spip en place, n’y quand il y avait déjà un spip en place avec « plugins/spip » voir aussi « lib »
En faite, il l’unique problème que j’ai eu, c’est en php 7.2 avec le moteur « php » en mode « dev » (mais bon spip lui-même n’est pas encore compatible php 7.2, donc…)
Les tests que j’ai fait sont :
Environnement : stable
Version PHP 5.4
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.4
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.4
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.4
Moteur:php
Mode: production

Environnement : stable
Version PHP 5.5
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.5
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.5
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.5
Moteur:php
Mode: production

Environnement : stable
Version PHP 5.6
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 5.6
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 5.6
Moteur:php
Mode: développement

Environnement : stable
Version PHP 5.6
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.0
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 7.0
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.0
Moteur:php
Mode: développement

Environnement : stable
Version PHP 7.0
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.1
Moteur:phpcgi
Mode: développement

Environnement : stable
Version PHP 7.1
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.1
Moteur:php
Mode: développement

Environnement : stable
Version PHP 7.1
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:phpcgi
Mode: développement
Warning: Use of undefined constant _DIR_TMP - assumed ‹ _DIR_TMP › (this will throw an Error in a future version of PHP) in /…/pclzip.php on line 28
Après avoir fait un clique sur « Commencer l’installation »
Warning: Use of undefined constant _DIR_TMP - assumed ‹ _DIR_TMP › (this will throw an Error in a future version of PHP) in /…/pclzip.php on line 28
Warning: Cannot modify header information - headers already sent by (output started at /…/pclzip.php:28) in /…/spip_loader.php on line 1228
Pour info, il y a quand même création des dossiers « lib » et « plugins/auto » mais spip_loader ne fonctionne pas !

Environnement : stable
Version PHP 7.2
Moteur:phpcgi
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:php
Mode: production

Environnement : stable
Version PHP 7.2
Moteur:php
Mode: développement
Warning: Use of undefined constant _DIR_TMP - assumed ‹ _DIR_TMP › (this will throw an Error in a future version of PHP) in /…/pclzip.php on line 28
Après avoir fait un clique sur « Commencer l’installation »
Warning: Use of undefined constant _DIR_TMP - assumed ‹ _DIR_TMP › (this will throw an Error in a future version of PHP) in /…/pclzip.php on line 28
Spip_loader semble quand même ok !

Franck

-----Message d’origine-----
De : Jacques <jacques@jack31.net>
Envoyé : dimanche 30 septembre 2018 22:09
À : spip-zone@rezo.net
Objet : Re: [SPIP Zone] spip_loader.php et création des dossiers plugins/auto et lib

En fait celui-là c’est la version que j’avais écrasée…
Le vrai c’est celui là :
https://core.spip.net/projects/spip-loader/repository/revisions/111805

Jacques

Le 30/09/2018 à 22:05, JLuc a écrit :

Le 30/09/2018 à 19:17, Jacques a écrit :

Bonsoir,
Avec
https://core.spip.net/projects/spip-loader/repository/revisions/111806
j’ai essayé d’améliorer.
Merci de vos retours

Il manque pas le plus important, dans ce commit ?
JL

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Franck · Octobre 17, 2018, 7:48

Yop

Bon alors, j’ai refait une série de test chez ovh par contre, cette fois, j’ai fait l’installation d’un SPIP 3.3.0-dev [24123] puis l’installation du plugin https://plugins.spip.net/simplog.html dans le dossier « plugins » ainsi que via « svp » du plugin https://plugins.spip.net/article_pdf.html ce qui m’a donc fait un plugin dans le dossier « plugins/auto » et d’uns lib dans le dossier « lib »

Une fois fait, j’ai fait les tests en (voir plus bas) et je n’ai pas eu de problèmes, les 2 plugs sont toujours là ainsi que la lib, il ne semble pas y avoir de fichiers en moins