Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix & Enable PhanNonClassMethodCall #29040

Merged
merged 24 commits into from
Mar 24, 2024

Conversation

mdeweerd
Copy link
Contributor

@mdeweerd mdeweerd commented Mar 23, 2024

Fix & Enable PhanNonClassMethodCall

Note: this PR builds on previous PRs.

⚠️ User::$users can be a list of User or an array with the "hierarchical user information" depending on whether fetchAll or get_full_tree was called (the latter can also be called indirectly).

@mdeweerd mdeweerd changed the title Fix & Enable PhanNonClassMethodCall Fix & Enable PhanNonClassMethodCall (WIP) Mar 23, 2024
@mdeweerd mdeweerd force-pushed the fix/PhanNonClassMethodCall branch 3 times, most recently from cdc354f to 7e193f6 Compare March 23, 2024 20:58
@mdeweerd mdeweerd changed the title Fix & Enable PhanNonClassMethodCall (WIP) Fix & Enable PhanNonClassMethodCal Mar 23, 2024
@mdeweerd mdeweerd changed the title Fix & Enable PhanNonClassMethodCal Fix & Enable PhanNonClassMethodCall Mar 23, 2024
@mdeweerd mdeweerd force-pushed the fix/PhanNonClassMethodCall branch 10 times, most recently from aef1fd0 to 3bd21e3 Compare March 24, 2024 10:14
@mdeweerd mdeweerd marked this pull request as ready for review March 24, 2024 10:23
@mdeweerd mdeweerd force-pushed the fix/PhanNonClassMethodCall branch 2 times, most recently from aadfb85 to 4a0cd45 Compare March 24, 2024 14:04
@mdeweerd mdeweerd force-pushed the fix/PhanNonClassMethodCall branch 2 times, most recently from 9610afb to 7540386 Compare March 24, 2024 14:13
@eldy eldy merged commit bd94ef3 into Dolibarr:develop Mar 24, 2024
7 checks passed
@mdeweerd mdeweerd deleted the fix/PhanNonClassMethodCall branch March 24, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants