zephir icon indicating copy to clipboard operation
zephir copied to clipboard

Type for class properties

Open ovr opened this issue 9 years ago • 4 comments

I suggest to add types for class properties

It gains:

  • @param docblocks for getter/setters
  • Type control for parameter in set method
  • Static analysis
namespace Test;


class Application {
    property <Di> di {get, set};

    /**
     * Optional?
     */
    property <Request> {get, set = null};
}

And require and optional

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9899659-type-for-class-properties?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

ovr avatar Mar 23 '15 08:03 ovr

+1

hylent avatar Mar 23 '15 09:03 hylent

+1

kse300489 avatar Mar 23 '15 09:03 kse300489

:+1:

Green-Cat avatar Mar 23 '15 09:03 Green-Cat

+1

igorgolovanov avatar Mar 23 '15 09:03 igorgolovanov