Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

"Go to main page" not working #7246

Closed
ridz1208 opened this issue Dec 18, 2020 · 6 comments · Fixed by #7258
Closed

"Go to main page" not working #7246

ridz1208 opened this issue Dec 18, 2020 · 6 comments · Fixed by #7258
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)

Comments

@ridz1208
Copy link
Collaborator

Describe the bug
When a user gets timedout due to inactivity they get redirected to the page illustrated below. when clicking "Got to main page" the user should be redirected to the login page.

As you can see in the right panel in the image below, that link is empty. I suspect this has something to do with the removal of the URL configuration in the database

Screenshot from 2020-12-17 21-14-57

@ridz1208 ridz1208 added Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) 23.0.0-testing labels Dec 18, 2020
@driusan
Copy link
Collaborator

driusan commented Dec 18, 2020

Is there a way to get to this page other than "timeout due to inactivity"?

@ridz1208
Copy link
Collaborator Author

I think of you sign out and then try to navigate to a specific module ?

@laemtl
Copy link
Contributor

laemtl commented Dec 21, 2020

We can find this link in 3 templates: 403, 404, and 500.tpl.

@laemtl
Copy link
Contributor

laemtl commented Dec 21, 2020

@ridz1208 Tested on my local environment and everything works.

@ridz1208
Copy link
Collaborator Author

@laemtl you are correct, the issue is on main, not 23.

The issue is because the tpl_data array is populated in an if statement that is never reached.
https://github.com/aces/Loris/blob/main/src/Http/Error.php#L58

@laemtl
Copy link
Contributor

laemtl commented Dec 22, 2020

@ridz1208 I can have a look, thanks !

driusan pushed a commit that referenced this issue Jan 4, 2021
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this issue Jun 29, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants