The views argument handler Drupal\datetime\Plugin\views\Argument\Date has Argument with a capital A in it while it should be argument instead.

This is giving problems for people working on a case sensitive file system, this will be more apparent once #2668236: Make the plugin discovery case sensitive is in.

CommentFileSizeAuthor
#3 2913957-3.patch475 bytesgeertvd

Comments

geertvd created an issue. See original summary.

geertvd’s picture

geertvd commented
Issue summary: View changes
geertvd’s picture

geertvd commented
Status: Active » Needs review
StatusFileSize
new 2913957-3.patch475 bytes
mpdonadio’s picture

mpdonadio
he/him
Primary language English
Location Philadelphia/PA/USA (UTC-5)
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +VDC

Did some searching through core, and I think this is the only oopsie with this. Also looks like all of the `use` and related usages are OK.

This may cause the need for some small rerolls of the RTBC views-related datetime patches, but those should be easy.

This is a bug that is impacting contrib (which almost makes it a Major), so this should be eligible for 8.4.x.

mpdonadio’s picture

mpdonadio
he/him
Primary language English
Location Philadelphia/PA/USA (UTC-5)
commented
Issue tags: +Quickfix

  • larowlan committed 6c9c95e on 8.5.x 
    Issue #2913957 by geertvd, mpdonadio: Fix invalid casing in date...

  • larowlan committed d7570a8 on 8.4.x 
    Issue #2913957 by geertvd, mpdonadio: Fix invalid casing in date...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed as 6c9c95e and pushed to 8.5.
Cherry-picked as d7570a8 and pushed to 8.4.x.

Nomination for the trivial patch of the month award :)

Status: Fixed » Closed (fixed)

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