Update service unit aggregate function to use new service unit location
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-gui |
Triaged
|
Low
|
Unassigned |
Bug Description
=== modified file 'app/models/
--- app/models/
+++ app/models/
@@ -394,12 +394,12 @@
*/
update_
var aggregate = this.get_
- var sum = Y.Array.reduce(
- Y.Object.
+ var units = service.
+ var sum = units && units.size() || 0;
Simply making this change causes the Google Analytics test to fail because at this point the delta hasn't propagated through to the service so it's units are still 0.
a possible solution:
bcsaller: move the _gaq code into the handlers module and do it on a serviceunitslist after change
Changed in juju-gui: | |
status: | New → Triaged |
importance: | Undecided → Low |