Magento-lts: Code reviews welcome

[sreichel]

Hey

i'm contributing here since years, now i ask for your help. Please do some reviews,

I spent weeks to add doc blocks and annotions to make work easier. These PRs are nearly one year old and still lack of reviews. They are not perfect and can/will be improved ....

I'd like to help with "serious" problems, but I also want to have the code style issues been fixed ....

Thanks!

[sreichel]

Just one review(s) left, thanks @tmotyl.

Review size ... (progress here #725, moved to #416)

nothing left ... (so far)

[sreichel]

Woooha :))

Thanks @colinmollenhour for latest reviews .... with Mage_Eav there is only one "bigger" PR left. Nice progress :+1:

[colinmollenhour]

I'm doing what I can, those things are brutal. :) Nice work though, and thanks also to @tmotyl for being first to review them!

[sreichel]

Only 17 small one left ... nice :+1:

[sreichel]

Thanks @colinmollenhour for merging last ones, thanks for reviews .... I already work on Adminhtml` and it su* .... it is painful.

[tmotyl]

@sreichel would it be possible to add some static analysis with whitelist for the modules which are fixed now, or is it too early?

[sreichel]

@tmotyl yes i can do, but ...

• even for level 0 or 1 you have to exclude sniffs.(e.g missing parent constructor or properbly undefined vars in set in try/catch-blocks)
• exclude some files completly where it is not possible to resolve return values (maybe /ignoreMethod option is added in later phpstan release)
• still use far outdated phpstan 0.8.x ... magento1-extension has to be updated for phpstan 0.12.x. Someone? (there is already _closed_ a PR for v0.12.x)

[tmotyl]

Can we use fork https://github.com/vianetz/phpstan-magento1/ (which sounds to be compatible) directly or does it require additional changes (then I would go for a fork in openmage)

[sreichel]

I'll try it out over the weekend.