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.

Sign up for GitHub

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

Jump to bottom

Warnungen beheben wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist #2163

Merged
merged 12 commits into from Nov 3, 2018
Merged

Warnungen beheben wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist #2163

merged 12 commits into from Nov 3, 2018

Conversation

bloep
Copy link
Member

@bloep bloep commented Nov 3, 2018 •
edited by staabm

closes #2147

@staabm staabm requested a review from gharlan November 3, 2018 11:40
gharlan
gharlan approved these changes Nov 3, 2018
View reviewed changes
Copy link
Member

@gharlan gharlan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getestet und funktioniert, sehr cool!

@staabm
Copy link
Member

staabm commented Nov 3, 2018 •
edited

@gharlan @bloep wollen wir in der CLI irgendwo ne info printen, dass es mehr commands gibt, wenn das setup durchgeführt wurde?

sodass man sich nicht wundert dass die console nur so wenig kann wenn man redaxo nur runtergeladen hat und die console öffnet...?

@gharlan
Copy link
Member

gharlan commented Nov 3, 2018

Mir persönlich nicht wichtig, dagegen habe ich aber natürlich nichts.

@staabm staabm changed the title cli ohne db warnings beheben wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist Nov 3, 2018
staabm
staabm reviewed Nov 3, 2018
View reviewed changes
redaxo/src/core/console.php
@@ -22,4 +24,8 @@

$application->setCommandLoader(new rex_console_command_loader());

$command = new rex_command_list();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bitte kommentar, was/warum wir das hier tun

staabm and others added 3 commits November 3, 2018 14:03
@staabm @bloep
Update redaxo/src/core/lib/console/list_command.php
eeaed77 
Co-Authored-By: bloep <der_marcell@hotmail.com>
@staabm @bloep
Update redaxo/src/core/lib/console/list_command.php
f4b08d2 
Co-Authored-By: bloep <der_marcell@hotmail.com>
@bloep
add comment
6abd6c6 
style output
@bloep
Copy link
Member Author

bloep commented Nov 3, 2018

image

@staabm staabm changed the title warnings beheben wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist Warnungen beheben wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist Nov 3, 2018
@staabm staabm merged commit 2e0bb2d into redaxo:master Nov 3, 2018
@bloep bloep deleted the cli-without-db branch November 5, 2018 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@staabm staabm staabm approved these changes

@gharlan gharlan gharlan approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

warnings wenn console commandos ausgeführt werden, obwohl noch kein setup erfolgt ist
3 participants
@bloep @staabm @gharlan