Skip to content
This repository has been archived by the owner on Mar 27, 2018. It is now read-only.

Hint IDEs about property. #65

Closed
Metalaka opened this issue Dec 9, 2014 · 3 comments
Closed

Hint IDEs about property. #65

Metalaka opened this issue Dec 9, 2014 · 3 comments
Assignees
Labels

Comments

@Metalaka
Copy link
Member

Metalaka commented Dec 9, 2014

Most of Hoa's @var tag in property phpDoc don't use the right syntax (according to phpdoc.org & IDEs hints).
The class name shouldn't be written before the var type.

eg:

namespace Hoa\Core\Protocol;

class Protocol {
    /**
     * Collections of sub-components.
     *
     * @var \Hoa\Core\Protocol array
     */
    private $_components   = array();
}

IDEs provide hints for \Hoa\Core\Protocol, but the property is an array...

@Hywan
Copy link
Member

Hywan commented Dec 16, 2014

Yup, it's something I have to fix everywhere… Also, my long term plan is to remove PHPDoc and replace it by Praspel. Don't know what to do for short term plan :-/.

@Metalaka
Copy link
Member Author

Fixed by hoa devtools:cs in Hoa/Devtools/Resource/PHPCSFixer/Fixer/PhpdocVar.php

@Hywan
Copy link
Member

Hywan commented Jun 25, 2015

Cool 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

3 participants