Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Date-Holidays CPAN distribution.

Report information
The Basics
Id: 26265
Status: resolved
Priority: 0/
Queue: Date-Holidays

People
Owner: jonasbn [...] cpan.org
Requestors: carlos.fuentes [...] rediris.es
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: 1.00



Subject: Method to upload holidays
Hello, Date::Holidays 0.08 It's more a suggestion than a bug, why don't you include a method for uploading holidays in Date::Holidays??? I'm from Spain, but the module Date::Holidays::ES hasn't got all the official holidays, so it would be nice if your module includes a method to set up a new holidays calendar, so when someone writes a Date::Holidays::XX he should implement this method, so everyone could modify the holidays. Probably you have already thought on this, but contact me if you want more details. Cheers, Carlos
Hello, It is not a bad idea at all, but have you addressed the lacking holidays with the author of Date::Holidays::ES to begin with? Most of the Date::Holidays::* distributions are data focused modules. I will surely investigate if it would be feasible to implement usage of external data. Please get back to me with you ideas and any possible feedback on your talk with Florian Merges. I do not know the reason to as why Florian would leave out some holidays. Either it is a bug on his side, or Spain has a more complex holiday structure. Some of the other distributions, which work with complex holiday structures, either by region or similar, simply have more complex APIs than Date::Holidays::Abstract in general specify, so you will have to have a look at Date::Holidays::DE or Date::Holidays::AU. jonasbn On Wed Apr 11 07:45:08 2007, persequitore wrote: Show quoted text
> Hello, > > Date::Holidays 0.08 > > It's more a suggestion than a bug, why don't you include a method for > uploading holidays in Date::Holidays??? I'm from Spain, but the module > Date::Holidays::ES hasn't got all the official holidays, so it would be > nice if your module includes a method to set up a new holidays calendar, > so when someone writes a Date::Holidays::XX he should implement this > method, so everyone could modify the holidays. > > Probably you have already thought on this, but contact me if you want > more details. > > Cheers, > Carlos
Subject: Re: [rt.cpan.org #26265] Method to upload holidays
Date: Thu, 19 Apr 2007 11:27:52 +0200
To: bug-Date-Holidays [...] rt.cpan.org
From: Carlos Fuentes Bermejo <carlos.fuentes [...] rediris.es>
Download PGP.sig
application/pgp-signature 186b

Message body not shown because it is not plain text.

Hello, Show quoted text
> It is not a bad idea at all, but have you addressed the lacking > holidays > with the author of Date::Holidays::ES to begin with?
No, I didn't contact him. Because the module is ok, but I was looking for a generic module, not associated with any country. Why? because I'm modifying the library Business::Hours (use in RT-IR) to take care about holidays. The number of days off will be defined in a variable of RT-IR, in this way the administrators of RT-IR don't need to install their specific country library (it's possible it doesn't exist, or even it doesn't meet their requirements), so they will just need to define the set of days off, and that's it. Show quoted text
> > Most of the Date::Holidays::* distributions are data focused > modules. I > will surely investigate if it would be feasible to implement usage of > external data. > > Please get back to me with you ideas and any possible feedback on your > talk with Florian Merges. > > I do not know the reason to as why Florian would leave out some > holidays. Either it is a bug on his side, or Spain has a more complex > holiday structure. > > Some of the other distributions, which work with complex holiday > structures, either by region or similar, simply have more complex APIs > than Date::Holidays::Abstract in general specify, so you will have to > have a look at Date::Holidays::DE or Date::Holidays::AU.
I don't think it's a bug, but as AU or DE, each province of Spain has its particular holidays, even each city has particular holidays, i.e Madrid (the city) has the following holidays: city days off + province days off + national days off, so that was the reason to implement Date::Holidays::Generic (sorry about the name), this module implements the functions of Date::Holidays::Super, plus a new method called load_holidays, whose goal is to load a set of specific holidays. Attached you can find the Date::Holidays::Generic and a example of use. The problem is that your class doesn't specify the load_holidays method, and it would be nice if it could be done. Cheers, Carlos

Message body is not shown because sender requested not to inline it.

Message body is not shown because sender requested not to inline it.

-- Carlos Fuentes Bermejo <carlos.fuentes@rediris.es> IRIS-CERT - http://www.rediris.es/cert RedIRIS/Red.es Edificio Bronce Plaza Manuel Gomez Moreno s/n 28020 Madrid. EspaƱa Tel: 91 212 76 20/25 Ext: 5583 www.rediris.es PGP key: http://www.rediris.es/keyserver
Hi, Uh, now I understand. Absolutely marvelous idea, I will have a look at your stuff. I am very positive about the idea, I will see if I can integrate it. Thanks - I will get back to you as soon as I have something running. jonasbn
Subject: Re: [rt.cpan.org #26265] Method to upload holidays
Date: Thu, 19 Apr 2007 12:28:01 +0200
To: bug-Date-Holidays [...] rt.cpan.org
From: Carlos Fuentes Bermejo <carlos.fuentes [...] rediris.es>
Hiya, Show quoted text
> Uh, now I understand. > > Absolutely marvelous idea, I will have a look at your stuff. I am very > positive about the idea, I will see if I can integrate it.
Cool. Show quoted text
> > Thanks - I will get back to you as soon as I have something running.
Meanwhile I'll write documentation for the module, and I'll upload it to cpan, if you agree. Cheers, Carlos
Download PGP.sig
application/pgp-signature 186b

Message body not shown because it is not plain text.

Release 1.00 has just been uploaded to CPAN, with support for this functionality. jonasbn