z0rglub a écrit:

Aujourd'hui, les sessions sont simplement utilisées pour être associées à un utilisateur, ensuite les données liées à l'utilisateur sont chargées dans $user. Je ne pense pas que l'impact soit si important, ou alors j'ai loupé quelque chose. Je considère que add_session_id, ça touche beaucoup de code, mais c'est simplissime à supprimer.

non l'impact n'est pas important; il y a juste beaucoup de fichiers à modifier mais find, et sed sont mes amis.

z0rglub a écrit:

Oui, une remarque : n'oublies pas mettre un commentaire au dessus de chaque paramètre de $conf dans include/config_default.inc.php à l'image des autres paramètres. Je ne sais pas si tu comptais le faire, mais l'extrait de code laisse supposer que non.

je n'ai pas mis les commentaires sur le forum mais je les mettrais dans le fichier.

z0rglub a écrit:

nicolas a écrit:

p.s: je reporterais toutes ces explications dans le wiki quand ce sera terminé.

Si tu veux, mais j'ai pour idée de réserver les spécifications du wiki pour les évolutions fonctionnelles, pas tellement pour les évolutions techniques. Enfin, si tu veux ajouter des explications dans le wiki, ce ne peut être que bénéfique.

En fait je pensais surtout au wiki pour faire évoluer la documentation.

Je finalise. Je devrais être en mesure de vous livrer quelque chose demain.

Autre problème avec le nouveau système de sessions : le path du cookie est "/", ce qui fait que je ne peux pas avoir 2 installations sur la même machine :-/ (voir comment c'était géré avec l'ancien système).

nicolas a écrit:

Warning: session_start(): Cannot send session cache limiter - headers already sent [...]

Je regarde et je corrige. Cela provient du fait que l'utilisation des sessions ajoute des entêtes http et cela doit se faire avant tout autre envoie au client d'où l'erreur.

Oui, je connais. C'est pour cela que j'ai remplacé la redirection par header HTTP en une redirection via HTML. La raison principale : à partir du moment où un webmaster modifie le fichier de configuration, tu peux être certain que dans 50% des cas, il va ajouter des caractères en dehors des balises <?php ?> (et donc envoie sur la page).

nicolas a écrit:

Pourquoi l'affichage en mode debug ne se fait pas dans le template ? Et même plus généralement pourquoi tous les affichages ne se font pas à travers le template ? Il ne devrait y avoir ni e cho ni print dans le code mis à part la classe template. Tu n'es pas d'accord ?

:-) C'est exactement la réponse que je voulais. Je suis d'accord avec toi pour le mode debug. La première version écrivait dans un vrai fichier de log séparé, mais c'était moins pratique à lire que directement dans le navigateur. Je trouverais un moyen de n'envoyer des octets que via le template.

Par contre, comment résoudre le problème de l'étourdi qui ajoute un espace après le "?>" de son include/config_local.inc.php ?

Pour régler le problème de l'activation de l'affichage des requêtes, voilà les fichiers que j'ai modifié:
template/yoga/footer.tpl:

 </div> <!-- copyright -->
 </div> <!-- the_page -->
-
+<!-- BEGIN debug -->
+  {debug.QUERIES_LIST}
+<!-- END debug -->
 </body>
 </html>

include/functions.inc.php:

 function pwg_query($query)
 {
-  global $conf,$page;
+  global $conf,$page,$debug;
   
   $start = get_moment();
   $result = mysql_query($query) or my_error($query."\n");
@@ -452,12 +452,12 @@
     $output.= '<pre>['.$page['count_queries'].'] ';
     $output.= "\n".$query;
     $output.= "\n".'(this query time : ';
-    $output.= number_format($time, 3, '.', ' ').' s)</b>';
+    $output.= '<b>'.number_format($time, 3, '.', ' ').' s)</b>';
     $output.= "\n".'(total SQL time  : ';
     $output.= number_format($page['queries_time'], 3, '.', ' ').' s)';
-    $output.= '</pre>';
+    $output.= '</pre>'."\n";
     
-    e cho $output;
+    $debug .= $output;
   }
   
   return $result;
@@ -471,9 +471,10 @@
   $now2 = explode( '.', $now[0] );
   $now2 = $now[1].'.'.$now2[1];
   $time = number_format( $now2 - $t2, 3, '.', ' ').' s';
+  $debug .= '<p>';
   $debug.= '['.$time.', ';
   $debug.= $count_queries.' queries] : '.$string;
-  $debug.= "\n";
+  $debug.= "</p>\n";
 }

include/page_tail.php:

@@ -54,6 +54,14 @@
           'SQL_TIME' => number_format($page['queries_time'],3,'.',' ').' s'));
 }
 
+if ($conf['show_queries']) 
+{
+  $template->assign_block_vars(
+    'debug',
+    array('QUERIES_LIST' => $debug)
+  );
+}
+
 //--------------------------------------------------------------------- contact

Qu'en pensez-vous ? Comment j'intègre ça ? Pierrick, veux-tu que je fasse d'abord un ajout dans le bugtracker ?

p.s: comme d'habitude le forum refuse que j'écrive e cho sans espace!

Dernière modification par nicolas (2006-01-21 15:38:16)

(ma remarque sur le patch à "/" est toujours sans réponse)

Nouveau petit soucis : lorsque j'utilise upgrade_feed.php, je charge indirectement include/functions_session.inc.php sans avoir chargé include/config_default.inc.php, du coup $conf n'est pas rempli. Ce qui provoque :

Notice: Undefined variable: conf in /home/z0rglub/work/www/pwg/bsf_perf/include/functions_session.inc.php on line 70
Notice: Undefined variable: conf in /home/z0rglub/work/www/pwg/bsf_perf/include/functions_session.inc.php on line 71
Notice: Undefined variable: conf in /home/z0rglub/work/www/pwg/bsf_perf/include/functions_session.inc.php on line 72
Notice: Undefined variable: conf in /home/z0rglub/work/www/pwg/bsf_perf/include/functions_session.inc.php on line 73

1 upgrades to apply

=== upgrade 5

Column has_high phpwebgallery_images added

Le soucis vient du fait qu'il y a autre chose que des fonctions dans include/functions_session.inc.php : il y a un "if" et des "ini_set". Je pense qu'il faudrait soit déplacer ces bouts de code, soit charger la configuration dans upgrade_feed.php. Sachant qu'upgrade_feed.php n'est utile que sur BSF, il sera supprimé pour les branches stables. Sachant également qu'upgrade_feed.php ne doit pas être disponible que pour les administrateurs.

Si je ne me trompe pas, le remember me ne marche plus avec le nouveau systeme. Le cookie envoye au client expire toujours a la fin de la session.
Il y a un moyen de corriger ca ?

Dernière modification par rvelices (2006-02-07 05:10:56)