developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#417 closed enhancement (fixed)

gérer les sessions_name

Reported by: bastnic Owned by: Julien
Priority: normal Milestone: Jelix 1.0.3
Component: jelix:core Version: 1.0
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Dans la config des sessions, pouvoir choisir le nom d'une session qui sera après utilisé par session_name.

Attachments (2)

417-jSession-session-name.patch (6.4 KB) - added by Julien 12 years ago.
Patch principal
417-jSession-session-name-ISO-8859-locales.patch (2.4 KB) - added by Julien 12 years ago.
Messages localisés en ISO-8859-1

Download all attachments as: .zip

Change History (22)

comment:1 Changed 12 years ago by laurentj

  • Component changed from jelix to jelix:core
  • Owner set to laurentj

comment:2 Changed 12 years ago by Julien

  • Documentation needed unset
  • Owner changed from laurentj to Julien

Hello,

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

comment:3 Changed 12 years ago by Julien

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.

Changed 12 years ago by Julien

Patch principal

Changed 12 years ago by Julien

Messages localisés en ISO-8859-1

comment:4 Changed 12 years ago by Julien

  • review set to review?

comment:5 Changed 12 years ago by Julien

  • review review? deleted

comment:6 Changed 12 years ago by Julien

  • Status changed from new to assigned

comment:7 Changed 12 years ago by laurentj

review set to review? review deleted.

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

comment:8 Changed 12 years ago by Julien

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.

comment:9 Changed 12 years ago by laurentj

  • review set to review?

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

comment:10 Changed 12 years ago by Julien

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.

comment:11 follow-up: Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.3
  • review changed from review? to review+
+        
+        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.

comment:12 Changed 12 years ago by laurentj

  • Type changed from new feature to enhancement

comment:13 Changed 12 years ago by laurentj

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

comment:14 in reply to: ↑ 11 Changed 12 years ago by Julien

Replying to laurentj:

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

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.

comment:15 Changed 12 years ago by Julien

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

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.

comment:16 Changed 12 years ago by laurentj

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.

comment:17 Changed 12 years ago by laurentj

au fait merci pour le patch :-)

comment:18 Changed 12 years ago by Julien

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.

comment:19 Changed 12 years ago by laurentj

  • Documentation needed set

comment:20 Changed 12 years ago by laurentj

  • Documentation needed unset
Note: See TracTickets for help on using tickets.