The comment date argument handlers for created all query the comment changed field. There are no argument handlers for the comment created field at all.

CommentFileSizeAuthor
#2 2689657-comment-date-arguments-2.patch4.04 KBcafuego

Comments

cafuego created an issue. See original summary.

cafuego’s picture

cafuego commented
Status: Active » Needs review
StatusFileSize
new 2689657-comment-date-arguments-2.patch4.04 KB

Attached patch re-labels the existing handlers to match their database field and adds additional handlers for the created field.

I suspect that's the best way to fix it, as changing the field on the mislabeled handlers would change the data that existing views return.

I've created a separate issue for Drupal 8.

kattekrab’s picture

kattekrab commented
Title: Comment date views arguments uses incorrect database column » D7 Comment date views arguments uses incorrect database column
jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Needs review » Reviewed & tested by the community

Thanks @cafuego.

  • DamienMcKenna committed 641848a on 7.x-3.x authored by cafuego 
    Issue #2689657 by cafuego: Comment date fields uses incorrect database...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #2855120: Plan for Views 7.x-3.16 release

Committed. Thanks.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented

Thanks! @DamienMcKenna. When did you become a views maintainer? Congrats! and hope to see your love spreading in views issue queue as you have done with other modules. :)

kattekrab’s picture

kattekrab commented

Yaaaaay!!!! :)

Thanks @DamienMcKenna

Status: Fixed » Closed (fixed)

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