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

bug in isArchivingDisabled() ? #3072

Closed
anonymous-matomo-user opened this issue Mar 28, 2012 · 1 comment
Closed

bug in isArchivingDisabled() ? #3072

anonymous-matomo-user opened this issue Mar 28, 2012 · 1 comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical. worksforme The issue cannot be reproduced and things work as intended.

Comments

@anonymous-matomo-user
Copy link

public function isArchivingDisabled()
{
$processOneReportOnly = !$this->shouldProcessReportsAllPlugins($this->getSegment(), $this->period);
if($processOneReportOnly)
{
// When there is a segment, archiving is not necessary allowed
// If browser archiving is allowed, then archiving is enabled
// if browser archiving is not allowed, then archiving is disabled
if(!$this->getSegment()->isEmpty()
&& !$this->isRequestAuthorizedToArchive()
&& Zend_Registry::get('config')->General->browser_archiving_disabled_enforce
)
{
Piwik::log("Archiving is disabled because of config setting browser_archiving_disabled_enforce=1"winking smiley;
return true;
}
return false;
}
$isDisabled = !$this->isRequestAuthorizedToArchive();
return $isDisabled;
}

Hallo to all,
I got to talk to you about this little function (from archive processing) again. It's essential because if this function returns false archives will be processed. In most cases this is a good thing, but here is my case:
I have a little plugin in piwik 1.6 with some tables. I want to use auto archivieng, berause of performanceissues. The problem here is, some of my tables use segmantation and some not. Now the function above returns always false, if segmantation is used. This leads to inconsistency, because the tables with segmantation will be archived immediately while the tables without have to wait until the crontab triggers the archiving.
So in my eyes this is a bug. Whats your opinion?

I think the source for the problem is this line:
Zend_Registry::get('config')->General->browser_archiving_disabled_enforce
When I disable browser_archiving, it returns '0' never the less. But according to the comments it should return true. This would disable the archiving (like the comments say). Furthermore I already tested to negotiate the line like this:
!Zend_Registry::get('config')->General->browser_archiving_disabled_enforce
and it works fine.
So if you agree with me that this is a bug, I would be glad if someone from the piwik team could fix it.
Thanks
Davud

@mattab
Copy link
Member

mattab commented Apr 4, 2012

I think you're not using 1.7.2 please upgrade, the code looks different around there?

If you still have the problemm with latest release (much possible) please create another ticket with the code triggering a problem (steps to reproduce). thx

This issue was closed.
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. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Development

No branches or pull requests

2 participants