Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#2906] - CS Added necessary spacing

? ? Success

User tests: Successful: Unsuccessful:

avatar dbhurley
dbhurley
28 Jan 2014
avatar dbhurley dbhurley - open - 28 Jan 2014
avatar dbhurley dbhurley - change - 28 Jan 2014
Labels Added: ?
avatar phproberto
phproberto - comment - 30 Jan 2014

I'm still getting an errors after applying your fix:

$ phpcs --standard=Joomla administrator/components/com_categories/controller.php

FILE: ...berto/www/jcms3x/administrator/components/com_categories/controller.php
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
 49 | ERROR | Expected 2 spaces after the longest type
 50 | ERROR | Expected 2 spaces after the longest variable name
--------------------------------------------------------------------------------

It should be:

    /**
     * Method to display a view.
     *
     * @param   boolean  $cachable   If true, the view output will be cached
     * @param   array    $urlparams  An array of safe url parameters and their variable types, for valid values see {@link JFilterInput::clean()}.
     *
     * @return  JController  This object to support chaining.
     *
     * @since   1.5
     */
avatar dbhurley dbhurley - change - 24 May 2014
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2014-05-24 18:24:40
avatar dbhurley dbhurley - close - 24 May 2014
avatar dbhurley dbhurley - close - 24 May 2014
avatar Bakual
Bakual - comment - 24 May 2014

Fixed the wrong spacing manually.

Add a Comment

Login with GitHub to post a comment