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

Update wizard.php #441

Merged
merged 2 commits into from Apr 9, 2019

Conversation

Projects
None yet
3 participants
@BenniGotchi
Copy link
Collaborator

BenniGotchi commented Mar 22, 2019

Fix fatal error in Install wizard, when trying to import Lansuite XML dump

Inconsitency in variable name removed

refs #440

Update wizard.php
Fix fatal error in Install wizard, when trying to import Lansuite XML dump

Inconsitency in variable name removed

refs #440
@BenniGotchi

This comment has been minimized.

Copy link
Collaborator Author

BenniGotchi commented Mar 22, 2019

@M4LuZ, @andygrunwald : Any Idea, why the automatic checks fail?
There are some warnings about line lenght:

23 | WARNING | Line exceeds 120 characters; contains 137
 |         | characters (Generic.Files.LineLength.TooLong)

Is this causing the fail?

@BenniGotchi BenniGotchi self-assigned this Mar 22, 2019

@andygrunwald

This comment has been minimized.

Copy link
Collaborator

andygrunwald commented Mar 23, 2019

@BenniGotchi The check fails because of this recent change: #403 (comment)

@M4LuZ

M4LuZ approved these changes Apr 9, 2019

@M4LuZ M4LuZ merged commit 8a97373 into master Apr 9, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@M4LuZ M4LuZ deleted the fix/440-install-wizard-xml-import-broken branch Apr 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.