Dotclear

Vous n'êtes pas identifié(e).

Annonce

13 février 2024 Sortie de Dotclear 2.29

#101 2015-02-08 19:07:33

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

@Philippe, y a un soucis avec ton lien pour le ticket...

Hors ligne

#102 2015-02-08 19:24:15

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

nikrou a écrit :

Et voici la RC3 qui corrige :
- tous les points rapportés par Pierre. Un énorme merci pour ce super rapport bien détaillé.
- le bug rapporté par Philippe pour l'annulation des filtres.

Tu ne vas pas me trouver drôle à force...

* onglet paramètres, deux messages quand on valide, les deux ont la bonne couleur mais sont au-dessus du breadcrumb (faudrait en avoir qu'un seul, en-dessous) :
[20:09:00] Configuration mise à jour.
[20:09:00] La configuration a été mise à jour.

onglet gérer les pages, là aussi le message est au-dessus, la couleur est ok et c'est horodaté. Idem pour l'ordonnancement.

Donc faut juste les mettre en dessous du breadcrumb.


* La validation w3c n'a pas été traitée. Idem pour le bug de conversion xhtml.

* Pour currywurst, le h2 est en dur et ne devrait pas l'être, sinon la div est ok.
Perso, je remplace ça :

		$title = $w->title ? html::escapeHTML($w->title) : '';

		$res = '<h2>'.$title.'</h2>';
		$res .= '<ul>';

		$pages_list = relatedHelpers::getPublicList($rs);
		foreach ($pages_list as $page) {
			$res .= '<li><a href="'.$page['url'].'">'.html::escapeHTML($page['title']).'</a></li>';
		}

		$res .= '</ul>';

        return $w->renderDiv($w->content_only, 'related-pages-widget '.$w->class, '', $res);

par :

		$res =
		($w->title ? $w->renderTitle(html::escapeHTML($w->title)) : '').
    '<ul>';

		$pages_list = relatedHelpers::getPublicList($rs);
		foreach ($pages_list as $page) {
			$res .= '<li><a href="'.$page['url'].'">'.html::escapeHTML($page['title']).'</a></li>';
		}

		$res .= '</ul>';

    return $w->renderDiv($w->content_only, 'related-pages-widget '.$w->class, '', $res);

Les encodages déconnent toujours et les class ne sont pas protégées par les codes de sécu.

Dernière modification par pierrevg (2015-02-08 22:29:54)

Hors ligne

#103 2015-02-09 07:00:01

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

Pour la validation, je suis responsable des erreurs dans la page d'aide. Et il reste une erreur de Dotclear : le chat n'est pas valide (sifflote...)

En ligne

#104 2015-02-09 12:19:09

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

Philippe a écrit :

Pour la validation, je suis responsable des erreurs dans la page d'aide. Et il reste une erreur de Dotclear : le chat n'est pas valide (sifflote...)

Tu n'as pas l'erreur du duplicate id ?

Hors ligne

#105 2015-02-10 07:00:44

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Et voici la RC4
Au menu :
- correction de l'emplacement des messages
- correction du h2

En revanche pour les codes de sécurité je ne vois pas le soucis. Je n'en mets jamais sur les classes php (un fichier qui ne contient qu'une classe) et je ne sais pas trop comment détecter les problèmes d'encodage ni sur quels fichiers.

Et pour finir je n'ai pas de problème de validation html. Je ne dis pas qu'il n'y en a pas.


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#106 2015-02-10 07:25:32

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

pierrevg a écrit :

Tu n'as pas l'erreur du duplicate id ?

Si, pour moi il reste 4 erreurs :

- l'input <input type="hidden" name="p" value="related" id="p"  /> est utilisé deux fois avec le même id
- il manque des balises dans deux des listes de définition de l'aide intégrée
- l'attribut for d'un label ne se réfère pas à un input <label for="cat_id" class="ib">Pages visibles dans le widget : </label>

Pas de souci d'encodage de fichiers toutefois.

Dernière modification par Philippe (2015-02-10 10:26:00)

En ligne

#107 2015-02-10 07:59:53

Mirovinben
M comme Mathusalem
Lieu : Dole (Jura)
Inscription : 2007-02-06
Site Web

Re : Plugin related et dc 2.7

nikrou a écrit :

- j'ai ajouté les filtres et la pagination sur la liste des pages

Super !...

nikrou a écrit :

