yii2-gii icon indicating copy to clipboard operation
yii2-gii copied to clipboard

remove useless function definitions

Open dizews opened this issue 8 years ago • 12 comments

I think it look like more clean then before

dizews avatar Sep 28 '15 18:09 dizews

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

klimov-paul avatar Sep 28 '15 18:09 klimov-paul

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

dizews avatar 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.

dizews avatar Sep 29 '15 13:09 dizews

These should work. @dizews could you check NetBeans?

samdark avatar Sep 29 '15 19:09 samdark

@samdark it work in the NetBeans.

dizews avatar Oct 01 '15 15:10 dizews

@cebe is it a problem for our doc generator?

samdark avatar Oct 01 '15 17:10 samdark

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

cebe avatar Oct 03 '15 21:10 cebe

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

samdark avatar Oct 03 '15 22:10 samdark

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

samdark avatar Jan 30 '17 21:01 samdark

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

cebe avatar Jan 30 '17 21:01 cebe

This can be merged now.

arogachev avatar Dec 06 '21 10:12 arogachev

@arogachev there are conflicts to be resolved first.

samdark avatar Dec 08 '21 09:12 samdark