[dev] [commit] r1012 - phplib

[Cătălin Frâncu]

De acord și eu că SmartyWrap::display() ar trebui să dispară și înlocuită cu ceva existent. Văd că momentan ea apelează doar fetchSkin(), dar această funcție nu prea are sens dacă nu i se specifică și numele template-ului. Cu alte cuvinte, un skin este doar layout-ul general al paginii, dar îi lipsește exact bucata centrală, conținutul propriu-zis.

Probabil ar trebui folosită una din funcțiile displayCommonPageWithSkin(), dacă doriți utilizarea skin-ului preferat de utilizator, sau displayWithoutSkin, dacă doriți să vă scrieți de la zero HTML-ul (foarte rar e cazul).

Cătălin

On 10/01/2013 06:45 AM, Grigoroiu Marian Alexandru wrote:
> Acum e în regulă.
>
>
> On Tue, Oct 1, 2013 at 4:32 PM, <automailer at dexonline.ro <mailto:automailer at dexonline.ro>> wrote:
>
>     Author: alinu
>     Date: Tue Oct  1 16:32:43 2013
>     New Revision: 1012
>
>     Log:
>     Am sters din greseala functia
>     static function display()
>
>     Modified:
>         phplib/SmartyWrap.php
>
>     Modified: phplib/SmartyWrap.php
>     ==============================================================================
>     --- phplib/SmartyWrap.php       Mon Sep 30 13:54:32 2013        (r1011)
>     +++ phplib/SmartyWrap.php       Tue Oct  1 16:32:43 2013        (r1012)
>     @@ -26,6 +26,10 @@
>           self::registerFunction('getDebugInfo', 'SmartyWrap::function_getDebugInfo');
>         }
>
>     +  static function display() {
>     +    print self::fetchSkin();
>     +  }
>     +
>         static function fetchSkin() {
>           $skin = session_getSkin();
>           self::addCss($skin);
>     _______________________________________________
>     Dev mailing list
>     Dev at dexonline.ro <mailto:Dev at dexonline.ro>
>     http://list.dexonline.ro/listinfo/dev
>
>
>
>
> --
> Toate cele bune,
> Grigoroiu Marian Alexandru
>
>
> _______________________________________________
> Dev mailing list
> Dev at dexonline.ro
> http://list.dexonline.ro/listinfo/dev
>