It appears that when using contextual filters in views the date module doesn't allow less granularity than 1 hour. I have no idea why but since we need this we uncommented the line and it doesn't appear to have a huge performance impact?

So i'm making an issue and assigning a patch to remove this. Maybe someone could shine a little light on why this was done and we can move forward.

CommentFileSizeAuthor
#7 date-views_granularity_issue-2026811-7-D7.patch3.1 KBkala4ek
#5 date-views_granularity_issue-2026811-5-D7.patch3.15 KBkala4ek
#1 2026811-date-views-contextual-argument-1.patch715 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
2026811-date-views-contextual-argument-1.patch715 bytes

Aand the patch file.

jesss’s picture

jesss CreditAttribution: jesss commented
Title: Views agrument in contextual filter doesn't have all granularity » Views argument in contextual filter doesn't have all granularity options
Issue summary: View changes
Status: Active » Needs review

This patch works for me.

Please add this functionality back in. Or at least explain why it was removed and suggest an alternative.

Status: Needs review » Needs work

The last submitted patch, 1: 2026811-date-views-contextual-argument-1.patch, failed testing.

eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin commented
Assigned: Unassigned » eugene.ilyin
kala4ek’s picture

kala4ek
Primary language Russian
Location 🇷🇺 Novosibirsk
CreditAttribution: kala4ek commented
Version: 7.x-2.6 » 7.x-2.x-dev
Assigned: eugene.ilyin » kala4ek
Status: Needs work » Needs review
FileSize
date-views_granularity_issue-2026811-5-D7.patch3.15 KB

Restored "minute" and "second" granularity settings for contextual filter.
Added new settings which allow determinate granularity by argument value.

Patch created by kala4ek and Evgeny_Yudkin.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs review » Needs work
Issue tags: +Needs tests

As for me this patch/change looks like needs tests. Let's get have it.

kala4ek’s picture

kala4ek
Primary language Russian
Location 🇷🇺 Novosibirsk
CreditAttribution: kala4ek commented
Assigned: kala4ek » Unassigned
Status: Needs work » Needs review
FileSize
date-views_granularity_issue-2026811-7-D7.patch3.1 KB

Really restored "minute" and "second" granularity settings.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs review » Fixed

Thanks, commited.

  • podarok committed 3bfb605 on 7.x-2.x authored by kala4ek 
    Issue #2026811 by kala4ek, forgenator: Fixed Views argument in...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.