@tsteur opened this Issue on May 28th 2018 Member

When you configure segments with "real time archiving" and only later enable "enforce disable browser archiving" so that segments are only processed during cli archiving, then it still shows "AND 1 RESULT IS AVAILABLE, USE UP AND DOWN ARROW KEYS TO NAVIGATE.SEGMENTED REPORTS ARE PRE-PROCESSED (FASTER, REQUIRES CRON)" for these segments even though they are not being pre-processed.

You can reproduce this by creating a regular segment "processed in real time". Then add the following config settings:

[General]
browser_archiving_disabled_enforce = 1
enable_general_settings_admin = 0
enable_browser_archiving_triggering = 0

This is not only problematic when people enable this setting later, but it could be also problematic when moving / migrating a DB to another Matomo where this setting is disabled.

@tsteur commented on May 28th 2018 Member

Or alternatively, Matomo needs to actually pre-process all segments.

@mattab commented on June 18th 2019 Member

Good find!

Or alternatively, Matomo needs to actually pre-process all segments.

For now I would just fix the bug itself. Maybe we could also actually pre-process all segments eventually (but let's create another issue).

@tsteur commented on October 22nd 2019 Member

We'll implement that when browser_archiving_disabled_enforce = 1 we archive all segments. It's easiest fix and most user friendly.

@sgiehl commented on December 2nd 2020 Member

Just looked through the code and if I see that correctly that was already implemented I think with #15499
@diosmosis is that correct?

@diosmosis commented on December 11th 2020 Member

@sgiehl I don't think so, if a segment is set to processed in real time and browser_archiving_disabled_enforce = 1, then core:archive shouldn't launch it. Would be easy to change though, I think the change would be in SegmentArchiving.

@sgiehl commented on December 14th 2020 Member
@diosmosis commented on December 14th 2020 Member

Yes, which means this has been implemented for quite a while. Tested locally and it works as @tsteur describes, so I'll close this for now.

This Issue was closed on December 14th 2020
Powered by GitHub Issue Mirror