fix(weekTitle): use the correct year when the week spans 2 years

mattlewis92 · Mar 26, 2016 · 638c741 · 638c741
1 parent d405915
commit 638c741
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/services/calendarTitle.js b/src/services/calendarTitle.js
@@ -11,8 +11,9 @@ angular
     }
 
     function week(viewDate) {
-      var weekTitleLabel = calendarConfig.titleFormats.week;
-      return weekTitleLabel.replace('{week}', moment(viewDate).isoWeek()).replace('{year}', moment(viewDate).format('YYYY'));
+      return calendarConfig.titleFormats.week
+        .replace('{week}', moment(viewDate).isoWeek())
+        .replace('{year}', moment(viewDate).startOf('week').format('YYYY'));
     }
 
     function month(viewDate) {

diff --git a/test/unit/services/calendarTitle.spec.js b/test/unit/services/calendarTitle.spec.js
@@ -20,6 +20,10 @@ describe('calendarTitle', function() {
     expect(calendarTitle.week(calendarDay)).to.equal('Week 18 of 2015');
   });
 
+  it('should use the start of the week for the year number', function() {
+    expect(calendarTitle.week(new Date('2016-01-01'))).to.equal('Week 53 of 2015');
+  });
+
   it('should give the correct month title', function() {
     expect(calendarTitle.month(calendarDay)).to.equal('May 2015');
   });