APickledWalrus / skript-placeholders

Placeholder integration for Skript.
MIT License
29 stars 9 forks source link

Does not work anymore? #7

Closed bloggy closed 4 years ago

bloggy commented 4 years ago

Test script:

command /testy:
    trigger:
        set {_ping} to placeholder "player_biome"
        broadcast "your ping is %{_ping}%"

result is <none>

APickledWalrus commented 4 years ago

Are you sure the player expansion is successfully loaded? Do any other placeholders work?

bloggy commented 4 years ago

I have also tested it with "server_tps", same result: none

I tried this as well:

on placeholder request with prefix "custom":
    if the identifier is "hey": 
        # %custom_hey%
        set the result to "hi there %player%!"

This throws an error: https://i.imgur.com/wJKc9Bm.png

Appreciate your help :-) really want to get this working

bloggy commented 4 years ago

Update: you were right: the expansions were not correctly loaded! Thanks a lot! But this issue still exists (or how can I create my own placeholder?):

on placeholder request with prefix "custom":
    if the identifier is "hey": 
        # %custom_hey%
        set the result to "hi there %player%!"

This throws an error: https://i.imgur.com/wJKc9Bm.png

Found this syntax to be working:

on papi request with prefix "skript":
    if the identifier is "coins":
        set the result to "100"

But when I create a script like this:

command /testy:
    trigger:
        set {_test} to placeholder "%%skript_coins%%"
        broadcast " %{_test}%"

the result is "N/A"

APickledWalrus commented 4 years ago

Do you have the skript expansion installed? Try using this but with a different name, such as bloggy_test.

bloggy commented 4 years ago

You are right - that fixed the issue - thanks a lot and keep up the good work :-) !!!

APickledWalrus commented 4 years ago

I’ll update the doc examples so that they are correct.