Améliorer la vérification des dossiers et les messages d'erreur, lors de l'installation.

[lipki]

Pendant l'installation de "default", les dossier à rendre accessible en écriture ne sont pas tous vérifier/signaler. Le dossier "public" n'est pas créer si les autorisations sont mauvaise, mais ça ne produit pas d'erreur.

Et le seul message que j'ai vu à propos du dossier cache, affichez cette url : www/dotclear/inc/../cache

Ce qui n'est pas très clair.

[insy]

J'ai fait une demande d'ajout sur Bitbucket avec un petit fix à ce sujet:  https://bitbucket.org/insy/dotclear/commits/6d0e55da6dc0e59e316afc521ddc1bdac78a4b24. Il essaye de créer le dossier à la 1ère demande. A voir si c'est pas préférable de le créer dès l'installation.

[lipki]

J'ai pas accès a ce lien

[insy]

Devrait être bon maintenant, j'ai passé mon repo en public.

[franck]

Penser à fermer le ticket #344 au passage.

[lipki]

on vire le problème du dossier public pour le mettre dans un autre ticket ?

[lipki]

Le message d'erreur qui ma dérangé :

Le répertoire de cache C:\programmes\wamp\www\dotclear-default\inc/../cache n'est pas accessible en écriture.

La je suis sous Windows et l'url affiché est vraiment moche, enfin je trouve.
De plus il s'affiche à la deuxième page d'installation, donc quand on est lancé dans l'instal, on doit retourner sur le ftp pour faire la correction.
Je pense que de le mettre sur la première page, dans un cadre "prérequis" serait plus clair et agréable.

[franck]

lipki, on peut gérer les deux, cache et public, en même temps, mais il faut :

• tenir compte du fait que le cache peut être déplacé ailleurs (via config.php)
• tenir compte du fait que le public peut aussi être ailleurs (via about:config)
• tenir compte des différents modes d'installation (clone depuis dépôt, dc_loader, mise à jour, …)
• tenir compte qu'il peut arriver des choses à ces deux dossiers après l'installation
• …

[lipki]

Dans le but de répondre à tout ça, j'ai commencer par vérifier quelque choses.
Le répertoire de cache n'est pas du tout utilisé pendant la phase d'installation, via le dossier admin/install.
Donc ce message d'erreur est plus gênant qu'autre choses, il ne répond pas aux critères :

• tenir compte des différents modes d'installation (clone depuis dépôt, dc_loader, mise à jour, …)
  
• tenir compte qu'il peut arriver des choses à ces deux dossiers après l'installation

Je propose de le supprimer purement et simplement.

--

Dans :

• admin/index.php
• admin/langs.php
• admin/update.php
• inc\dbschema\upgrade.php
• Le plugin Maintenance
• Le thème Ductile

Le dossier de cache est utilisé sans que son absence ne provoque d'erreur.

On peux déplacer cette vérification dans le Dashboard.
et pourquoi pas, sur chacune de ces pages ... ou pas.

--

Pour le répertoire public, c'est un poil plus complexe.
S'il n'existe pas, la classe dcMedia ne s'instancie pas, ce qui provoque des erreurs bloquante (ex : page blanche), à plusieurs endroit dans l'admin. Partout ou il n'est pas instancier dans un try catch.

Le message d'erreur peux apparaitre sur le dashboard, et dans tout les try catch Ce qui n'est pas le cas pour l'instant.

[lipki]

 https://bitbucket.org/dotclear/dotclear/pull-request/53/ticket-1406/diff

Déplacement du message d'erreur du dossier cache pendant la phase d'installation Vers le Dashboard. Ajout d'un message d'erreur concernant le dossier public, sur le Dashboard. Deux phrases ajouté pour la traduction. "Cache directory %s does not exist." "Directory %s is not writable."

Il reste les erreurs provoqué par dcMedia non instancié. Mais j’attends de voir.

[lipki]

le PR a changer de place mais reste disponible pour de plus ample critique.
 https://bitbucket.org/lipki/dotclear/pull-request/4/ticket-1406/diff

[Franck Paul <carnet.franck.paul@…>]

(In [b956d78197c5]) Merged in lipki/dotclear/ticket #1406-2 (pull request #59) - fixes #1406

[lipki]

Ajout :

Actuellement les messages sont au nombre de quatre :

Cache directory %s does not exist. Cache directory %s is not writable.

Directory %s does not exist. Directory %s is not writable.

---

Modification des messages d'erreurs. 1/ n'afficher ce message qu'aux superadmin 2/ le rendre un peu plus aidant

"Dotclear a besoin d'un répertoire de cache accessible en écriture pour fonctionner correctement. Vous devez en créer un à l'emplacement désigné à la ligne "DC_TPL_CACHE" du fichier inc/config.php de votre installation et lui mettre les droits de lecture, écriture et exécution pour tout le monde."

"Dotclear a besoin d'un répertoire de cache accessible en écriture pour fonctionner correctement. Vous devez contacter votre administrateur."

"Il n'y a pas de répertoire /public/ accessible en écriture à l'emplacement indiqué dans about:config pour "public_path". Vous devez créer ce répertoire à l'emplacement indiqué ou modifier ce paramétrage et lui mettre les droits de lecture, écriture et exécution pour tout le monde."

"Il n'y a pas de répertoire /public/ accessible en écriture à l'emplacement indiqué dans about:config pour "public_path". Vous devez contacter votre administrateur."

[kozlika]

"lui mettre les droits" > "lui attribuer les droits"

[Franck Paul <carnet.franck.paul@…>]

(In [ceda30178c06]) Merged in lipki/dotclear/ticket #1406-2 (pull request #71) - addresses #1406

[franck <carnet.franck.paul@…>]

(In [55ede39e4b9a]) Updates error messages (might be reviewed later), fixes #1406