- j'ai laissé scindé les pages et les pages visible dans le widget. Cela rendait les choses confuses à mon sens

Je préfère nettement ainsi...

Hors ligne

#108 2015-02-10 10:45:02

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

nikrou a écrit :

- correction de l'emplacement des messages

euh ? non, il faut les placer sous le breadcrumb et non au-dessus.

nikrou a écrit :

En revanche pour les codes de sécurité je ne vois pas le soucis. Je n'en mets jamais sur les classes php (un fichier qui ne contient qu'une classe) et je ne sais pas trop comment détecter les problèmes d'encodage ni sur quels fichiers.

Pour les codes de sécurité, je me réfère bêtement à la doc dc...
Pour l'encodage, j'utilise pspad qui me le signifie gentiment. Je peux t'indiquer quels fichiers sont visés mais comme tu ne sais comment le corriger, n'est-ce pas un peu une perte de temps ?

nikrou a écrit :

Et pour finir je n'ai pas de problème de validation html. Je ne dis pas qu'il n'y en a pas.

Je ne comprends même pas ta phrase... ou alors c'est de l'humour ?

Hors ligne

#109 2015-02-10 17:51:24

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Et voici la RC5.
Au programme :
- correction du html (les labels et les ids en double)
- pour la liste des pages, je n'ai pas (encore) trouvé qui insère le message au dessus du breadcrumb


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#110 2015-02-10 19:02:36

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

@nikrou : pour la validation, il reste juste deux erreurs de mon fait dans les pages d'aide related/locales/(lang)/related_pages.html.

il faut remplacer les deux dernières listes de définition par des paragraphes

<h4>Widget</h4>
    <p>Ce plugin prévoit un widget que vous pourrez placer dans le bandeau de votre choix depuis le panneau Widget de présentation.
	La liste des pages listées dépendra des choix que vous aurez faits dans "ordonnancer la liste publique"</p>
    <h4>Fichier de template</h4>
    <p>Ce plugin ajoute un fichier de thème, external.html. Ce ficher est situé dans le répertoire default-templates du plugin.
	Vous pouvez le personnaliser en en en plaçant une copie dans le répertoire du thème
	que vous utilisez puis en y effectuant vos modifications</p>

et en anglais

<h4>Widget</h4>
    <p>This plugin includes a widget to display a list of related pages on public side.
	The list will depend on choices made in "Arrange public list"</p>
    <h4>Template theme file</h4>
    <p>This plugin includes a template theme file, external.html. This file is located in plugin's default-templates folder.
	You may adapt it to your needs by placing and editing a copy in your themes tpl directory.</p>

Merci :)

En ligne

#111 2015-02-10 19:40:55

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Ok merci Philippe. C'est corrigé.


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#112 2015-02-11 09:52:45

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

Encore quelques efforts, la validation de la page d'édition d'une page connexe est encore à revoir. Pour le fichier inc/page.php

1) <textarea cols="0" rows="0" name="post_content" id="post_content" >/** external content **/</textarea>
Bad value 0 for attribute cols on element textarea: Zero is not a positive integer.

=>mettre une valeur (1) même si le textarea est caché, idem pour rows

2) <input type="file" id="up_file" name="up_file" size="20" />
Attribute size not allowed on element input at this point.

=>supprimer l'attribut size

3) deux occurrences de <option value=""></option>
Element option without attribute label must not be empty.

Là je ne sais pas : ajouter un label sur le contrôle ou supprimer l'option vide ?

4) <label class="required" title="Champ obligatoire" for="page_relatedfile">
The for attribute of the label element must refer to a form control.

C'est un peu laborieux à signaler, si le développement devait continuer, je préfèrerais faire les corrections sur un dépôt Github comme au début de ce sujet ;)

En ligne

#113 2015-02-11 10:03:08

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

Correction : le 3) est le fait de dotclear (date et format du billet)...

En ligne

#114 2015-02-11 19:34:30

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Merci Philippe. Je viens de faire un gros commit. Je ne pousse pas souvent vers github mais je peux le faire plus souvent.

Et la version 1.6-RC6 est disponible


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#115 2015-02-11 20:43:44

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

De mon côté, ça semble tout bon, les deux erreurs de validation restantes sont dues à Dotclear, je créerai un ticket plus tard.

En ligne

