gérer les sessions_name

Hello,

comme je vais écrire la doc sur jSession (ticket #354), je vais rajouter cette possibilité dans le code avant.

Hello,

voici les patches pour arriver à gérer le nom de la session.

Il y a 2 patches, car j'ai rajouté des messages d'erreur, et comme on a 2 charsets (je sais pas si c'est gérable en 1 seul patch, donc voilà, 2 fichiers ;) )

Il suffit de rajouter le paramètre "name" dans la rubrique [sessions] du fichier de config.

Patch principal

Messages localisés en ISO-8859-1

review set to review? review deleted.

Review ou pas review ? Les patchs sont prêt ou dois-je attendre ? :-)

Non non, me suis gouré dans les boutons en allant trop vite ;)

les patchs sont OK.

Je suis pas sur la bonne machine là (en déplacement), alors je ferai mon premier commit dans Jelix une autre fois ;)

Je voulais de toute façon que tu valides le patch avant.

ok, donc avant de faire un commit, il faut faire un review?, et quand je dit ok (review+), tu peux commiter :-)

Ok, j'étais pas certain du process, comme c'était la première fois ;)

J'attends donc ton review+ pour faire mon premier commit.

J'ai déja mis à jour la doc, en spécifiant que le patch n'était pas encore intégré. Quand se sera fait, je modifierai pour dire que c'est dispo à partir de la 1.0.3.

+        
+        if(isset($params['name'])){
+            $params['name'] = trim($params['name']);

Inutile de faire un trim, parse_ini_file le fait déjà.

+            if($params['name'] === ''){
+                throw new jException('jelix~errors.jsession.name.empty');
+            }

Pour des raisons de perfs, je préfère que l'on supprime ce test, et que le message d'erreur pour la regexp dise un truc dans le genre "Le nom de session ne peut être vide et ne doit contenir que des caractères alpha-numériques.". Supprime donc le jsession.name.empty.

Pour les messages en iso, pas besoin de faire un patch séparé, tu peux normalement faire un seul patch pour tout.

Après les corrections indiquées, tu peux commiter. N'oublie pas qu'il faut appliquer le patch à la fois dans trunk/ mais aussi dans branches/1.0.x/ si on veut que ce soit dans la 1.0.3.

J'ai oublié de dire : il y a des lignes vides avec des espaces en trop. Supprimer ces espaces.

Replying to laurentj:

{{{ + + if(isset($paramsname?)){ + $paramsname? = trim($paramsname?); }}} Inutile de faire un trim, parse_ini_file le fait déjà. {{{ + if($paramsname? === ){ + throw new jException('jelix~errors.jsession.name.empty'); + } }}} Pour des raisons de perfs, je préfère que l'on supprime ce test, et que le message d'erreur pour la regexp dise un truc dans le genre "Le nom de session ne peut être vide et ne doit contenir que des caractères alpha-numériques.". Supprime donc le jsession.name.empty. Pour les messages en iso, pas besoin de faire un patch séparé, tu peux normalement faire un seul patch pour tout. Après les corrections indiquées, tu peux commiter. N'oublie pas qu'il faut appliquer le patch à la fois dans trunk/ mais aussi dans branches/1.0.x/ si on veut que ce soit dans la 1.0.3.

Je suis prêt à faire le commit.

Par contre, je ne suis pas du tout sûr de la procédure pour mettre à jour la branche. Je dois faire un checkout de la branche, puis après un svn copy des fichiers qui vont bien du trunk vers la branche, puis un commit global à partir du répertoire principal qui contient à la fois trunk et branches ? Car pour l'instant j'ai que le trunk en local.

Sinon j'ai vu qu'on pouvait faire de la copy directe sur le serveur de trunk vers branches, mais faudrait apparement le faire pour chaque fichier ? Et je pense pas que ca apparaitrait dans un commit comme il faut.

Désolé, mais j'ai jamais utilisé les branches avant ;) Merci de m'éclairer.

Euh, c'est bon, j'ai géré au final, avec la solution que je pensais (checkout global, copie des fichiers en local dans la branche, commit global)

Si y a une autre technique, ça m'intéresse bien entendu ;)

@+

PS : je clos le ticket.

copie des fichiers en local dans la branche,

surtout pas ! Dans le cas présent, tu as eu de la chance. Car si la correction avait concerné des fichiers qui avaient été largement modifié entre la branche et le trunk, du coup tu aurais apporté des modifications incomplètes ou destructrices dans la branche. Il faut toujours soit appliqué le patch avec la commande patch, soit appliquer les modifs à la main. Voir la fr/patchs que je viens de compléter.

au fait merci pour le patch :-)

Ok c'est clair à présent. J'avoue m'y être un peu perdu avec le copy, le merge, etc... Pas très facile à appréhender au début.

Mais Ok, pas de problème si ça fonctionne avec l'application du patch et correction éventuelle à la main.

Pas de problème pour le patch. Demain je modifierai la page de manuel concernée pour annoncer que ce sera disponible à partir de la version 1.0.3.