Skip to content
This repository was archived by the owner on Jan 30, 2026. It is now read-only.

fix div by zero by workaround#70

Open
ProxiBlue wants to merge 1 commit intoAOEpeople:masterfrom
ProxiBlue:master
Open

fix div by zero by workaround#70
ProxiBlue wants to merge 1 commit intoAOEpeople:masterfrom
ProxiBlue:master

Conversation

@ProxiBlue
Copy link

I am not entirely sure what causes the zero'd values, as I did not dig that deep.

Randomly, $this->stackLog['timetracker_0'][$metric . '_total'] is null or zero, so I bypassed teh issue by detecting this, and then setting the wanted result values to 0 (since we want to div by 0)

I have not used the module long enough to see what issue this causes, but the error itself is obviously gone.

Interesting is that this only started after session were set to Redix via

https://github.com/colinmollenhour/Cm_RedisSession

This is a loose assumption, not specifically tested, but all was fine until it was added.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant