Consider adding a flag to make dev tools generate chainable setters

Question

Consider adding a flag to make dev tools generate chainable setters

Bruno Škvorc Apr '13

Created Apr '13	Last Reply Apr '13	Replies 2	Views 483	Votes 1

Bruno Škvorc 25.5k

Apr '13

I would like to see an additional flag go along with "model" or "all-models": --chainable This would make every setter return $this, for chaining purposes, which would then allow us to do this:

$myModel
    ->setProperty1('myProp')
    ->setProperty2(5)
    ->save();

1

Nikolaos Dimopoulos · Answer 1 · 2013-04-14T10:24:47-07:00

@swader That is a great suggestion. I will try and get something done today if not tomorrow and let you know.

Just a FYI, NFRs like this one should be posted on Github as issues. It makes it a lot easier for us to handle :)

Thanks for the suggestion. I will update the issue once the NFR is implemented.

Bruno Škvorc · Answer 2 · 2013-04-15T05:34:36-07:00

Bruno Škvorc
25.5k

Apr '13

Ah, ok, roger that, will post feature requests there from now on, thanks :)

Nikolaos Dimopoulos · Answer 3 · 2013-04-15T13:55:18-07:00

@swader This has been addressed

https://github.com/phalcon/phalcon-devtools/pull/64

Devtools will now create chainable setters by default.

Bruno Škvorc · Answer 4 · 2013-04-15T14:12:19-07:00

Bruno Škvorc
25.5k

Apr '13

Wow, that was super fast, thanks!