BungeeTabListPlus icon indicating copy to clipboard operation
BungeeTabListPlus copied to clipboard

[Bug/Feature] Handle Curly Brackets Properly

Open ColaIan opened this issue 3 years ago • 2 comments

Description: As title

How to reproduce: Use PlaceholderAPI placeholder in Math expansion

Example: Use ${viewer math_{mmoitems_stat_movement_speed}*1000}

[18:39:55 WARN]: [PlaceholderAPI] [Math] Invalid Placeholder. Provided text '{mmoitems' is no valid precision number.

Use ${viewer math_:_{mmoitems_stat_movement_speed}*1000}

[18:45:06 WARN]: [PlaceholderAPI] [Math] Invalid Placeholder. '{mmoitems_stat_movement_speed ' is not a valid math expression.

Expected Output: Image

ColaIan avatar May 01 '21 10:05 ColaIan

Having the same issue myself, just with a different placeholder, but also one that recursively uses another one.

BrainStone avatar May 07 '21 21:05 BrainStone

Just as a quick tip for anyone else struggling with this: A quick workaround would be to use the Shortcut expansion of PlaceholderAPI like this:

  • Download the expansion using /papi ecloud download Shortcut followed by /papi reload
  • There should now be a shortcuts folder in the PlaceholderAPI one. Open it.
  • Create a txt file. In my example we call it example.txt
  • Put the complete placeholder, including the % into the txt file (i.e. %math_{server_online}+1%)
  • Save the file.
  • Use shortcut_<filename> in your BTLP configuration. In our example would this be either ${viewer shortcut_example} or ${player shortcut_example}

While this sure is more effort does it allow the usage of {placeholder} patterns inside BTLP without breaking. I hope this isn't needed anymore once this has been fixed properly.

Andre601 avatar May 07 '21 21:05 Andre601