Jump to content

[anomalie] editorial.tpl


Recommended Posts

Je viens de noter une anomalie avec des tests en doublon dans editorial.tpl .

Rien de méchant dans le sens ou ça n'affecte en rien le fonctionnement , mais pas mega terrible en soi :

On fait ici :

si(condition1) alors action 1,
sinon , si(condition1) alors action 1,

Et ça se répète pour plusieurs valeurs :

    {if $editorial->body_logo_subheading}
{$editorial->body_logo_subheading|stripslashes}
   {elseif $editorial->body_logo_subheading}
{$editorial->body_logo_subheading}{/if} 
   {if $editorial->body_title}{$editorial->body_title|stripslashes}
   {elseif $editorial->body_title}{$editorial->body_title|stripslashes}{/if}
   {if $editorial->body_subheading}{$editorial->body_subheading|stripslashes}
   {elseif $editorial->body_subheading}{$editorial->body_subheading|stripslashes}{/if}
   {if $editorial->body_paragraph}{$editorial->body_paragraph|stripslashes}
   {elseif $editorial->body_paragraph}{$editorial->body_paragraph|stripslashes}{/if}



Je ne sais pas depuis quand cette incohérence s'est glissée , mais le code est issue du svn à jour de ce jour..., et existe sur ma 1.4.2.5

  • Like 1
Link to comment
Share on other sites

  • 1 month later...

Bonjour, signalez nous ce genre de choses sur la forge, sur le forum on perd bien souvent les sujets de type bugfix / amélioration (et ce serait encore mieux si on avait déjà le patch ou le bon fichier à mettre, mais je n'en demande pas tant non plus ;)

Link to comment
Share on other sites

Bonjour, signalez nous ce genre de choses sur la forge, sur le forum on perd bien souvent les sujets de type bugfix / amélioration (et ce serait encore mieux si on avait déjà le patch ou le bon fichier à mettre, mais je n'en demande pas tant non plus ;)

 

Bah là il n'y a que quelques delete à opérer , mais entendu j'irais faire un tour sur la forge ;)

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...