There's a minor typo in the Date docblock in both
- core/modules/views/src/Plugin/views/argument/Date.php
- core/modules/datetime/src/Plugin/views/argument/Date.php

The @see tag incorrectly points to
\Drupal\views\ManyTonOneHelper
instead of
\Drupal\views\ManyToOneHelper

This typo is exposed in the public API documentation, e.g.
https://api.drupal.org/api/drupal/core%21modules%21datetime%21src%21Plug...

(I'm not sure of the way to raise/address things like this, but a tiny patch will follow in case it's of use.)

Issue fork drupal-3284665

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

    Comments

    Jimaginary created an issue. See original summary.

    jrsouth’s picture

    StatusFileSize
    new1.14 KB

    Tiniest patch file ever is attached, fixing the typo in two affected files.

    cilefen’s picture

    Version: 9.3.x-dev » 9.4.x-dev
    Status: Active » Needs review
    longwave’s picture

    Status: Needs review » Reviewed & tested by the community

    Thanks, there are some things the spell checker can't catch! ("ton" is a word in this case)

    Status: Reviewed & tested by the community » Needs work

    The last submitted patch, 2: manytononehelper_typo-3284665-2.patch, failed testing. View results

    arunkumark’s picture

    Here is the patch for 9.5.x and Drupal 10.0.x.

    rakhi soni’s picture

    Assigned: jrsouth » Unassigned
    Status: Needs work » Needs review
    StatusFileSize
    new1.14 KB

    kindly review patch for 9.4x branch,,

    Status: Needs review » Needs work

    The last submitted patch, 7: manytoonehelper-typo-3284665-7.patch, failed testing. View results

    rassoni’s picture

    StatusFileSize
    new2.04 KB
    new3.16 KB
    rassoni’s picture

    Status: Needs work » Needs review
    longwave’s picture

    Status: Needs review » Needs work

    The other changes in #9 are out of scope, we should only be fixing the two spelling mistakes here.

    arunkumark’s picture

    Status: Needs work » Needs review
    StatusFileSize
    new1.13 KB

    As per the comment #11, re-rolled the patch #10.

    longwave’s picture

    Status: Needs review » Reviewed & tested by the community

    Thank you.

    • catch committed 17381d9 on 10.0.x
      Issue #3284665 by arunkumark, Rashmisoni, Jimaginary, Rakhi Soni,...
    • catch committed 436f587 on 10.1.x
      Issue #3284665 by arunkumark, Rashmisoni, Jimaginary, Rakhi Soni,...
    • catch committed 532a97e on 9.4.x
      Issue #3284665 by arunkumark, Rashmisoni, Jimaginary, Rakhi Soni,...
    • catch committed de637ad on 9.5.x
      Issue #3284665 by arunkumark, Rashmisoni, Jimaginary, Rakhi Soni,...
    catch’s picture

    Status: Reviewed & tested by the community » Fixed

    Committed/pushed to 10.1.x, cherry-picked to 10.0.x, 9.5.x, and 9.4.x thanks!

    Status: Fixed » Closed (fixed)

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