The Archive node does not provide for selection of dates in 2006 [#56916]

Hi,
I am new user of Drupal. I guess you ppl will be able put this bug-fix in right place..
The archives modole provides for a selection of dates . That has no option for 2006.
I made rectification in the code.. Its here

// Prepare the values of the form fields.
$years = drupal_map_assoc(range(2000, 2006));

Just change 2005 to 2006 as shown above and it will work fine.

Comment	File	Size	Author
#4	archive_cvs-head_auto-year.patch	927 bytes	dww
#4
#3	archive_4-6_auto-year.patch	946 bytes	dww
#3
#2	archive_4-6_2010.patch	890 bytes	dww
#2
#1	archive_4-6_2006.patch	890 bytes	dww
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

dww

we/he/they

CreditAttribution: dww commented 2 April 2006 at 08:15

Assigned:

Unassigned

» dww

File	Size
archive_4-6_2006.patch	890 bytes

here's a patch that actually accomplishes this. i checked and this isn't an issue in cvs/4.7, since the end date was moved to 2010 in revision 1.87 (during the form API conversion). however, it seems a little weird to have that many years in the drop-down, though it's a pain to keep changing this code every year. not sure what's the best solution. so, i'll provide 3 patches and let someone else decide
what's the best:

1) hard-code 2006
2) hard-code 2010 (to match head)
3) get the current year via time() and use that

here's the patch for 2006. i'll put the other two in 2 more followups.

Comment #2

dww

we/he/they

CreditAttribution: dww commented 2 April 2006 at 08:15

File	Size
archive_4-6_2010.patch	890 bytes

here's the one for 2010

Comment #3

dww

we/he/they

CreditAttribution: dww commented 2 April 2006 at 08:16

File	Size
archive_4-6_auto-year.patch	946 bytes

and here's my personal favorite (so we never have to mess with this again) -- get the current year automatically.

Comment #4

dww

we/he/they

CreditAttribution: dww commented 2 April 2006 at 08:20

File	Size
archive_cvs-head_auto-year.patch	927 bytes

and, if y'all go with archive_4-6_auto-year.patch, here's a version that applies cleanly to the current cvs head (revision 1.87).

Comment #5

jblack CreditAttribution: jblack commented 8 April 2006 at 05:21

Patch is simple, sensible and safe. The patch is not critical for 4.7.0 (which is already RC) as the listed date is good until 2010. The patch should go in prior to 4.7.1 or 4.7.2.

Comment #6

dww

we/he/they

CreditAttribution: dww commented 8 April 2006 at 05:26

Version:	x.y.z	» 4.6.6
Priority:	Minor	» Normal

thanks for the vote of confidence, jblack. ;) however, this is a fairly nasty bug in 4.6 (where the end date is still 2005). so, i'm changing the version and the priority back so that this can be fixed in 4.6.x asap. i agree it can wait to go into 4.7.1...