-
Notifications
You must be signed in to change notification settings - Fork 282
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
Setup wizard: Show module dependencies on requirements page #4864
base: main
Are you sure you want to change the base?
Setup wizard: Show module dependencies on requirements page #4864
Conversation
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.
Plus what we discussed :)
modules/setup/application/views/scripts/form/setup-requirements.phtml
Outdated
Show resolved
Hide resolved
035104c
to
20208a4
Compare
6b2819a
to
a2c019e
Compare
modules/setup/library/Setup/Requirement/WebModuleRequirement.php
Outdated
Show resolved
Hide resolved
use Icinga\Module\Setup\Requirement\SetRequirement; | ||
use Icinga\Module\Setup\Requirement\WebModuleRequirement; | ||
|
||
class ModuleDependency |
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.
How does this work? It's not a page nor a wizard.
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.
setup-requirements.phtml
(line 12) gets the wizard and calls getRequirements()
.
The class ModuleDependency
contains only this method, i.e. if no SetupWizard
is provided for the current module (e.g. Cube), this class returns module and lib dependencies.
* Also accept array with two values * evaluate() returns true if `$version` is not defined
4633c02
to
2e8cc09
Compare
* Enhance class WebLibraryRequirement
f4c3c9c
to
136ebab
Compare
Note to myself:
|
fixes #4850