The docblock of the $value parameter states that it's of string type but in fact it is not.
https://git.drupalcode.org/project/drupal/-/blob/9.1.3/core/lib/Drupal/C...

CommentFileSizeAuthor
#5 3196050-5.patch831 bytesanmolgoyal74
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi created an issue. See original summary.

Shashwat Purav’s picture

Assigned: Unassigned » Shashwat Purav
Status: Active » Needs work
Shashwat Purav’s picture

Shashwat Purav’s picture

anmolgoyal74’s picture

Status: Needs work » Needs review
FileSize
831 bytes
Shashwat Purav’s picture

Assigned: Shashwat Purav » Unassigned

Thank you @anmolgoyal74 for the patch.

sulfikar_s’s picture

Status: Needs review » Reviewed & tested by the community

Hi, I've tested the patch on #5, and is working correctly. It changes the string to mixed.

Changing the status to RTBC!

alexpott’s picture

Version: 9.2.x-dev » 9.1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed 1749b10 and pushed to 9.2.x. Thanks!

Will backport to 9.1.x once the bugfix release is out.

  • alexpott committed 1749b10 on 9.2.x
    Issue #3196050 by anmolgoyal74, Chi: Fix documentation of...
alexpott’s picture

Status: Patch (to be ported) » Fixed

Backported to 9.1.x

  • alexpott committed 1f8fc0f on 9.1.x
    Issue #3196050 by anmolgoyal74, Chi: Fix documentation of...

Status: Fixed » Closed (fixed)

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