Formulaire personnalisable.

Pour discuter de phpBB3 - Pas de support dans cette section !
Pour du support veuillez vous rendre dans la section de support

Modérateur : Équipe

Règles du forum
A lire impérativement : Règlement de phpBB-fr.com

Attention, aucun support n'est donné dans cette section.
Répondre
Avatar du membre
cabot
Equipe site
Equipe site
Messages : 28249
Enregistré le : 13 févr. 2011 à 19:42
Localisation : Rarement sur la plaque
Contact :

Re: Formulaire de présentation.

Message par cabot »

N'oublions pas que le but est de faire une doc généraliste et pas un MOD qui respecterait le coding guideline de phpBB au plus près. :)
Zoddo
Equipe site
Equipe site
Messages : 7074
Enregistré le : 16 mars 2011 à 17:44
Localisation : Laval, France
Contact :

Re: Formulaire de présentation.

Message par Zoddo »

Il est occupé à gamer Geo :mrgreen:
[Doc] Installer une extension - [Extension] Post Models - [Recrutement] Équipe des Extensions
Si vous considérez votre requête comme résolue, merci d'ajouter l'attribut de sujet [Réglé]
Geo
Trifouilleur de code
Messages : 11279
Enregistré le : 25 févr. 2010 à 05:03
Localisation : Nantes, France
Contact :

Re: Formulaire de présentation.

Message par Geo »

Je vais essayer de trouver 5 mins, mais entre la recherche de la KB, la mise à niveau sur .com pour les extensions etc j'ai pas bcp de temps en ce moment :)
Tout mes MODs/Extensions sont maintenant au statut abandonné.
Avatar du membre
cabot
Equipe site
Equipe site
Messages : 28249
Enregistré le : 13 févr. 2011 à 19:42
Localisation : Rarement sur la plaque
Contact :

Re: Formulaire de présentation.

Message par cabot »

Nous avons fait courir Censure pendant 3 semaines sur ce projet depuis le premier jet, et plusieurs intervenants se sont investis pour le faire avancer, peut-on avoir un retour ? Ou à défaut, si le code ne présente pas de faille, peut-on valider cette doc ?
Geo
Trifouilleur de code
Messages : 11279
Enregistré le : 25 févr. 2010 à 05:03
Localisation : Nantes, France
Contact :

Re: Formulaire de présentation.

Message par Geo »

Alors mon retour:

Code : Tout sélectionner

$my_subject   = utf8_normalize_nfc($user->lang['PRESENTATION'] . $user->data['username'], '', true);
$my_text   = utf8_normalize_nfc($message, '', true);
Premièrement la fonction utf8_normalize_nfc n'a qu'un argument, il y a donc eu confusion avec request_var().
De plus les clés de langues sont déjà en utf8 donc l'appel à utf8_normalize_nfc est superflux sachant que les vars concaténés ont déjà été décodées au préalable avec la même fonction.

Code : Tout sélectionner

		'icon_id'  			=> false,
icon_id est un entier, donc y envoie 0 et non un booléen.

Code : Tout sélectionner

		'post_edit_locked'  => 0,
A contrario si on suis la logique post_edit_locked devrait être un booléen car il n'a que deux valeurs possibles.

Code : Tout sélectionner

		$message = $user->lang['SUBMIT_FORM'] . '<br />' . sprintf($user->lang['REDIRECT'], '<a href="' . $redirect_url . '">', '</a>');
Il y a une méthode dédiée dans la classe user plutôt que d'utiliser sprintf en dur : $user->lang('KEY', $arg, $arg ...)

Code : Tout sélectionner

$template->assign_vars(array(
    'S_ERROR'               => (isset($error) && sizeof($error)) ? true : false,
    'ERROR_TITLE'           => $user->lang['ERROR_TITLE'],
	'ERROR_TEXT'            => (isset($error) && sizeof($error)) ? implode('<br />', $error) : '',
));	
L'usage de !empty() serait plus approprié, ne pas oublier que sizeof() parcours le tableau en entier, elle est donc beaucoup plus lourde d'usage quand on veux juste vérifier qu'un tableau est rempli plutôt que d'en récupérer la taille exacte.

Général: Ta nomenclature de variable devrait concorder avec les clés des tableau ou tu les assigne en tête de fichier.
Genre: 'topic_title' => $my_subject,
C'est pas grave en soi, mais ça permettras aux débutant de mieux comprendre le code.

