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

Error Message notification is not hidden when navigating to a new page #17104

Closed
eric215 opened this issue Jan 17, 2021 · 6 comments · Fixed by #17292
Closed

Error Message notification is not hidden when navigating to a new page #17104

eric215 opened this issue Jan 17, 2021 · 6 comments · Fixed by #17292
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. c: Usability For issues that let users achieve a defined goal more effectively or efficiently. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Milestone

Comments

@eric215
Copy link

eric215 commented Jan 17, 2021

When a server error results in the message below being displayed, transitioning to another page via the menu does not remove it, it continues to be displayed at the top of each page until a hard refresh is done.

Ver 4.1.1

Oops… there was a problem during the request. Maybe the server had a temporary issue, or maybe you requested a report with too much data. Please try again. If this error occurs repeatedly please contact your Matomo administrator for assistance.

@tsteur tsteur added the Can't reproduce (yet) For issues that are reported by several people, but can't be reproduced reliably and need more data label Jan 17, 2021
@tsteur
Copy link
Member

tsteur commented Jan 17, 2021

@eric215 any chance you know the steps to reproduce this? So far I haven't seen this before AFAIK except for maybe #15575 which has a different error message.

@eric215
Copy link
Author

eric215 commented Jan 17, 2021

If you have access to an Apache webserver, use this code in the .htaccess file to force a 406 ("Not Acceptable") error on the "Behaviours --> Transition page".

This will cause the error message I noted to be displayed every time without seriously affecting the other functionality

RewriteCond %{QUERY_STRING} actionName
RewriteRule ^.* - [R=406,L]

@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. c: Usability For issues that let users achieve a defined goal more effectively or efficiently. and removed Can't reproduce (yet) For issues that are reported by several people, but can't be reproduced reliably and need more data labels Jan 17, 2021
@tsteur tsteur added this to the 4.4.0 milestone Jan 17, 2021
@tsteur
Copy link
Member

tsteur commented Jan 17, 2021

Thanks @eric215 I can reproduce it this way Basically, when someone changes the reporting page, then the error messages should be cleared.

@tsteur tsteur added the Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. label Jan 17, 2021
@Findus23
Copy link
Member

It would be great if there could be one fix that fixes this issue, #15575, #15498, #12934 and more related issues at once as it seems like the notification session is simply never cleared unless you log out and login again.

@tsteur
Copy link
Member

tsteur commented Jan 18, 2021

@Findus23 they are a bit unrelated to this one unfortunately. We will eventually work on them though but I suppose these happen less and are much more complicated to solve so it's not as much of a priority right now

@flamisz flamisz self-assigned this Mar 2, 2021
@flamisz
Copy link
Contributor

flamisz commented Mar 2, 2021

I reproduced the bug a different way (maybe it's useful for someone):

I changed the return of the plugins/Transitions/Controller.php -> renderPopover() method:

+ Common::sendResponseCode(406);
- return $view->render();

This way I got the error message on the Behaviour -> Transitions page.

@mattab mattab changed the title Error Message not cleared after Server Error Error Message notification is not hidden when navigating to a new page May 17, 2021
@mattab mattab modified the milestones: 4.7.0, 4.5.0 Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. c: Usability For issues that let users achieve a defined goal more effectively or efficiently. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants