[dev] [commit] r1013 - templates/common templates/common/bits wwwbase wwwbase/ajax wwwbase/js

Cătălin Frâncu cata at francu.com
Thu Oct 3 18:28:36 EEST 2013


Arată bine. Aș elimina doar un if:

> Added: wwwbase/ajax/getImageTags.php

> +$lines = VisualTag::get_all_by_imageId($imageId);
> +
> +if(!empty($lines)) {
> +  foreach($lines as $line) {
> +    ...
> +  }
> +}

If-ul exterior poate fi șters. Dacă nu au fost găsite linii, atunci 
foreach() nu va face nimic oricum.

Cătălin



More information about the Dev mailing list