Voilà pour mon retour.
Tout mes MODs/Extensions sont maintenant au statut abandonné.
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Plop,
Merci pour le retour Geo.

Je ne suis pas certain d'avoir tout compris, mais en cherchant je devrais bien finir par trouver.
Donc (si j'ai bien compris) :

Code : Tout sélectionner

$my_subject   = utf8_normalize_nfc($user->lang['PRESENTATION'] . $user->data['username'], '', true);
$my_text   = utf8_normalize_nfc($message, '', true);
Deviens :

Code : Tout sélectionner

$my_subject   = $user->lang['PRESENTATION'] . $user->data['username'];
$my_text   = $message;
Et :

Code : Tout sélectionner

            'icon_id'              => false,
Deviens :

Code : Tout sélectionner

            'icon_id'              => 0,
Ensuite,

Code : Tout sélectionner

            'post_edit_locked'  => 0,
Deviens :

Code : Tout sélectionner

            'post_edit_locked'  => false,
Puis,

Code : Tout sélectionner

$template->assign_vars(array(
    'S_ERROR'               => (isset($error) && sizeof($error)) ? true : false,
    'ERROR_TITLE'           => $user->lang['ERROR_TITLE'],
    'ERROR_TEXT'            => (isset($error) && sizeof($error)) ? implode('<br />', $error) : '',
));
Deviens :

Code : Tout sélectionner

$template->assign_vars(array(
    'S_ERROR'               => !empty($error) ? true : false,
    'ERROR_TITLE'           => $user->lang['ERROR_TITLE'],
    'ERROR_TEXT'            => !empty($error) ? implode('<br />', $error) : '',
)); 
Pour :

Code : Tout sélectionner

        $message = $user->lang['SUBMIT_FORM'] . '<br />' . sprintf($user->lang['REDIRECT'], '<a href="' . $redirect_url . '">', '</a>'); 
Je ne sais pas trop comment m'y prendre.
Un truc comme ça ?

Code : Tout sélectionner

        $message =  $user->lang['SUBMIT_FORM'] . '<br />' . $user->lang['REDIRECT'] .  '<a href="' . $redirect_url . '">' . '</a>'; 
Idem, quand tu dis "Ta nomenclature de variable devrait concorder avec les clés des tableau ou tu les assigne en tête de fichier.". Tu parle des variables comme celle-ci ? :

Code : Tout sélectionner

$realname     = utf8_normalize_nfc(request_var('realname', '', true));
La je suis perdu, quelqu'un à un gps?

Le retour de Geo m'a fait bobo à la tête. :mrgreen:

Ps : Je n'arrive pas à comprendre pourquoi le formulaire poste en double ...
Geo
Trifouilleur de code
Messages : 11279
Enregistré le : 25 févr. 2010 à 05:03
Localisation : Nantes, France
Contact :

Re: Formulaire de présentation.

Message par Geo »

Censure a écrit : Pour :

Code : Tout sélectionner

        $message = $user->lang['SUBMIT_FORM'] . '<br />' . sprintf($user->lang['REDIRECT'], '<a href="' . $redirect_url . '">', '</a>');
Je ne sais pas trop comment m'y prendre.
Un truc comme ça ?

Code : Tout sélectionner

        $message =  $user->lang['SUBMIT_FORM'] . '<br />' . $user->lang['REDIRECT'] .  '<a href="' . $redirect_url . '">' . '</a>';
Je t'ai mis un exemple :cry:

Code : Tout sélectionner

		$message = $user->lang['SUBMIT_FORM'] . '<br />' . $user->lang('REDIRECT', '<a href="' . $redirect_url . '">', '</a>');
		trigger_error($message); 
Censure a écrit : Idem, quand tu dis "Ta nomenclature de variable devrait concorder avec les clés des tableau ou tu les assigne en tête de fichier.". Tu parle des variables comme celle-ci ? :

Code : Tout sélectionner

$realname     = utf8_normalize_nfc(request_var('realname', '', true)); 
La je suis perdu, quelqu'un à un gps?
ça par exemple:

Code : Tout sélectionner

		'topic_title'  		=> $my_subject,
Essaye d'unifier le nom de tes variables dans la mesure du possible.
Tout mes MODs/Extensions sont maintenant au statut abandonné.
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Je t'ai mis un exemple :cry:

Code : Tout sélectionner

    		$message = $user->lang['SUBMIT_FORM'] . '<br />' . $user->lang('REDIRECT', '<a href="' . $redirect_url . '">', '</a>');
    		trigger_error($message); [/quote]
Ah vi, je devais pas être bien réveiller ...
Essaye d'unifier le nom de tes variables dans la mesure du possible.
Ah ok, je ferais ça dés que possible ^^

Edit : Premier post mis à jour.
Edit : Une idée sur le problème de doublon lors de l'envoi? :oops:
FoFa
Resp. des supporters
Resp. des supporters
Messages : 10515
Enregistré le : 15 nov. 2008 à 18:04
Localisation : Auxonne
Contact :

Re: Formulaire de présentation.

Message par FoFa »

Hello

A priori tu as deux "submit_post" d'où le doublon. Suis sur téléphone donc peux pas vérifier de manière concrète.
De retour parmi vous :wink:
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Oh bah mince alors, je n'ai pas vu le doublon ....
C'est corrigé.
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Y aurait-il d'autres corrections à apporter? :P
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Ou pas :mrgreen:
Avatar du membre
Jester
Administrateur
Administrateur
Messages : 36051
Enregistré le : 01 janv. 2005 à 00:00
Localisation : Là, pas loin...
Contact :

Re: Formulaire de présentation.

Message par Jester »

Patience. Ce sont les vacances :mrgreen: mais tu sembles avoir bien travaillé ;)
"Quand le Troll parle, l'homme avisé l'écoute."
Personnaliser son forum - La documentation de phpBB-fr.com - Les téléchargements
N'oubliez pas de modifier votre message en ajoutant le tag [Réglé] quand votre sujet est résolu.
Avatar du membre
Censure
Posteur habitué
Posteur habitué
Messages : 241
Enregistré le : 23 oct. 2012 à 15:37
Localisation : Infiltrate !
Contact :

