relativedelta(date1, date2).weeks fails

[John Anderson]

Did you forget to implement weeks or am I missing something?

Thanks,
John Anderson

[Gustavo Niemeyer (niemeyer)]

Hi John,

There's really no weeks attribute, but there's a 'weeks' keyword which is why you may be thinking the former exists.

You can easily get weeks by doing "relativedelta(date1, date2).days / 7", though.

[John Anderson (john-appmagic)]

Why don't you have a weeks attribute, you seem to have attribute for everything else?

This leads to awkward code, e.g.:

                recurrence = {'hourly':'hours', 'daily':'days', 'weekly':'weeks', 'monthly':'months', 'yearly': 'years'}[object.recurrence]
                delta = relativedelta(end - start)

                if recurrence == 'weeks': # relativedelta doesn't support weeks for some strange reason
                    delta = int(delta.days / 7)
                else:
                    delta = getattr(delta, recurrence)

John

On Mar 10, 2011, at 6:15 AM, Gustavo Niemeyer wrote:

> Hi John,
>
> There's really no weeks attribute, but there's a 'weeks' keyword which
> is why you may be thinking the former exists.
>
> You can easily get weeks by doing "relativedelta(date1, date2).days /
> 7", though.
>
> ** Changed in: dateutil
> Status: New => Invalid
>
> --
> You received this bug notification because you are a direct subscriber
> of the bug.
> https://bugs.launchpad.net/bugs/727525
>
> Title:
> relativedelta(date1, date2).weeks fails
>
> Status in python-dateutil:
> Invalid
>
> Bug description:
> Did you forget to implement weeks or am I missing something?
>
> Thanks,
> John Anderson
>
> To unsubscribe from this bug, go to:
> https://bugs.launchpad.net/dateutil/+bug/727525/+subscribe

[Gustavo Niemeyer (niemeyer)]

Alright, that's a sane argument.

I'll reopen this.

[Paul G (p-ganssle)]

I migrated this over to the github issue tracker: https://github.com/dateutil/dateutil/issues/45

And I have submitted a fix as well: https://github.com/dateutil/dateutil/pull/49