Selected day "toggles" even though multidate is false

[KaarloH]

When selecting already selected day (and multidate: false), it toggles it off and clears the input field. This can be very confusing especially if using autoclose feature.

[KaarloH]

Quick fix seems to be changing behaviour of _toggle_multidate -method at line 1015:

        _toggle_multidate: function(date){
            var ix = this.dates.contains(date);
            if (!date){
                this.dates.clear();
            }
            else if (ix == -1){
                this.dates.push(date);
            }
            else if ((typeof this.o.multidate === 'number' && this.o.multidate > 1) || this.o.multidate === true){
                this.dates.remove(ix);
            }
            if (typeof this.o.multidate === 'number')
                while (this.dates.length > this.o.multidate)
                    this.dates.remove(0);
        },

[kimbo6365]

This seems to already be discussed in #775

@eternicode says that a feature will be implemented in a coming release, which prevents the date from being deselected. In the meantime, I've implemented a quick workaround:

In the defaults @ line 1394, add a new option, allowDeselection

var defaults = $.fn.datepicker.defaults = {
    allowDeselection: false,
    ...
};

In the _toggle_multidate function @ line 1015, modify the "else if (ix !== -1)" statement to:

else if (ix !== -1 && this.o.allowDeselection){
            this.dates.remove(ix);
        }

[it-can]

@kimbo6365 this works perfectly! PLEASE MERGE THAT FIX!!

[saiful]

@kimbo6365 It works like a charm. I hope it can be merged. 👍

[fernandobritofl]

@kimbo6365 It's working!

[rohitranjan1991]

can anyone change this to the gitub repository so that it can be used with bower . Thanks in advance!!

[sarus]

This issue should be merged with #873 as they relate to the same thing and two different PRs exist.

[frostney]

Works perfectly. Please merge this.

[n1ghtmare]

@kimbo6365 Outstanding, I hope this is merged ! 👍

[swatijoshi]

OSM! I was in trouble how I will achieve this without disturbing your code!
Please merge this same in your latest updates as well as in docs.It is very IMP option.
also add this same in #775
Thanks again :)

[acrobat]

Fixed in #843