refactor code

git-archimport-id: opensource@polytechnique.org--2005/platal--mainline--0.9--patch-92

diff --git a/htdocs/fiche_referent.php b/htdocs/fiche_referent.php
index 55c22ec..3003171 100644 (file)
--- a/htdocs/fiche_referent.php
+++ b/htdocs/fiche_referent.php
@@ -23,22 +23,22 @@
 require_once("xorg.inc.php");
 new_simple_page('fiche_referent.tpl',AUTH_COOKIE);
 
-//$isnetscape = !empty($_SESSION['skin_compatible']);
-
-if (!isset($_REQUEST['user']))
-  exit;
+if (!isset($_REQUEST['user'])) {
+    exit;
+}
 
-//presuppose magic_quote à 'on'
 $reqsql = "SELECT  prenom, nom, user_id, promo, cv, a.alias AS bestalias
              FROM  auth_user_md5 AS u
        INNER JOIN  aliases       AS a ON (u.user_id=a.id AND FIND_IN_SET('bestalias',a.flags))
        INNER JOIN  aliases       AS a1 ON (u.user_id=a1.id AND a1.alias = '{$_REQUEST['user']}' AND a1.type!='homonyme')";
 $result = $globals->db->query($reqsql);
-if (mysql_num_rows($result)!=1)
-        exit;
+if (mysql_num_rows($result)!=1) {
+    exit;
+}
 
-if (list($prenom, $nom, $user_id, $promo, $cv, $bestalias) = mysql_fetch_row($result))
-  mysql_free_result($result);
+if (list($prenom, $nom, $user_id, $promo, $cv, $bestalias) = mysql_fetch_row($result)) {
+    mysql_free_result($result);
+}
 
 $page->assign('prenom', $prenom);
 $page->assign('nom', $nom);
@@ -49,7 +49,7 @@ $page->assign('bestalias', $bestalias);
 
 //recuperation des infos professionnelles
 $reqsql = 
-   "SELECT e.entreprise, s.label as secteur , ss.label as sous_secteur , f.fonction_fr as fonction,
+   "SELECT e.entreprise, s.label as secteur , ss.label as ss_secteur , f.fonction_fr as fonction,
            e.poste, e.adr1, e.adr2, e.adr3, e.cp, e.ville,
           gp.pays, gr.name, e.tel, e.fax
    FROM entreprises AS e
@@ -64,38 +64,23 @@ $reqsql =
 
 $result = $globals->db->query($reqsql);
 
