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

Share this post


Link to post
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 ;)

Share this post


Link to post
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 ;)

Share this post


Link to post
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
 Share

×
×
  • Create New...

Important Information

Cookies ensure the smooth running of our services. Using these, you accept the use of cookies. Learn More