#116 2015-02-12 06:53:16

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Merci Philippe. J'attends un deuxième avis et je publie la version 1.6


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#117 2015-02-12 07:49:30

Mirovinben
M comme Mathusalem
Lieu : Dole (Jura)
Inscription : 2007-02-06
Site Web

Re : Plugin related et dc 2.7

Je laisse pierrevg donner son avis.
Pour ma part, ça me convient en l'état.

Hors ligne

#118 2015-02-12 09:03:05

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Mirovinben a écrit :

Je laisse pierrevg donner son avis.
Pour ma part, ça me convient en l'état.

Ok. On attend l'avis du grand inspecteur. :-)


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#119 2015-02-12 10:32:52

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

Vous êtes foutu, vous pouvez plus vous passer de moi...  Hi yaaaaah !

Hors ligne

#120 2015-02-12 11:21:16

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

Mis à part :
- messages doublés à la mauvaise place pour enregistrement des params,
- message à la mauvaise place pour gérer les pages,
- message à la mauvaise place pour ordonner les pages,
- libellé vachement long qui décale tout et rend le formulaire de filtre assez moche "Pages visibles dans le widget : ",
- bug de la conversion wiki -> xhtml d'une page related
- aperçu (bouton en bas de la zone de rédaction) lance bien le lightbox mais plein de vide (ça fonctionne pour les billets et est bugué pour les pages... ticket),
- possible de mettre 80 au lieu de 60 en ligne 40 de \plugins\related\tpl\index.tpl
- et enfin les encodage DOS au lieu de UNIX (LF)
ça me semble bien...

Maintenant que t'as plein de temps, tu peux m'aider sur ça ?

Merci ;)

Hors ligne

#121 2015-02-14 13:22:37

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Merci Pierre pour ta persévérance. J'ai tout corrigé cette fois, même les encodages DOS.
La version 1.6 a été poussé vers dotaddict.


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#122 2015-02-14 17:29:50

pierrevg
Membre
Inscription : 2005-04-13
Site Web

Re : Plugin related et dc 2.7

L'url de support semble fausse dans le plugin (define).
Le message de réussite de la conversion vers xhtml n'est pas de la bonne couleur (gris au lieu de vert), celui de l'enregistrement suite à la conversion est ok par contre. Note que je ne sais pas s'il ne faudrait pas qu'il reste gris, je m'interroge...
Par contre, j'ai beau avoir enregistré cette page incluse en xhtml (suite à la conversion), elle reste en wiki (formatage du texte en sidebar à droite) et avec la barre de Legacy.

C'est pas de la persévérance mais bien du sadisme calculé, ne t'y trompe pas ;p

Hors ligne

#123 2015-02-14 19:46:05

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

pierrevg a écrit :

L'url de support semble fausse dans le plugin (define).
Le message de réussite de la conversion vers xhtml n'est pas de la bonne couleur (gris au lieu de vert), celui de l'enregistrement suite à la conversion est ok par contre. Note que je ne sais pas s'il ne faudrait pas qu'il reste gris, je m'interroge...
Par contre, j'ai beau avoir enregistré cette page incluse en xhtml (suite à la conversion), elle reste en wiki (formatage du texte en sidebar à droite) et avec la barre de Legacy.

C'est pas de la persévérance mais bien du sadisme calculé, ne t'y trompe pas ;p

Et tout ça pour un plugin que tu n'utilises pas en plus ! Du coup je crois que tu as raison c'est du sadisme.

En plus tu n'as même pas noté qu'il y avait un blême avec les icônes dans les favoris !


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

#124 2015-02-14 19:47:28

Philippe
Stagiaire
Lieu : Toulon
Inscription : 2004-06-13
Site Web

Re : Plugin related et dc 2.7

Nouveau problème : les icônes du plugin ont disparu côté admin

En ligne

#125 2015-02-14 20:35:19

nikrou
Membre
Inscription : 2007-10-02
Site Web

Re : Plugin related et dc 2.7

Philippe a écrit :

Nouveau problème : les icônes du plugin ont disparu côté admin

J'ai vu ça aussi mais je ne comprends pas du tout le problème.


Je préfère être atteint de parkinson plutôt qu'alzheimer : je prefère renverser un peu de ma bière plutôt qu'oublier de la boire !

Présentez vos photos simplement avec Phyxo !

Hors ligne

Vous n'êtes pas identifié(e).

Pied de page des forums

Sites map