Changeset 4899

Timestamp:

02/10/2017 01:26:17 AM (8 years ago)

Author:

coreymckrill

Message:

Budget Tool: Correctly calculate Cost Per Person Per Day

The formula wasn't taking into account the number of days the event lasted.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/budget-tool.js (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/budget-tool.js

@@ -100 +100 @@
 
         render: function() {
-            var data = {
-                'income': 0,
-                'expenses': 0,
-                'variance': 0,
-                'variance_raw': 0,
-                'per_person': 0
-            };
+            var attendees = wcb.table.collection.findWhere({type: 'meta', name: 'attendees'}),
+                days      = wcb.table.collection.findWhere({type: 'meta', name: 'days'}),
+                data = {
+                    'income': 0,
+                    'expenses': 0,
+                    'variance': 0,
+                    'variance_raw': 0,
+                    'per_person': 0
+                };
 
             _.each(wcb.table.collection.where({type: 'income'}), function(item) {
@@ -118 +120 @@
             data['variance'] = data['income'] - data['expenses'];
             data['variance_raw'] = data['variance'];
-            var attendees = wcb.table.collection.findWhere({type: 'meta', name: 'attendees'});
-            data['per_person'] = attendees ? data['expenses'] / attendees.get('value') : 0;
+            data['per_person'] = (attendees && days) ? data['expenses'] / attendees.get('value') / days.get('value'): 0;
 
             data = _.mapObject(data, function(v, k) {