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

[4.0.0-RC1] Install Tool $this->reload() doesn’t work #267

Closed
ausi opened this issue May 15, 2015 · 1 comment
Closed

[4.0.0-RC1] Install Tool $this->reload() doesn’t work #267

ausi opened this issue May 15, 2015 · 1 comment
Assignees
@leofeyer
Labels
bug
Milestone
4.0

Comments

@ausi
Copy link
Member

ausi commented May 15, 2015

If you go to /contao/install and import an example website you get the error message “The import failed!” even if the import was successful. The same issue appears if you create an admin user, you get the message “Please fill in all fields to create an admin user!”.

This is because we are catching the RedirectResponseException in BackendInstall.php:135 and BackendInstall.php:563 which was thrown in BackendInstall.php:484 and BackendInstall.php:555.
@leofeyer leofeyer added the bug label May 17, 2015
@leofeyer leofeyer added this to the 4.0.0 milestone May 17, 2015
@leofeyer leofeyer self-assigned this May 17, 2015
leofeyer added a commit that referenced this issue May 28, 2015
@leofeyer
Correctly reload the page in the install tool (see #267).
a08ab8c
@leofeyer
Copy link
Member

Fixed in a08ab8c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leofeyer leofeyer

Labels
bug
Projects
None yet
Milestone
4.0
Development

No branches or pull requests
2 participants
@ausi @leofeyer