-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Type hinting fixes #8222
Type hinting fixes #8222
Conversation
using scrutinizer patch
using scrutinizer patch
A new inspection was created. |
💣 Test Failed. 💣 |
* Get the directory for the given app. | ||
* If the app is defined in multiple directories, the first one is taken. (false if not found) | ||
* @param string $appid | ||
* @return string|false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only string
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function can return false, not likely but still
@owncloud-bot retest this please |
otherwise 👍 |
🚀 Test Passed. 🚀 |
Not that i think those should return false, but it can happen at the moment |
👍 |
@icewind1991 @DeepDiver1975 please review
changes suggested by scrutinizer