-$i = 0;
-while(list($adr_pro[$i]['entreprise'], $adr_pro[$i]['secteur'], $adr_pro[$i]['ss_secteur'],
-           $adr_pro[$i]['fonction'], $adr_pro[$i]['poste'],
-          $adr_pro[$i]['adr1'], $adr_pro[$i]['adr2'], $adr_pro[$i]['adr3'],
-          $adr_pro[$i]['cp'], $adr_pro[$i]['ville'],
-          $adr_pro[$i]['pays'], $adr_pro[$i]['region'],
-          $adr_pro[$i]['tel'], $adr_pro[$i]['fax']) = mysql_fetch_row($result)){
-    if(!empty($adr_pro[$i]['entreprise']) || !empty($adr_pro[$i]['secteur']) ||
-       !empty($adr_pro[$i]['fonction']) || !empty($adr_pro[$i]['poste']) ||
-       !empty($adr_pro[$i]['adr1']) || !empty($adr_pro[$i]['adr2']) || !empty($adr_pro[$i]['adr3']) ||
-       !empty($adr_pro[$i]['cp']) || !empty($adr_pro[$i]['ville']) ||
-       !empty($adr_pro[$i]['pays']) || !empty($adr_pro[$i]['tel']) || !empty($adr_pro[$i]['fax'])
-      ){
-    $i++;
-   }
+while($tmp = mysql_fetch_assoc($result)) {
+    if (trim(join('',$tmp))) {
+        $adr_pro[] = $tmp;
+    }
 }
-unset($adr_pro[$i]);
-$nb_infos_pro = $i;
-$page->assign('nb_infos_pro', $nb_infos_pro);
 $page->assign_by_ref('adr_pro', $adr_pro);
 mysql_free_result($result);
 
 /////  recuperations infos referent
+$pays = $secteurs = $ss_secteurs = Array();
 
 //expertise
 $result = $globals->db->query("SELECT expertise FROM mentor WHERE uid = $user_id");
-
-if(mysql_num_rows($result) > 0)
 if(list($expertise) = mysql_fetch_row($result)) {
-    mysql_free_result($result);
     $page->assign('expertise', $expertise);
 }
+mysql_free_result($result);
 
 //secteurs
 $result = $globals->db->query("SELECT s.label, ss.label
@@ -103,29 +88,24 @@ $result = $globals->db->query("SELECT s.label, ss.label
                       LEFT JOIN emploi_secteur AS s ON(m.secteur = s.id)
                       LEFT JOIN emploi_ss_secteur AS ss ON(m.secteur = ss.secteur AND m.ss_secteur = ss.id)
                        WHERE uid = $user_id");
-$nb_secteurs = mysql_num_rows($result);
-$i = 1;
-while(list($secteurs[$i], $ss_secteurs[$i]) = mysql_fetch_row($result))
-  $i++;
-unset($secteurs[$i]);
+while(list($sec, $ssec) = mysql_fetch_row($result)) {
+    $secteurs[]    = $sec;
+    $ss_secteurs[] = $ssec;
+}
 mysql_free_result($result);
-$page->assign('nb_secteurs', $nb_secteurs);
-$page->assign_by_ref('secteurs', $secteurs);
-$page->assign_by_ref('ss_secteurs', $ss_secteurs);
 
 //pays
 $result = $globals->db->query("SELECT gp.pays
                        FROM mentor_pays AS m
                       LEFT JOIN geoloc_pays AS gp ON(m.pid = gp.a2)
                        WHERE uid = $user_id");
-$nb_pays = mysql_num_rows($result);
-$i = 1;
-while(list($pays[$i]) = mysql_fetch_row($result)){
-  $i++;
+while (list($p) = mysql_fetch_row($result)) {
+    $pays[] = $p;
 }
-unset($pays[$i]);
 mysql_free_result($result);
-$page->assign('nb_pays', $nb_pays);
+
+$page->assign_by_ref('secteurs', $secteurs);
+$page->assign_by_ref('ss_secteurs', $ss_secteurs);
 $page->assign_by_ref('pays', $pays);
 
 $page->run();

diff --git a/templates/fiche.tpl b/templates/fiche.tpl
index b5a13db..49599c9 100644 (file)
--- a/templates/fiche.tpl
+++ b/templates/fiche.tpl
@@ -79,7 +79,7 @@ function chgMainWinLoc( strPage ) {
         {/if}
         X {$x.promo}{if $x.applis_join} - Formation : {$x.applis_join|smarty:nodefaults}{/if}
         {if $x.is_referent}
-        [<a href="fiche_referent.php?user={$x.forlife}" class='popup'>Ma fiche référent</a>]
+        [<a href="fiche_referent.php?user={$x.forlife}" class='popup2'>Ma fiche référent</a>]
         {/if}
       </div>
     </td>

diff --git a/templates/fiche_referent.tpl b/templates/fiche_referent.tpl
index 922b3d9..7c07436 100644 (file)
--- a/templates/fiche_referent.tpl
+++ b/templates/fiche_referent.tpl
@@ -26,7 +26,7 @@
     <span>X{$promo}&nbsp;-&nbsp;</span> <a href="mailto:{$bestalias}@polytechnique.org">{$bestalias}@polytechnique.org</a>
   </div>
 
-  {if $expertise != '' || ($nb_secteurs > 0)  || ($nb_pays > 0) }
+  {if $expertise != '' || $secteurs|count || $pays|count }
 
   <h2>Informations de référent :</h2>
   
@@ -37,7 +37,7 @@
       <span>{$expertise|nl2br}</span>
     </div>
     {/if}
-    {if $nb_secteurs > 0}
+    {if $secteurs|count}
     <div class="rubrique_referent">
       <em>Secteurs :</em><br />
       <ul>
@@ -47,7 +47,7 @@
       </ul>
     </div>
     {/if}
-    {if $nb_pays > 0}
+    {if $pays|count}
     <div class="rubrique_referent">
       <em>Pays :</em>
       <ul>