goToDate

[alamkanak]

goToDate

[Hugo-Pereira]

bump
got the same problem, goToDate isn't working

[rincyrose]

Got same problem. goToDate is not working as expected.

[akashdev1]

same problem.. Goto works only for date within present month.. other month it wont work.. plz fix

[Hugo-Pereira]

That is weird, with me it doesn't work, even for the current month.
Also, I made it work by multiplying the dateDifference with a value, but that only "works" for devices with the same screen.

Unfortunatelly I don't have enough time to go through the code and try to find out what the real problem is. One thing is for sure though, the width is being calculated wrongly

[Hugo-Pereira]

Hi folks!

I was able to 'solve' the issue through a work around. I am calling init every in the goToDate method.

1. replace the call to init() in the constructor with this

     mToday = Calendar.getInstance();
        mToday.set(Calendar.HOUR_OF_DAY, 0);
        mToday.set(Calendar.MINUTE, 0);
        mToday.set(Calendar.SECOND, 0);
        init();

1. delete the code that sets the mToday variable in the init() method
2. replace the goToDate method with this one:

  public void goToDate(Calendar date) {

        mToday = date;
        init();
}

1. replace the two occurences of

boolean sameDay = isSameDay(day, mToday);

with

boolean sameDay = isSameDay(day, Calendar.getInstance());

Not pretty, but it does the job.

[akashdev1]

where is method edit()??? can u share the work around library? that's working for you..

[Hugo-Pereira]

Sorry, I meant init!

It is easy enough to apply the work around, don't think I should share it...

[akashdev1]

when i call init(date) in method goToDate method i get this error -->The method init() in the type WeekView is not applicable for the arguments (Calendar)

[Hugo-Pereira]

Sorry, I was still changing the fix and copy pasted the wrong version.

I edited my post again, it should be alright now.

I might make a pull request because I did other fixes, but it will take time because I will have to trim out the changes I made to best fit my project.

[akashdev1]

Thanks for the response. i did exactly has you instructed but hard luck it dint work. :-(

[entropitor]

I need to check on this, but when I was hacking on the library, I think I fixed it when adding some brackets to https://github.com/alamkanak/Android-Week-View/blob/master/library/src/main/java/com/alamkanak/weekview/WeekView.java#L1204

changing:

int dateDifference = (int) (date.getTimeInMillis() - today.getTimeInMillis()) / (1000 * 60 * 60 * 24);

to

int dateDifference = (int) ((date.getTimeInMillis() - today.getTimeInMillis()) / (1000 * 60 * 60 * 24));

I'll soon be working some more on the library, and if this is the solution, I will make a PR (if nobody has done it yet by then)

[entropitor]

I checked it now, that solves the problem indeed. I'm currently playing with some other stuff, I'll make a PR after that's done.

[entropitor]

@alamkanak This problem should be fixed and this issue can be closed.

[Hugo-Pereira]

this doesn't work for me.
I checked and mWidthPerDay is 0 when the goToDate is called

edit: apparently I am calling goToDate too soon

[entropitor]

@Hugo-Pereira my PR #53 will fix that

[alamkanak]

Problem fixed. See #69

Do a single and very simple change:

Replace this
mRefreshEvents = true;

with
mRefreshEvents = false;

Method goToDate(Calendar date) should looks like:

public void goToDate(Calendar date) {
    mScroller.forceFinished(true);
    date.set(Calendar.HOUR_OF_DAY, 0);
    date.set(Calendar.MINUTE, 0);
    date.set(Calendar.SECOND, 0);

    mRefreshEvents = false;

    Calendar today = Calendar.getInstance();
    today.set(Calendar.HOUR_OF_DAY, 0);
    today.set(Calendar.MINUTE, 0);
    today.set(Calendar.SECOND, 0);

    int dateDifference = (int) (date.getTimeInMillis() - today.getTimeInMillis()) / (1000 * 60 * 60 * 24);
    mCurrentOrigin.x = - dateDifference * (mWidthPerDay + mColumnGap);


    invalidate();
}