Objet : devel-adl
Archives de la liste
- 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
- [Devel] r275 - branches/dui, svn, 20/05/2007
- Re: [Devel] r275 - branches/dui, David MENTRE, 25/05/2007
- Re: [Devel] r275 - branches/dui, Loïc Dayot, 27/05/2007
- Re: [Devel] r275 - branches/dui, David MENTRE, 25/05/2007
Archives gérées par MHonArc 2.6.16.