Fix onDateSelected not called

[fobidlim]

Fix HorizontalCalendarListener#onDateSelected not called when first date selected.
Fix #90

[davcpas1234]

Fix #95

This will only work if the first selected date is the oldest

In order to fix this if the first selected date is the newest:

if ((snapDistance[0] != -1) && (snapDistance[0] != -2) && (snapDistance[0] != 1) && (snapDistance[0] != 2)){
    return snapView;
}

[fobidlim]

Thank you for reviewing.

[oddEvenAndroid]

What is the snapDistance ?

[jjimenez0611]

Yes.. What is the snapDistance ?

where put this code:
if ((snapDistance[0] != -1) && (snapDistance[0] != -2) && (snapDistance[0] != 1) && (snapDistance[0] != 2)){
return snapView;
}

[jjimenez0611]

with this fix, the method calls two times when I made click on item and not when I scroll

if ((snapDistance[0] != -1) && (snapDistance[0] != -2) && (snapDistance[0] != 1) && (snapDistance[0] != 2)){
return snapView;
}

[jjimenez0611]

This solution only works when the user scroll on the calendar, but if the user click on item the method notifyCalendarListener call two times because the snapDistance[0] is equal to 1 always...

[jjimenez0611]

Hi this is my solution, I check if the calendar is on the first position or in the last position, with the method isItemDisabled.. The others solution work in some cases, but for some reason the behavior of the calendar is different when the user scroll or the user click on some item or if the user scroll fast the values of snapDistance always get different values, for example, if the user click on item the values are -1,0 or 1,0 if the user scroll fast the values are 0,0, if the user scroll slow the values are for example -31,0 and then 0,0.... So to solve the problem is call the method when the values are different of 0,0 and are not in the begin or end of the scroll

@OverRide
public View findSnapView(RecyclerView.LayoutManager layoutManager) {
View snapView = super.findSnapView(layoutManager);

    if (calendarView.getScrollState() != RecyclerView.SCROLL_STATE_DRAGGING) {
        int selectedItemPosition;
        if (snapView == null) {
            // no snapping required
            selectedItemPosition = horizontalCalendar.getSelectedDatePosition();
        } else {
            selectedItemPosition = layoutManager.getPosition(snapView);
            int[] snapDistance = calculateDistanceToFinalSnap(layoutManager, snapView);
            boolean beforeItemIsDisable = horizontalCalendar.isItemDisabled(selectedItemPosition-1);
            boolean nextItemIsDisable = horizontalCalendar.isItemDisabled(selectedItemPosition+1);
            if (((snapDistance[0] != 0) || (snapDistance[1] != 0)) && !beforeItemIsDisable && !nextItemIsDisable) {
             return snapView;
            }
        }

        notifyCalendarListener(selectedItemPosition);
    }

    return snapView;
}