Closed (fixed)
Project:
Drupal core
Component:
archive.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Jul 2004 at 12:00 UTC
Updated:
22 Aug 2006 at 19:45 UTC
Jump to comment: Most recent file
I made a couple of changes to archive.module to facilitate browsing the archives by date.
The attached patch is against 4.4.2 and is untested against CVS.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | _p_9414_archive.patch | 5.09 KB | morbus iff |
| #4 | archive_AH050508b.patch | 5.15 KB | ahoeben |
| #2 | archive_AH050508.patch | 4.78 KB | ahoeben |
| archive_AH040720.patch | 5.22 KB | ahoeben |
Comments
Comment #1
(not verified) commentedPatch don't work with archive.module (drupal 4.4.2).
Hunk #1 FAILED at 1.
Hunk #2 FAILED at 26.
Hunk #3 FAILED at 49.
Hunk #4 FAILED at 64.
Hunk #5 FAILED at 85.
Hunk #6 FAILED at 131.
Hunk #7 FAILED at 218.
Hunk #8 FAILED at 236.
Hunk #9 FAILED at 245.
9 out of 9 hunks FAILED
Comment #2
ahoeben commentedBlast from the past... Not sure what I did wrong to have my original patch fail, but here is a new patch applying the same changes to Drupal 4.6
Comment #3
Cvbge commentedHello,
thanks for your patch, that's a feature I wanted :)
But please change day "0" to "ANY" or similar.
Comment #4
ahoeben commentedNew patch. Changed '0' in day dropdown to 'All days'. You should be able to change that string using a locale (but I have no experience with locales in Drupal).
Also fixed a tiny bug that would cause the year dropdown to stop being useful on jan 1st 2006.
Comment #5
ahoeben commentedPatch applies to cvs version as well...
Comment #6
morbus iffAttached updated patch for HEAD. I'll be fiddling with it some more too.
Comment #7
morbus iffComment #8
adrian commentedI vote we completely ditch the legacy archive module and go with codemonkeyx' module (it's in contrib as archive)
(example : http://codemonkeyx.net/archive)
Comment #9
adrian commentedThe module in contrib needs very little love actually ..
The biggest thing is that there is no css in most themes for it, meaning it can be hard to browse.
If we do use this module, it means we can remove most of the calendar css out of drupal.css , and perhaps move it to modules/event/event.css (provided event still uses it).
The module is also a lot simpler =)
Comment #10
junyor commentedI'm all for it. Open up a new issue and you'll get my +1. :)
Comment #11
morbus iffYep - I started that issue at http://drupal.org/node/29676.
There's a bit to do with it re: HEAD - I don't like the arch_data CSS classes (especially when his other entries use archive), there's no documentation, and it has the same GMT problem as the existing one does. Also, I'm getting bunk dates on my HEAD dev server (days highlighted, but no posts showing when clicked, etc.). I'll be working with all that stuff over on the other issue.
Comment #12
matt_paz commentedAs long as we're adding this, any chance of adding in an optional block that displays a list of links to archives by month?
* August 2005 (244 Items)
* July 2005 (100 Items)
* June 2005 (80 Items)
* May 2005 (60 Items)
* April 2005 (32 Items)
* March 2005 (20 Items)
* February 2005 (10 items)
* January 2005 (2 Items)
Comment #13
morbus iffIt's not a block, but something very similar to that is in the codemonkeyx version.
Comment #14
Crell commentedThis uses the old form code instead of the new Form API, so no longer applies.
Did we ever get around to using the codemonkeyx archive module instead of the old one? If so, is this patch needed anymore? I don't want to close it, but someone who knows the archive module status better than I do should take a quick look. :-)
Comment #15
gábor hojtsyArchive module now removed from core. The codemonkey version was picked by some people, and I created a project for it. Work continues there. Submit feature requests there. BTW this one is already handled because of the nature of that module.
http://drupal.org/project/archive
Comment #16
(not verified) commented