Accéder au contenu.
Menu Sympa

devel - Re: [Devel] r275 - branches/dui

Objet : devel-adl

Archives de la liste

Re: [Devel] r275 - branches/dui


Chronologique Discussions 
  • From: David MENTRE <dmentre AT linux-france.org>
  • To: devel AT agendadulibre.org
  • Subject: Re: [Devel] r275 - branches/dui
  • Date: Fri, 25 May 2007 21:21:06 +0200
  • List-archive: <http://lolut.utbm.info/pipermail/devel>
  • List-id: Developpement de l'Agenda du Libre <devel.agendadulibre.org>
  • Organization: none

Bonjour,

svn AT agendadulibre.org writes:

> + $return .= " <li><a
> href='moderatetags.php?op=catListTag&id={$catRecord->id}' title=\"".
> sprintf(_("Liste des tags de la cat�gorie %s"), $catRecord->name). "\">".
> sprintf(_("%03s tags"), $catRecord->n). "</a>
> <strong>{$catRecord->name}</strong> <sapn
> class='description'>{$catRecord->description}</span>";
> + if ($catRecord->id>0) $return .= " - <a
> href='moderatetags.php?op=catEdit&id={$catRecord->id}' title=\"".
> sprintf(_("Modifier la cat�gorie %s"), $catRecord->name). "\">".
> _("Modifier"). "</a>";
> + if ($catRecord->n==0) $return .= " - <a
> href='moderatetags.php?op=catDel&id={$catRecord->id}' title=\"".
> sprintf(_("Supprimer la cat�gorie %s"), $catRecord->name). "\">".
> _("Supprimer"). "</a>";

Une remarque de style : pour mon propre code je limite les lignes à 80
caractères. C'est parfois un peu contraingnant mais ça force à bien
structurer le code. Par exemple tu pourrais écrire :

if ($catRecord->n==0)
$return .= " - <a href='moderatetags.php?op=catDel&id={$catRecord->id}'
title=\""
. sprintf(_("Supprimer la cat�gorie %s"), $catRecord->name)
. "\">" . _("Supprimer") . "</a>";

C'est peut-être un peu plus verbeux mais je pense plus lisible.

Amicalement,
d.
--
GPG/PGP key: A3AD7A2A David MENTRE <dmentre AT linux-france.org>
5996 CC46 4612 9CA4 3562 D7AC 6C67 9E96 A3AD 7A2A





Archives gérées par MHonArc 2.6.16.

Haut de le page