Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(top counter): display too much resources #5703

Merged
merged 2 commits into from
Sep 29, 2017
Merged

fix(top counter): display too much resources #5703

merged 2 commits into from
Sep 29, 2017

Conversation

lpinsivy
Copy link
Contributor

Add SELECT DISTINCT service_id from centreon_acl table

ref: #5616

Add SELECT DISTINCT to do not count twice services
@lpinsivy lpinsivy added this to the 2.8.14 milestone Sep 25, 2017
@thiuyendang
Copy link
Contributor

Issue has been reproduced in 2.8.13 Centreon Web, centOS6, by using a user linked to several ACL access groups and services linked to several services groups.
Fix is OK.

"AND centreon_acl.group_id IN (".$obj->grouplistStr.")";
$query_svc_status .= "AND s.service_id IN ( " .
"SELECT DISTINCT service_id " .
"FROM centreon_acl ".
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bad coding style

@ganoze ganoze merged commit 3e04f5e into 2.8.x Sep 29, 2017
@ganoze ganoze deleted the lpinsivy-patch-3 branch September 29, 2017 12:17
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.

5 participants