Uploaded image for project: 'Minecraft: Java Edition'
  1. Minecraft: Java Edition
  2. MC-174587

Greedy strings cannot contain trailing spaces in a function

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • None
    • 1.15.2, 20w11a, 20w12a, 20w14a, 20w16a, 20w17a, 20w18a, 20w20b, 1.16 Pre-release 5, 1.16.1, 1.16.2 Pre-release 1, 1.16.2, 1.16.3, 1.16.4 Pre-release 1, 1.16.4, 20w45a, 20w46a, 20w48a, 20w49a, 20w51a, 21w03a, 1.16.5, 21w10a, 21w11a, 21w13a, 21w15a, 21w16a, 21w19a, 1.17 Pre-release 1, 1.17, 1.17.1, 21w37a, 1.18.1, 22w03a, 1.18.2 Pre-release 1, 1.18.2, 22w11a, 22w12a, 1.19.1, 1.19.2, 1.19.3 Release Candidate 1, 1.19.3, 23w03a, 23w05a, 1.19.4, 23w12a, 23w18a, 1.20.1, 1.20.2, 23w42a, 24w04a
    • Confirmed
    • Commands

      The bug

      Because commands in a function are trimmed by java.lang.String#trim, greedy strings cannot contain trailing spaces (U+0000 .. U+0020) in a function.

      Affected commands

      • /advancement grant <targets> only <advancement> <criterion>
      • /advancement revoke <targets> only <advancement> <criterion>
      • /say <message>

      How to reproduce

      1. Install the datapack in Attachments.
      2. /function mc-174587:test

        revoked, the criterion "criterion with a trailing space " is not granted correctly.

      • mc-174587/advancement/test.json
        {
            "criteria": {
                "criterion with a trailing space ": {
                    "trigger": "minecraft:impossible"
                }
            }
        }
      • mc-174587/functions/test.mcfunction
        advancement grant @s only mc-174587:test criterion with a trailing space 
        execute if entity @s[advancements={mc-174587:test=true}] run say granted
        execute if entity @s[advancements={mc-174587:test=false}] run say revoked

            Unassigned Unassigned
            intsuc intsuc
            Votes:
            10 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              CHK: