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

AWstats Configuration link is bonkers #862

Closed
swelljoe opened this issue Oct 16, 2017 · 2 comments
Closed

AWstats Configuration link is bonkers #862

swelljoe opened this issue Oct 16, 2017 · 2 comments
Labels

Comments

@swelljoe
Copy link

Clicking on "Logs and Reports->AWStats Configuration" causes the "System Settings->Virtualmin Configuration" menu to be opened/highlighted. I believe there's some sort of regex checking for "config.cgi" and it's not smart enough to notice that "virtualmin-awstats/config.cgi" is not the same as "config.cgi".

@iliajie iliajie added the bug label Oct 17, 2017
@iliajie
Copy link
Collaborator

iliajie commented Oct 17, 2017

Yes, it's confirmed. I also noticed that clicking AWstats Report opens over existing page.

What I could do? I could make such links open in the new tab.

It's also should be possible to edit Perl part and make sure that the content that you want to be displayed is wrapped in:

<div data-dcontainer="1">
   ...
</div>

@iliajie
Copy link
Collaborator

iliajie commented Oct 19, 2017

It's fixed. Links are going to be opened on a new tab. All works cool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants