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

Fix triggers to sum event counts properly #2323

Merged
merged 10 commits into from Nov 25, 2018

Conversation

Projects
None yet
2 participants
@knight-of-ni
Copy link
Member

knight-of-ni commented Nov 23, 2018

This pr creates a new update sql file by merging contents of 1.31.25.sql and 1.31.36.sql.

I did this rather quickly and needs verification.

For example, the top most UPDATE statement may be able to be removed or merged into the bottom UPDATE statement. @connortechnology to confirm.

@knight-of-ni

This comment has been minimized.

Copy link
Member

knight-of-ni commented Nov 23, 2018

hang on. Github is at it again.....

Andrew Bauer
@knight-of-ni

This comment has been minimized.

Copy link
Member

knight-of-ni commented Nov 23, 2018

Note: I am intentionally overtiring a ptz control preset because adding these to sql updates is no longer needed. zmcamtool.pl takes care of that now.

@knight-of-ni

This comment has been minimized.

Copy link
Member

knight-of-ni commented Nov 23, 2018

Nope, this isn't right. After a while the counts shown on the web console go back to the incorrect values.

knight-of-ni and others added some commits Nov 24, 2018

@knight-of-ni

This comment has been minimized.

Copy link
Member

knight-of-ni commented Nov 24, 2018

The latest changes to this PR appear to be working. I've got two ZoneMinder systems, one el7 the other f28, that are tracking events correctly now.

@connortechnology connortechnology merged commit 94944ab into ZoneMinder:master Nov 25, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment