diff options
author | Stefan Giehl <stefan@matomo.org> | 2020-02-18 01:33:56 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-02-18 01:33:56 +0300 |
commit | e8cb9c2ae10dceec3fde93acb58569bb8d9c1b59 (patch) | |
tree | 6deddcf330a3bdb3c1a1098cab170bae4f5270b4 /plugins/CoreHome | |
parent | 7f95e432da0a0fb2b648a4c4bb5c42a7d004d6a8 (diff) |
Fixes some comparison issues (#15583)
* ensure new date objects are returned
* return chosen period when comparing with previous year
Diffstat (limited to 'plugins/CoreHome')
-rw-r--r-- | plugins/CoreHome/angularjs/common/services/periods.js | 2 | ||||
-rw-r--r-- | plugins/CoreHome/angularjs/period-selector/period-selector.controller.js | 11 |
2 files changed, 10 insertions, 3 deletions
diff --git a/plugins/CoreHome/angularjs/common/services/periods.js b/plugins/CoreHome/angularjs/common/services/periods.js index 9e6ba80f03..95f6f0c3dd 100644 --- a/plugins/CoreHome/angularjs/common/services/periods.js +++ b/plugins/CoreHome/angularjs/common/services/periods.js @@ -63,7 +63,7 @@ }, getDateRange: function () { - return [this.dateInPeriod, this.dateInPeriod]; + return [new Date(this.dateInPeriod.getTime()), new Date(this.dateInPeriod.getTime())]; } }; diff --git a/plugins/CoreHome/angularjs/period-selector/period-selector.controller.js b/plugins/CoreHome/angularjs/period-selector/period-selector.controller.js index 58ed8295e3..83e100947e 100644 --- a/plugins/CoreHome/angularjs/period-selector/period-selector.controller.js +++ b/plugins/CoreHome/angularjs/period-selector/period-selector.controller.js @@ -257,9 +257,16 @@ currentDateRange[0].setFullYear(currentDateRange[0].getFullYear() - 1); currentDateRange[1].setFullYear(currentDateRange[1].getFullYear() - 1); + if (vm.selectedPeriod === 'range') { + return { + comparePeriods: ['range'], + compareDates: [piwikPeriods.format(currentDateRange[0]) + ',' + piwikPeriods.format(currentDateRange[1])], + }; + } + return { - comparePeriods: ['range'], - compareDates: [piwikPeriods.format(currentDateRange[0]) + ',' + piwikPeriods.format(currentDateRange[1])], + comparePeriods: [vm.selectedPeriod], + compareDates: [piwikPeriods.format(currentDateRange[0])], }; } else { console.warn("Unknown compare period type: " + vm.comparePeriodType); |