Dotclear

Ticket #1406 (closed enhancement: fixed)

Opened 11 years ago

Last modified 11 years ago

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

Reported by: lipki Owned by: lipki
Priority: normal Milestone: 2.5.3
Component: module:core Version: 2.5
Severity: trivial Keywords:
Cc:

Description

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.

Change History

comment:1 Changed 11 years ago by 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.

comment:2 Changed 11 years ago by lipki

J'ai pas accès a ce lien

comment:3 Changed 11 years ago by insy

Devrait être bon maintenant, j'ai passé mon repo en public. Au passage, petite question, comment gères-tu les traductions ?

Version 1, edited 11 years ago by insy (previous) (next) (diff)

comment:4 Changed 11 years ago by kozlika

  • Milestone changed from 2.5.1 to A definir

comment:5 Changed 11 years ago by franck

  • Milestone changed from A definir to 2.5.2

comment:6 Changed 11 years ago by franck

Penser à fermer le ticket #344 au passage.

comment:7 Changed 11 years ago by Evelf

  • Owner changed from xave to Team

comment:8 Changed 11 years ago by franck

  • Milestone changed from 2.5.2 to 2.5.3

comment:9 Changed 11 years ago by lipki

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

comment:10 Changed 11 years ago by 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.

Last edited 11 years ago by lipki (previous) (diff)

comment:11 Changed 11 years ago by 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

comment:12 Changed 11 years ago by franck

  • Owner changed from Team to lipki

comment:13 Changed 11 years ago by 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.

comment:14 Changed 11 years ago by 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.

Last edited 11 years ago by lipki (previous) (diff)

comment:15 Changed 11 years ago by 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

comment:16 Changed 11 years ago by Franck Paul <carnet.franck.paul@…>

  • Status changed from new to closed
  • Resolution set to fixed

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

comment:17 Changed 11 years ago by lipki

  • Status changed from closed to reopened
  • Resolution fixed deleted

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."

comment:18 Changed 11 years ago by kozlika

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

comment:19 Changed 11 years ago by Franck Paul <carnet.franck.paul@…>

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

comment:20 Changed 11 years ago by franck <carnet.franck.paul@…>

  • Status changed from reopened to closed
  • Resolution set to fixed

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

Note: See TracTickets for help on using tickets.

Sites map