Skip to content

Commit e3c4253

Browse files
fnagelfnagel
committed
Calendar: Introduce getter / setter for date and viewDate
1 parent 5ee64dd commit e3c4253

File tree

2 files changed

+71
-47
lines changed

2 files changed

+71
-47
lines changed

‎demos/calendar/dropdown-month-year.html

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,22 +11,26 @@
1111
$.widget( "ui.calendar", $.ui.calendar, {
1212
_buildTitleMonth: function() {
1313
var select = $( "<select>" ),
14-
date = this.date.clone(),
14+
date = this._getDate().clone(),
1515
i = 0,
1616
option;
1717

1818
this._on( select, {
1919
change: function() {
2020
this._off( select );
21-
this.date.setFullDate( this.date.year(), select.val(), this.date.day() );
21+
this._getDate().setFullDate(
22+
this._getDate().year(),
23+
select.val(),
24+
this._getDate().day()
25+
);
2226
this._updateView();
2327
}
2428
} );
2529

2630
for ( ; i < 12; i++ ) {
27-
date.setFullDate( select.val(), i, this.date.day() );
31+
date.setFullDate( select.val(), i, this._getDate().day() );
2832
option = $( "<option>", { val: i, text: date.monthName() } );
29-
if ( this.date.month() === i ) {
33+
if ( this._getDate().month() === i ) {
3034
option.prop( "selected", true );
3135
}
3236
select.append( option );
@@ -43,14 +47,18 @@
4347
this._on( select, {
4448
change: function() {
4549
this._off( select );
46-
this.date.setFullDate( select.val(), this.date.month(), this.date.day() );
50+
this._getDate().setFullDate(
51+
select.val(),
52+
this._getDate().month(),
53+
this._getDate().day()
54+
);
4755
this._updateView();
4856
}
4957
} );
5058

5159
for ( ;i <= current.getFullYear() + 10; i++ ) {
5260
option = $( "<option>", { val: i, text: i } );
53-
if ( this.date.year() === i ) {
61+
if ( this._getDate().year() === i ) {
5462
option.prop( "selected", true );
5563
}
5664
select.append( option );

‎ui/widgets/calendar.js

Lines changed: 57 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -95,19 +95,19 @@ return $.widget( "ui.calendar", {
9595

9696
this._setLocale( this.options.locale, this.options.dateFormat );
9797

98-
this.date = new $.ui.date( this.options.value, this._calendarDateOptions );
99-
this.viewDate = this.date.clone();
100-
this.viewDate.eachDay = this.options.eachDay;
98+
this._setDate( new $.ui.date( this.options.value, this._calendarDateOptions ) );
99+
this._setViewDate( this._getDate().clone() );
100+
this._getViewDate().eachDay = this.options.eachDay;
101101

102102
this._on( this.element, {
103103
"click .ui-calendar-prev": function( event ) {
104104
event.preventDefault();
105-
this.date.adjust( "M", -this.options.numberOfMonths );
105+
this._getDate().adjust( "M", -this.options.numberOfMonths );
106106
this._updateView();
107107
},
108108
"click .ui-calendar-next": function( event ) {
109109
event.preventDefault();
110-
this.date.adjust( "M", this.options.numberOfMonths );
110+
this._getDate().adjust( "M", this.options.numberOfMonths );
111111
this._updateView();
112112
},
113113
"mousedown .ui-calendar-calendar button": "_select",
@@ -155,28 +155,28 @@ return $.widget( "ui.calendar", {
155155
);
156156
return;
157157
case $.ui.keyCode.PAGE_UP:
158-
this.date.adjust( pageAltKey ? "Y" : "M", -1 );
158+
this._getDate().adjust( pageAltKey ? "Y" : "M", -1 );
159159
break;
160160
case $.ui.keyCode.PAGE_DOWN:
161-
this.date.adjust( pageAltKey ? "Y" : "M", 1 );
161+
this._getDate().adjust( pageAltKey ? "Y" : "M", 1 );
162162
break;
163163
case $.ui.keyCode.END:
164-
this.date.setDay( this.date.daysInMonth() );
164+
this._getDate().setDay( this._getDate().daysInMonth() );
165165
break;
166166
case $.ui.keyCode.HOME:
167-
this.date.setDay( 1 );
167+
this._getDate().setDay( 1 );
168168
break;
169169
case $.ui.keyCode.LEFT:
170-
this.date.adjust( "D", -1 );
170+
this._getDate().adjust( "D", -1 );
171171
break;
172172
case $.ui.keyCode.UP:
173-
this.date.adjust( "D", -7 );
173+
this._getDate().adjust( "D", -7 );
174174
break;
175175
case $.ui.keyCode.RIGHT:
176-
this.date.adjust( "D", 1 );
176+
this._getDate().adjust( "D", 1 );
177177
break;
178178
case $.ui.keyCode.DOWN:
179-
this.date.adjust( "D", 7 );
179+
this._getDate().adjust( "D", 7 );
180180
break;
181181
default:
182182
return;
@@ -191,25 +191,25 @@ return $.widget( "ui.calendar", {
191191
},
192192

193193
_updateView: function() {
194-
if ( this.options.numberOfMonths > 1 && this.date.year() === this.viewDate.year() ) {
195-
this.viewDate.adjust( "M", this.options.numberOfMonths *
196-
( this.date.month() > this.viewDate.month() ? 1 : -1 )
194+
if ( this.options.numberOfMonths > 1 && this._getDate().year() === this._getViewDate().year() ) {
195+
this._getViewDate().adjust( "M", this.options.numberOfMonths *
196+
( this._getDate().month() > this._getViewDate().month() ? 1 : -1 )
197197
);
198198
} else {
199-
this.viewDate.setTimestamp( this.date.timestamp() );
199+
this._getViewDate().setTimestamp( this._getDate().timestamp() );
200200
}
201201

202202
this.refresh();
203203
},
204204

205205
_needsRefresh: function() {
206-
if ( this.date.month() !== this.viewDate.month() ||
207-
this.date.year() !== this.viewDate.year()
206+
if ( this._getDate().month() !== this._getViewDate().month() ||
207+
this._getDate().year() !== this._getViewDate().year()
208208
) {
209209

210210
// Check if the needed day is already present in our grid due
211211
// to eachDay option changes (eg. other-months demo)
212-
return !this._getDateElement( this._getDayId( this.date ) ).length;
212+
return !this._getDateElement( this._getDayId( this._getDate() ) ).length;
213213
}
214214

215215
return false;
@@ -220,7 +220,7 @@ return $.widget( "ui.calendar", {
220220
},
221221

222222
_updateDayElement: function( state ) {
223-
var id = this._getDayId( this.date ),
223+
var id = this._getDayId( this._getDate() ),
224224
button = this._getDateElement( id ).children( "button" );
225225

226226
this.grid.attr( "aria-activedescendant", id );
@@ -293,16 +293,16 @@ return $.widget( "ui.calendar", {
293293
_buildMultiplePicker: function() {
294294
var element, header,
295295
rowBreak = $( "<div>" ),
296-
currentDate = this.viewDate,
297-
months = this.viewDate.months( this.options.numberOfMonths - 1 ),
296+
currentDate = this._getViewDate(),
297+
months = this._getViewDate().months( this.options.numberOfMonths - 1 ),
298298
labelledBy = [],
299299
i = 0;
300300

301301
for ( ; i < months.length; i++ ) {
302302

303303
// TODO: Shouldn't we pass date as a parameter to build* fns
304-
// instead of setting this.date?
305-
this.viewDate = months[ i ];
304+
// instead of setting this.viewDate?
305+
this._setViewDate( months[ i ] );
306306
this.gridId = this.id + "-" + i;
307307
labelledBy.push( this.gridId + "-title" );
308308

@@ -326,7 +326,7 @@ return $.widget( "ui.calendar", {
326326
.attr( "aria-labelledby", labelledBy.join( " " ) )
327327
.append( rowBreak );
328328

329-
this.viewDate = currentDate;
329+
this._setViewDate( currentDate );
330330
},
331331

332332
_buildHeaderButtons: function() {
@@ -380,11 +380,11 @@ return $.widget( "ui.calendar", {
380380
},
381381

382382
_buildTitleMonth: function() {
383-
return $( "<span>" ).text( this.viewDate.monthName() );
383+
return $( "<span>" ).text( this._getViewDate().monthName() );
384384
},
385385

386386
_buildTitleYear: function() {
387-
return $( "<span>" ).text( this.viewDate.year() );
387+
return $( "<span>" ).text( this._getViewDate().year() );
388388
},
389389

390390
_buildGrid: function() {
@@ -393,7 +393,7 @@ return $.widget( "ui.calendar", {
393393
tabindex: 0,
394394
"aria-readonly": true,
395395
"aria-labelledby": this.gridId + "-month-label",
396-
"aria-activedescendant": this._getDayId( this.date )
396+
"aria-activedescendant": this._getDayId( this._getDate() )
397397
} );
398398

399399
this._addClass( table, "ui-calendar-calendar" );
@@ -408,8 +408,8 @@ return $.widget( "ui.calendar", {
408408
week = $( "<th>" ),
409409
row = $( "<tr role='row'>" ),
410410
i = 0,
411-
weekDayLength = this.viewDate.weekdays().length,
412-
weekdays = this.viewDate.weekdays();
411+
weekDayLength = this._getViewDate().weekdays().length,
412+
weekdays = this._getViewDate().weekdays();
413413

414414
if ( this.options.showWeek ) {
415415
this._addClass( week, "ui-calendar-week-col" );
@@ -431,7 +431,7 @@ return $.widget( "ui.calendar", {
431431
},
432432

433433
_buildGridBody: function() {
434-
var days = this.viewDate.days(),
434+
var days = this._getViewDate().days(),
435435
i = 0,
436436
rows = "";
437437

@@ -492,7 +492,7 @@ return $.widget( "ui.calendar", {
492492
var attributes, content,
493493
classes = [ "ui-state-default" ];
494494

495-
if ( day === this.date && selectable ) {
495+
if ( day === this._getDate() && selectable ) {
496496
classes.push( "ui-state-focus" );
497497
}
498498
if ( this._isCurrent( day ) ) {
@@ -616,7 +616,7 @@ return $.widget( "ui.calendar", {
616616
},
617617

618618
_headerButtonsState: function() {
619-
var months = this.viewDate.months( this.options.numberOfMonths - 1 ),
619+
var months = this._getViewDate().months( this.options.numberOfMonths - 1 ),
620620
i = 0;
621621

622622
for ( ; i < months.length; i++ ) {
@@ -648,9 +648,9 @@ return $.widget( "ui.calendar", {
648648
for ( ; i < this.options.numberOfMonths; i++ ) {
649649
this.element.find( ".ui-calendar-title" ).eq( i ).replaceWith( this._buildTitle() );
650650
this.element.find( ".ui-calendar-calendar" ).eq( i ).replaceWith( this._buildGrid() );
651-
this.viewDate.adjust( "M", 1 );
651+
this._getViewDate().adjust( "M", 1 );
652652
}
653-
this.viewDate.adjust( "M", -this.options.numberOfMonths );
653+
this._getViewDate().adjust( "M", -this.options.numberOfMonths );
654654
},
655655

656656
_getTranslation: function( key ) {
@@ -664,6 +664,22 @@ return $.widget( "ui.calendar", {
664664
} );
665665
},
666666

667+
_getDate: function() {
668+
return this.date;
669+
},
670+
671+
_setDate: function( date ) {
672+
this.date = date;
673+
},
674+
675+
_getViewDate: function() {
676+
return this.viewDate;
677+
},
678+
679+
_setViewDate: function( date ) {
680+
this.viewDate = date;
681+
},
682+
667683
value: function( value ) {
668684
if ( arguments.length ) {
669685
this.valueAsDate( this._parse( value ) );
@@ -730,11 +746,11 @@ return $.widget( "ui.calendar", {
730746

731747
if ( dateAttributes ) {
732748
this._setLocale( this.options.locale, this.options.dateFormat );
733-
this.date.setAttributes( this._calendarDateOptions );
734-
this.viewDate.setAttributes( this._calendarDateOptions );
749+
this._getDate().setAttributes( this._calendarDateOptions );
750+
this._getViewDate().setAttributes( this._calendarDateOptions );
735751
}
736752
if ( create || refresh ) {
737-
this.viewDate.setTimestamp( this.date.timestamp() );
753+
this._getViewDate().setTimestamp( this._getDate().timestamp() );
738754
}
739755
if ( create ) {
740756
this.element.empty();
@@ -750,7 +766,7 @@ return $.widget( "ui.calendar", {
750766
_setOption: function( key, value ) {
751767
if ( key === "value" ) {
752768
if ( this._isValid( value ) ) {
753-
this.date.setTimestamp( value.getTime() );
769+
this._getDate().setTimestamp( value.getTime() );
754770
} else {
755771
value = null;
756772
}
@@ -778,7 +794,7 @@ return $.widget( "ui.calendar", {
778794
}
779795

780796
if ( key === "eachDay" ) {
781-
this.viewDate.eachDay = value;
797+
this._getViewDate().eachDay = value;
782798
}
783799
}
784800
} );

0 commit comments

Comments
 (0)