Support Day of Month argument [#199366]

here is a patch that should provide a "Day of Month" argument as described by the third paragraph of the arguments documentation: http://drupal.org/node/54455

it applies to 5.x-1.6 version of the module

Comment	File	Size	Author
#3	views2-day.diff	2.02 KB	mgorven
#3
	views_node.inc_.day_.of_.month_.patch	1.73 KB	guardian

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Brian@brianpuccio.net CreditAttribution: Brian@brianpuccio.net commented 2 January 2008 at 13:25

Status:

Needs review

» Reviewed & tested by the community

I've tested this modification in two Drupal installs as a workaround for my problem and it performs as it should. I see no issues with this patch as it stands, especially since it follows the form of the other arguments, such as year and month.

Please commit.

Thanks, guardian, for posting.

Comment #2

guardian CreditAttribution: guardian commented 5 January 2008 at 20:59

good to know,
thx for the follow up

Comment #3

mgorven CreditAttribution: mgorven commented 10 August 2008 at 19:38

File	Size
views2-day.diff	2.02 KB

I have just coded up this feature for Views 2 on Drupal 6. It seems to work for me, but would be nice if someone else could test it.

For some reason the day displayed in the title was off by one, and I had to add one to correct it. I'm not sure why this is the case since I based it on the created_month field and I would expect the day and month arguments to behave the same.

Comment #4

pkej CreditAttribution: pkej commented 18 August 2008 at 22:04

I wonder if this patch creates the following warnings on my site:

warning: Invalid argument supplied for foreach() in /home/1/v/vogol/www/includes/menu.inc on line 376.
warning: Invalid argument supplied for foreach() in /home/1/v/vogol/www/includes/menu.inc on line 376.

Except from that it works as advertized.