joomla-projects / cms-naked

[READ-ONLY] This repo is no longer in active development. Please see https://github.com/joomla/joomla-cms | Repository for the Frontend Working Group
GNU General Public License v2.0
3 stars 12 forks source link

Code Style in layout file #63

Open Fedik opened 9 years ago

Fedik commented 9 years ago

I not invent anything new, just want to sum up

Example:

/**
 *  This layout render emptiness
 *  
 * @var   Type     $variable    Description 
 * @var   JForm    $form        Form for render 
 * @var   string   $something   Something else
 */
extract($displayData);

is there anything that I missed? Please add it :wink:

phproberto commented 9 years ago

I think we can skip the required part. The doc block tells those overriding it which variables are present so I don't think we need to explicitly set anything as required.

Fedik commented 9 years ago

ok, I changed my post :wink: