Skip to content
Snippets Groups Projects
Commit 81d93e4f authored by ms217's avatar ms217
Browse files

added GoAccess index file compatiblity to awstats_index.php.master

parent 2ece9ca0
No related branches found
No related tags found
1 merge request!1075Resolve "Add support for GoAccess Log Analyzer"
Loading
  • Contributor

    On my system this breaks the first entry in the menu, it links to goaindex.html but i do not have such a file.

    your condition checks if there is a [DATE]/awsindex.html but for the current month there is no such folder and your condition results in goaindex.html.

    i think for the current month the condition should check if a file awsindex.html exists without the [DATE]-folder-prefix.

    Edited by Robert Breithuber
  • Contributor

    i would suggest a much simpler condition with a reasonable default, something like this should be sufficient:

    $awstatsindex = 'awsindex.html';
    if(!file_exists($value.'/awsindex.html') && file_exists($value.'/goaindex.html')) $awstatsindex = 'goaindex.html';
    Edited by Robert Breithuber
  • Developer

    @brt thanks for your comments. Please open a issue next time, as we can't read the comments on every commit.

  • Thom :tools: @thom

    mentioned in issue #6147 (closed)

    ·

    mentioned in issue #6147 (closed)

    Toggle commit list
  • Contributor

    @thom sorry, i already thought about opening an issue, but i was somehow unsure/busy/lazy, next time ... promised!

    after fixing it through conf-custom/awstats_index.php.master so that it works on my setup, it shifted out of my focus.

  • Developer

    No problem, thanks for your proposed fix. See #6147 (closed) and attached MR.

0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment