-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Properly encapsulate require_once for app.php #8372
Conversation
Signed-off-by: Julius Härtl <jus@bitgrid.net>
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.
I had exactly this just minutes ago :)
@@ -133,6 +133,7 @@ public static function loadApps($types = null) { | |||
* load a single app | |||
* | |||
* @param string $app | |||
* @throws Exception |
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 should be added to requireAppFile
😉
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.
Added an annotation for Error to requireAppFile
, but Exception can be thrown by loadApps
though self::disable
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Codecov Report
@@ Coverage Diff @@
## master #8372 +/- ##
============================================
- Coverage 51.76% 51.69% -0.07%
Complexity 25395 25395
============================================
Files 1599 1599
Lines 95110 95111 +1
Branches 1376 1376
============================================
- Hits 49232 49166 -66
- Misses 45878 45945 +67
|
If an app has an appinfo/app.php file, that includes a variable called $app, this would lead to a type error when calling getAppInfo otherwise.