yii2-gii remove useless function definitions

remove useless function definitions

Open dizews opened this issue 9 years ago • 12 comments

I think it look like more clean then before

Sep 28 '15 18:09 dizews

No, these definitions are placed for the IDE type-hinting.

Sep 28 '15 18:09 klimov-paul

@klimov-paul I thought it was for type-hinting and made it more clean

Sep 28 '15 18:09 dizews

@klimov-paul do you want to say that my

+ *
+ * @method <?= $modelFullClassName ?>[]|array all($db = null)
+ * @method <?= $modelFullClassName ?>|array|null one($db = null)

would not work as IDE type-hinting? phpstorm work well.

Sep 29 '15 13:09 dizews

These should work. @dizews could you check NetBeans?

Sep 29 '15 19:09 samdark

@samdark it work in the NetBeans.

Oct 01 '15 15:10 dizews

@cebe is it a problem for our doc generator?

Oct 01 '15 17:10 samdark

it does not consider @method annotations yet so application docs will show docs for original implementation.

Oct 03 '15 21:10 cebe

OK. Then this pull request should wait till it's supported.

Oct 03 '15 22:10 samdark

@cebe is there any change in regards to doc tool?

Jan 30 '17 21:01 samdark

created an issue https://github.com/yiisoft/yii2-apidoc/issues/136

Jan 30 '17 21:01 cebe

This can be merged now.

Dec 06 '21 10:12 arogachev

@arogachev there are conflicts to be resolved first.

Dec 08 '21 09:12 samdark