[dev] [commit] r935 - phplib

[Cătălin Frâncu]

Aici am câteva observații. De fapt erau pentru commit-ul anterior, dar 
mi-au scăpat atunci. :-)

- E bine să pui mesaj la fiecare commit, chiar și de o singură linie.

- E important ca versiunea pentru "case 'visualTag'" să rămână 20, sau 
lista de include-uri să fie numerotată crescător. Mecanismul de acolo ne 
permite să enumerăm modulele JS și CSS dorite în orice ordine, iar el le 
include în ordinea necesară. Cifra 18 suprascria valoarea pentru 
'easyui', care în felul ăsta nu mai era inclus niciodată. Nu mai știu 
dacă am precizat, vă recomand să rulați "meld ." în directorul curent 
înainte de commit, ca să nu scăpați mici greșeli din astea.

- Aș prefera ca template-urile Smarty să se numească la fel ca fișierele 
care le includ, de exemplu wwwwbase/Crawler/index.php și 
templates/crawler/index.ihtml, nu crawler.ihtml (eventual de corectat și 
diferența între literă mare/mică). Altfel, dacă se adună prea multe 
perechi de fișiere cu nume diferite, ne vom orienta mai greu prin cod.

- poți renunța la smartyDisplay și folosi în loc 
SmartyWrap::displayWithoutSkin('crawler/crawler.ihtml'). Știu că numele 
sunt un pic criptice, și scuze pentru asta, dar este important ca totul 
să treacă prin funcția fetch() (SmartyWrap.php linia 97). Se întâmplă 
două chestii notabile acolo: ți se includ automat modulele JS și CSS pe 
care le declari în index.php, și se adaugă niște outputfilters pentru 
Smarty, care țin cont de diverse preferințe ale utilizatorului. 
Outputfilters sunt puțin importante pentru tine, dar oricum.

- Partea cu JS este mai elegant să fie făcută într-un singur loc. Noi am 
scris codul ăsta cu SmartyWrap::addJs('jquery, 'crawler'). N-o fi el 
ideal, dar este unitar. N-aș vrea să ajungem să depindem de N variante 
diferite de jquery, căci asta duce la bătăi de cap pe termen lung.

Cătălin

On 2013-08-18 18:50, automailer at dexonline.ro wrote:
> Author: alinu
> Date: Sun Aug 18 18:50:02 2013
> New Revision: 935
> 
> Log:
> 
> 
> Modified:
>    phplib/SmartyWrap.php
> 
> Modified: phplib/SmartyWrap.php
> ==============================================================================
> --- phplib/SmartyWrap.php	Sun Aug 18 16:21:38 2013	(r934)
> +++ phplib/SmartyWrap.php	Sun Aug 18 18:50:02 2013	(r935)
> @@ -31,6 +31,11 @@
>      print self::fetchSkin();
>    }
> 
> +  static function smartyDisplay($skin) {
> +
> +    self::$theSmarty->display($skin);
> +  }
> +
>    static function fetchSkin() {
>      $skin = session_getSkin();
>      self::addCss($skin);
> @@ -193,9 +198,10 @@
>        case 'wotd':             self::$jsFiles[15] = 'wotd.js?v=1';
>        case 'struct':           self::$jsFiles[16] = 'struct.js?v=3'; 
> break;
>        case 'jcrop':            self::$jsFiles[17] =
> 'jquery.Jcrop.min.js?v=2'; break;
> +
>        case 'easyui':           self::$jsFiles[18] =
> 'jquery.easyui.min.js?v=1'; break;
>        case 'select2':          self::$jsFiles[19] =
> 'select2.min.js?v=2'; break;
> -      case 'visualTag':        self::$jsFiles[20] = 'visualTag.js'; 
> break;
> +      case 'visualTag':        self::$jsFiles[18] = 'visualTag.js'; 
> break;
>        default:
>          FlashMessage::add("Cannot load JS script {$id}");
>          util_redirect(util_getWwwRoot());
> _______________________________________________
> Dev mailing list
> Dev at dexonline.ro
> http://list.dexonline.ro/listinfo/dev