Re: Formulaire de présentation.

Message par Censure »

Pas de problème Jester.
C'est surtout pour Ayael, qui a demandé ce formulaire. Ça m'embêterait de "donner" quelques choses avec des "failles". :mrgreen:

Ps : Bonne vacances pour ceux qui y sont :wink:
Skouat
Resp. Traducteurs
Resp. Traducteurs
Messages : 15959
Enregistré le : 02 avr. 2008 à 20:47
Contact :

Re: Formulaire de présentation.

Message par Skouat »

Hello !

Passage rapide, car pas beaucoup de temps devant moi...


Ne faudrait-il pas distinguer visuellement les Titres de niveau 1 et les titres de niveau 2 ?
Surtout quand tu présentes un sommaire.
_____________________
Un titre ne contient pas de ponctuation.
Donc les . et , en fin de titre sont à enlever.
_____________________
Ajouter un formulaire à phpBB.
Le 1er titre de la doc est en théorie égale au nom du sujet ;)
_____________________
Tu te répètes en début de doc
Ce tutorial va vous apprendre à ajouter un formulaire personnalisable à phpBB.
Cette documentation vous permettra de disposer d'un formulaire de présentation pour votre forum.
Et perso je prèfère ta dernière formulation.
______________________
Pour cet exemple, nous créerons un simple formulaire. Ce formulaire contiendra :
Dans le chaptire Les fichiers du formulaire. on s'attendrai à voir une énumération de fichier alors que tu fais une énumération de champs de formulaire.
Donc à voir (et ca rejoint le commentaire de répétition précédemment cité) à déplacer carrément ce bloc dans la vrait partie introduction de la doc.
Et à cette emplacement tu mets un commentaire qui va énumérer et indiquer le rôle des fichiers qui vont être créés par la suite.
______________________
formulaire personnalisable, formulaire de présentation, un simple formulaire, etc.
Euh c'est quoi comme type de formulaire sur lequel tu souhaites faire bosser tes futurs lectueurs ?
______________________
Copiez et collez le code ci-dessous dans votre éditeur de texte.
Remplacer 138 par l'id du forum voulu.
Il y a toujours le problème de infinitif, impératif précédemment cité.
Soit tu donnes les instructions aux lecteurs à l'impératif, soit à l'indicatif, mais pas un mixte des deux
______________________
Pensez à modifier l'id du forum ou sera stockés les présentations. Pour cela, chercher :
et non ou
chercher ou cherchez ?


Désolé je m'arrête là pour le moment, car il faut que je file.

A+ :wink:
Répondre

Retourner vers « Discussions sur phpBB3 »

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 2 invités