#papi-updates

1 messages · Page 9 of 1

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/Statistics-Expansion] New comment on issue #1: Playtime issues

There are several issues/inconveniences I see with the formatTime method used in this expansion.

  1. Why does it use Duration?
    The only value provided is Seconds, so why are you turning seconds into Duration only to turn it back into seconds? Just pass the long directly.

  2. What if number is 0 or less?
    What would happen if the provided number is 0 or less (-1, -2, ...)
    When leaving the duration thing away would the expansion - according to my tests - return an empty String, because ...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
ChrisWard316
[PlaceholderAPI/CheckItem-Expansion] Issue opened: #9 Name Contains

Hello i was using check item for a while and i noticed you had an enchant option so i done /papi ecloud download checkitem to get the latest update:

%checkitem_mat:prismarine_shard,namecontains:♆ Rare Gem ♆,amt:3%

My item:
/minecraft:give chris_61 prismarine_shard{display:{Name:"[{"text":"♆ Rare Gem ♆","italic":false,"color":"dark_gray","bold":true}]"},Enchantments:[{id:"vanishing_curse",lvl:1}]}

the placeholder doesn't work and it used to, ive narrowed it down to the sp...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
ChrisWard316
[PlaceholderAPI/CheckItem-Expansion] New comment on issue #9: Name Contains

Even when i don't do colours or bold:

/minecraft:give chris_61 prismarine_shard{display:{Name:"[{"text":"♆ Rare Gem ♆","italic":false}]"},Enchantments:[{id:"vanishing_curse",lvl:1}]} 1

the below returns no:
/papi parse me %checkitem_mat:prismarine_shard,namecontains:♆ Rare Gem ♆%

Even if i do it this time without the special ♆ so it's plain:

/minecrraft:give chris_61 prismarine_shard{display:{Name:"[{"text":"Rare Gem","italic":false}]"},Enchantments:[{id:"vanishin...

#
ChrisWard316
[PlaceholderAPI/CheckItem-Expansion] New comment on issue #9: Name Contains

I have figured it out:

when you use the give command and the item NAME has capitals in it (R and G in this instance) the placeholder doesn't work no matter what if you put capitals in the placeholder name check or not:

The command has to be the following with no capitals in the name:
give chris_61 prismarine_shard{display:{Name:"[{"text":"♆ rare gem ♆","italic":false}]"},Enchantments:[{id:"vanishing_curse",lvl:1}]} 1

and the placeholder doesn't matter if it has caps or not it...

still narwhalBOT
#
ChrisWard316
[PlaceholderAPI/CheckItem-Expansion] New comment on issue #9: Name Contains

This is looking like it's leading onto my second github issue for example:

i get an enchanted book out my creative inventory and run this command:
/papi parse me %checkitem_nameequals:Enchanted Book%

It returns no but if i stick it in an anvil and rename it to enchanted book (no capitals) the above returns yes. I'm guessing its also the same as to why it won't pick up Enchantments on items as minecraft adds capitals to the start of the names.

still narwhalBOT
#
B-Knight49
[PlaceholderAPI/Pinger-Expansion] Issue opened: #6 Pinger incompatibility with ProtocolSupportStuff

Trying to ping a server with ProtocolSupportStuff by using this expansion throws the following:

[23:16:46 WARN]: [io.netty.channel.DefaultChannelPipeline] An exceptionCaught() event was fired, and it reached at the tail of the pipeline. It usually means the last handler in the pipeline did not handle the exception.
java.nio.channels.ClosedChannelException: null
at io.netty.channel.AbstractChannel$AbstractUnsafe.write(...)(Unknown Source) ~[patched_1.15.2.jar:git-Paper-298]

From wh...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Kqliber
[PlaceholderAPI/EzRanksPro-Expansion] Issue opened: #4 %ezrankspro\_rank\_cost\_% displaying incorrectly with multipliers/discounts

Hi,

the placeholder above does not take into account multipliers or discounts when they are applied to the player. The placeholder correctly returns the amount without a multiplier, however when a multiplier is active, it will not change the value.
In my rankups.yml, I have my next rank set to 400k, which is then multiplied up to 600k which is not shown when parsed.
https://gyazo.com/cb40c743155ae147d1cd1e72190a0544

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] Issue opened: #313 Placeholder request: player count per world

Issue

What is the issue? Describe it like you would tell a friend.

Neither Multiverse placeholders nor ListPlayers placeholders (nor anywhere else I can find) includes a count of players in a world. Use case: Lobby signs could show the number of players in selected worlds. Apologies if my search did not find an existing PAPI placeholder.

Expected behaviour

What should PlaceholderAPI do?

Return an integer for the number of players in the specified world.

still narwhalBOT
still narwhalBOT
#
rErEaT
[PlaceholderAPI/PlaceholderAPI] Issue opened: #314 \[Feature\] Whitespace

Hi!

I am using %server_date_HH:mm:ss% variable from placeholderapi.
This variable will say 11:15:32 but always with difference lenght what cames from font?!

Is it able to set to {server_date_HH:mm:ss+} for being the same length?
Or i have to report this to PlaceholderAPI? :(
Would be nice if you can implement a {whitespaces} variable what works everywhere

still narwhalBOT
still narwhalBOT
still narwhalBOT
#
aBooDyy
[PlaceholderAPI/PlaceholderAPI] New comment on issue #303: error code

Hello there,

It seems like your machine/host/server couldn't connect to the eCloud for automatic expansion download, which is why you're getting that error.
You can manually download expansions from here. You can also disable the plugin from trying to connect to the eCould from the config (by disabling cloud_enabled option) to avoid getting the error.

As Andre mentioned, use the appropriate template for your i...

#
rErEaT
[PlaceholderAPI/Server-Expansion] Issue opened: #13 \[Feature\] Whitespace

Hi!

I am using %server_date_HH:mm:ss% variable from placeholderapi.
This variable will say 11:15:32 but always with difference lenght what cames from font?!

Is it able to set to {server_date_HH:mm:ss+} for being the same length?
Or i have to report this to PlaceholderAPI? :(
Would be nice if you can implement a {whitespaces} variable what works everywhere

still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/Server-Expansion] New comment on issue #13: \[Feature\] Whitespace

I mentioned it twice and it's sad that I have to mention it a third time, but here we go.

You can't have the text on the same line, because not everyone has "Force Unicode Font" set to ON and therefore will have a different font than you have. And the expansion can't just see what font you use and therefore can't pad the output accordingly.
It could only have one specific amount of spaces and that's it.

still narwhalBOT
still narwhalBOT
still narwhalBOT
#
JaHollYT
[PlaceholderAPI/PlaceholderAPI] Issue opened: #317 CheckItems\-But

Hey, i want to use CheckItem ( https://github.com/PlaceholderAPI/PlaceholderAPI/wiki/Placeholders#checkitem ) in a Placeholder, but i dont understand how to use namecontains or lorecontains.
I got an Item called Darkphone with the same name in the lore. When i trie to check namecontains:Darkphone it returns false.
Everything else like the custommodeldata or mat works fine.

%checkitem_mat:stone,amt:1,lorecontains:Darkphone,namecontains:Darkphone% is not working.
%checkitem_mat:stone,amt:...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
unsivilaudio
[PlaceholderAPI/Player-Expansion] Issue opened: #30 \[FR\] %player\_biome% placeholder returns unformatted text\.

Can we get an supplementary placeholder that returns proper title text, e.g. "Warm Ocean" instead of "WARM_OCEAN". I thought it might be useful information to display to my players on their scoreboard. Yes I could just write a Js hook for this, but I feel it would increase server overhead, when the string could just as easily be formatted in papi. Perhaps, just %player_biome_formatted% would be nice to have. Thanks.

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
JaHollYT
[PlaceholderAPI/CheckItem-Expansion] Issue opened: #14 CheckItem\-Name\-Bug

Hey, i want to use CheckItem ( https://github.com/PlaceholderAPI/PlaceholderAPI/wiki/Placeholders#checkitem ) in a Placeholder, but i dont understand how to use namecontains or lorecontains.
I got an Item called Darkphone with the same name in the lore. When i trie to check namecontains:Darkphone it returns false.
Everything else like the custommodeldata or mat works fine.

%checkitem_mat:stone,amt:1,lorecontains:Darkphone,namecontains:Darkphone% is not working.
%checkitem_mat:stone,amt:...

still narwhalBOT
still narwhalBOT
still narwhalBOT
#
RoinujNosde
[PlaceholderAPI/PlaceholderAPI] Issue opened: #320 SimpleClans TopClans Placeholders

Type

What kind of change is it? (Multiple selections possible)

  • [ ] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [X] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Info

Please provide any information that is useful including links and images. ...

still narwhalBOT
#
BredyAK
[PlaceholderAPI/PlaceholderAPI] Issue opened: #321 Javascript expansion broken after PAPI reloaded

Issue

If I use the command /papi reload then my javascript expansion will broken, the {javascript_***} placeholders no longer works untill I restart server.

Expected behaviour

Keep Javascript expansion running smoothly after I reloaded PAPI.

Actual behaviour

All expansions are correctly reloaded, except Javascript.

How to reproduce

These expansions are installed on my server:

  • Javascript
  • LuckPerms
  • Math
  • Multiverse
  • Player
  • PlayerPoint...
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Niall7459
[PlaceholderAPI/PlaceholderAPI] Issue opened: #324 Animations expansion update 1\.0\.5

11/06/2020
Change request on description for animations expansion as per release 5

  • [x] Placeholders changed

Hyperlink to wiki: https://github.com/PlaceholderAPI/PlaceholderAPI/wiki/Placeholders#animations

Updated raw markdown description, please ensure the links and formatting is kept:
START
/papi ecloud download Animations

Placeholder format:

%animations_Text%

With implicit boolean flag set to true

%animations_Text%

With explicit options set
...

still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #324: Animations expansion update 1\.0\.5

You should follow the provided issue templates.

Also, does the info you provide here not really make much sense, especially when it should be updated on the wiki since it follows specific styling guides.
I suggest displaying this information:

%animations_<tag>Text</tag>%

For placeholders, use {} instead of %% (%animations_<rainbow>Welcome {player_name}</rainbow>%

Current list of valid types:

  • blink
  • fadein
  • fadeout
  • glow
  • left
  • pause
  • pulse
  • rainbo...
still narwhalBOT
#
Niall7459
[PlaceholderAPI/PlaceholderAPI] New comment on issue #324: Animations expansion update 1\.0\.5

The format on types has been updated

  • Left Align: <align left width=32>Text</align>
  • Right Align: <align right width=32>Text</align>
  • Appear: <appear>Text</appear>
  • Faded Appear: <appear fade>Text</appear>
  • Erase: <erase>Text</erase>
  • Faded Erase: <erase fade>Text</erase>
  • Fading In: <fade in>Text</fade>
  • Fading Out: <fade out>Text</fade>
  • Format/Round <format format=#.## mode=HALF_EVEN>Number</format>
  • Healthbar: `<healthbar symbol=♥ fill=&c empty=&7 ...
still narwhalBOT
still narwhalBOT
#
vlOd2
[PlaceholderAPI/PlaceholderAPI] Issue opened: #325 AnimatedScoreboard PlaceHolders issue for Player Names

#13 # Issue

What is the issue? Describe it like you would tell a friend.
When players with "_" in their names the placeholders on AnimatedScoreboard dont show for them and cause an issue! And yes i checked the console and its an PlaceHolderApi

Expected behaviour

What should PlaceholderAPI do?
Dont cause an issue while displaying the placeholders

Actual behaviour

What does PlaceholderAPI actually do?
Doesnt display the placeholders,cause spam in console and doesnt...

#
vlOd2
[PlaceholderAPI/PlaceholderAPI] Issue opened: #326 Animated Scoreb

#13 # Issue

What is the issue? Describe it like you would tell a friend.

When players with "_" in their names the placeholders on AnimatedScoreboard dont show for them and cause an issue! And yes i checked the console and its an PlaceHolderApi

Expected behaviour

What should PlaceholderAPI do?

Dont cause an issue while displaying the placeholders or the scoreboard

Actual behaviour

What does PlaceholderAPI actually do?

Doesnt display the placeholders,cause sp...

still narwhalBOT
still narwhalBOT
still narwhalBOT
#
JaHollYT
[PlaceholderAPI/CheckItem-Expansion] Issue opened: #15 CheckItems\-Bug

Hey, i want to use CheckItem ( https://github.com/PlaceholderAPI/PlaceholderAPI/wiki/Placeholders#checkitem ) in a Placeholder, but i dont understand how to use namecontains or lorecontains.
I got an Item called Darkphone with the same name in the lore. When i trie to check namecontains:Darkphone it returns false.
Everything else like the custommodeldata or mat works fine.

%checkitem_mat:stone,amt:1,lorecontains:Darkphone,namecontains:Darkphone% is not working.
%checkitem_mat:stone,amt:...

#
BredyAK
[PlaceholderAPI/Javascript-Expansion] Issue opened: #21 Javascript expansion broken after PAPI reloaded

Issue

If I use the command /papi reload then my javascript expansion will broken, the {javascript_***} placeholders no longer works untill I restart server.

Expected behaviour

Keep Javascript expansion running smoothly after I reloaded PAPI.

Actual behaviour

All expansions are correctly reloaded, except Javascript.

How to reproduce

These expansions are installed on my server:

  • Javascript
  • LuckPerms
  • Math
  • Multiverse
  • Player
  • PlayerPoint...
still narwhalBOT
#
aBooDyy
[PlaceholderAPI/PlaceholderAPI] New comment on issue #327: ChatColor doesn't work properly

Hello there,

PlaceholderAPI job is pretty much replacing the placeholder with its value.
You haven't mentioned anything about where you're using the "chat color". This might be from your chat plugin or the plugin you're using.

PlaceholderAPI has changed the way of providing placeholders by other plugins, so this is probably why v1.8.7 is working with your chat color placeholders but not other placeholders. In this case, your chat color placeholders are most likely outdated and you migh...

still narwhalBOT
#
BredyAK
[PlaceholderAPI/Javascript-Expansion] New comment on issue #21: Javascript expansion broken after PAPI reloaded 
> jsexpansion reload
[22:59:36 INFO]: JavascriptExpansion reloading...
[22:59:36 WARN]: [PlaceholderAPI] Failed to set ScriptEngine for javascript placeholder: chatprefix
[22:59:36 WARN]: [PlaceholderAPI] Failed to set ScriptEngine for javascript placeholder: chatregion
[22:59:36 WARN]: [PlaceholderAPI] Failed to set ScriptEngine for javascript placeholder: chatsuffix
[22:59:36 WARN]: [PlaceholderAPI] Failed to set ScriptEngine for javascript placeholder: suffixquality
[22:59:36 WA...
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
itaquito
[PlaceholderAPI/Javascript-Expansion] Issue opened: #22 Javascript placeholders stop working when ChatControl is active\.

Hello!

I made some Javascript placeholder in order to make some very specific tasks on my server, all of them worked fine, no errors on the console when I use them or nothing that meant that they aren't working properly.
The issue started happening just when I install ChatControl, now the placeholders aren't loading and this error appears on the console every time the server starts and also when I reload PAPI.
https://pastebin.com/ebxDaiup

If I uninstall ChatControl, the placeholders ...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
darbyjack
[PlaceholderAPI/PlaceholderAPI] New branch created: 1\.16
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
TrizzerArts
[PlaceholderAPI/PlaceholderAPI] Issue opened: #334 Essentials placeholder kit doesnt work

Issue

What is the issue? Describe it like you would tell a friend.
The bug I have is when I use the placeholder %essentials_kit_time_untilavailable_%. The bug begins with the delay that each kit of the essentials has, I am trying to put this placeholder to my kit that has a delay of 7889400 but at the time of visualizing the placeholder, the time appears in 0 and not in the time that corresponds to the kit. And if I add a delay of 1209600 to my kit, you can see the time. The placehold...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
mybe
[PlaceholderAPI/PlaceholderAPI] Issue opened: #339 Plugin doesn't work on 1\.16

Issue

PlaceholderAPI does not work correctly on 1.16.

Expected behaviour

/papi bcparse me %server_tps%
This should return our server's TPS.

We also use ChatControl, and our join message should appear like:

  • username

Actual behaviour

It actually returns %server_tps%.

Our ChatControl behaves instead like:

  • %player

How to reproduce

We are running a Paper server, updated to the latest version of Paper and PlaceholderAPI.
/papi bcparse me %server_tp...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
cj89898
[PlaceholderAPI/Expansion-ScoreboardObjectives] New branch created: master
still narwhalBOT
#
cj89898
[PlaceholderAPI/Expansion-ScoreboardObjectives] New tag created: 3\.3\.1
still narwhalBOT
still narwhalBOT
#
ChimneySwift
[PlaceholderAPI/Server-Expansion] New comment on issue #7: Feature Request: %mspt%

For those on PaperMC, I've created a fork with the following additional placeholders:

%server_mspt%
%server_mspt_10s%
%server_mspt_1m%
%server_mspt_colored%
%server_mspt_10s_colored%
%server_mspt_1m_colored%

Which give colored and uncolored variants (Same colors as TPS, 0-25ms is green, 25-50 is orange and 50+ is red) of the MSPT averaged over 1 second, 10 seconds and 1 minute depending on which placeholder you use.

T...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sparoney
[PlaceholderAPI/PlaceholderAPI] New comment on issue #339: Plugin doesn't work on 1\.16

Join the HelpChat discord and provide a startup log, as well as some detail on "placeholders don't work"

Alright I'll do that, but by "placeholders don't work" I meant the same as OP, it just says "%player_name%" instead of the player's actual name

Assuming that you have not downloaded the player expansion and reloaded papi

Oh my god I'm so dumb. I apologise, that is my fault.

still narwhalBOT
#
CryptoMorin
[PlaceholderAPI/PlaceholderAPI] Pull request opened: #340 Performance Improvements

Type

  • [x] Internal change (Doesn't affect end-user).
  • [ ] External change (Does affect end-user).
  • [ ] Other: __________

Description

Provide additional information if needed.
Performance improvements for most of the classes.
The major improvement is the PlaceholderReplacer class which isn't replaced with the main RegEx replacer. I'm not sure if you want to use this, because it's a little hard to understand. If you want I can remove PlaceholderReplacer but it's much bett...

still narwhalBOT
still narwhalBOT
#
TrizzerArts
[PlaceholderAPI/Essentials-Expansion] Issue opened: #22 Essentials placeholder kit doesnt work

Issue

What is the issue? Describe it like you would tell a friend.

The bug I have is when I use the placeholder %essentials_kit_time_untilavailable_%. The bug begins with the delay that each kit of the essentials has, I am trying to put this placeholder to my kit that has a delay of 7889400 but at the time of visualizing the placeholder, the time appears in 0 and not in the time that corresponds to the kit. And if I add a delay of 1209600 to my kit, you can see the time. The placeho...

still narwhalBOT
still narwhalBOT
#
thecenters
[PlaceholderAPI/PlaceholderAPI] Issue opened: #341 Errors in 1\.16

Using 2.10.6 on 1.16.1 I'm getting these errors on the server. I reached out to the developer of TNE and they informed me it was an issue outside of their plugin.

01.07 12:51:59 [Server] WARN [PlaceholderAPI] Plugin PlaceholderAPI v2.10.6 generated an exception while executing task 1009
01.07 12:51:59 [Server] INFO java.lang.NullPointerException: null
01.07 12:51:59 [Server] INFO at net.tnemc.core.common.api.TNEAPI.getHoldings(TNEAPI.java:289) ~[?:?]
01.07 12:51:59 [Server] INFO at ne...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
DarkSoul034
[PlaceholderAPI/PlaceholderAPI] Issue opened: #342 Satistic placeholders increasing by double the true value

Issue

What is the issue? Describe it like you would tell a friend.
I'm using the Statistic placeholders to track player stats in my spigot 1.16.1 server. However, when some statistics such as "blocks broken" and "items crafted" are increased, the value increases by double that of what happened. E.g I break 1 block, the scoreboard displaying the placeholder increases by 2 instead. I craft 4 wooden planks, the scoreboard increases by 8

Expected behaviour

What should Placehol...

still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #343: PAPI Displaying Hex Color Codes Incorrectly

PlaceholderAPI does nothing more than make a translateAlternateColorCodes('&', text) on the placeholders which only translates the provided character (&) with the next following one (a-f, 0-9) or something like that.
This isn't really a PlaceholderAPI issue as the plugin does nothing special in terms of translating color codes.
Additionally is the plugin not fully 1.16 as a PR (#333) still needs to be merged.

So tl;dr is, that this is nothing that really needs fixing as it is nothing t...

still narwhalBOT
still narwhalBOT
#
ME1312
[PlaceholderAPI/PlaceholderAPI] Issue opened: #344 SubServers\.Client Implementation

Type

  • [x] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [ ] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Information

SubServers.Client now provides PlaceholderAPI placeholders as of the latest snapshot. It does this using caching so, to keep the use of...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #346: Add cmdparse command

And where is the benefit of the existing commands /papi parse and /papi bcparse? I don't see an actual reason not to add this. Furthermore my additions are much more abstract and general than the existing parse and broadcast commands. For example you can even replace /papi bcparse me ... by /papi cmdparse me tellraw @a ... and do much more things I can't even think of.

The parse and bcparse commands are used to find out, what the placeholders would return based on the provided pl...

still narwhalBOT
#
stonar96
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #346: Add cmdparse command

I accept your opinions. However I still think this is very general and useful thing.

I really can't see a reason why a command should execute another command.

This is actually a very common thing and there are multiple examples. Even in Minecraft there is the /execute ... run ... command. Other examples are sudo ... or screen -dmS name ....

I didn't see anyone use the bcparse command either, so my guess is, that this command isn't really used. The parse command is useful, a...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
xtomyserrax
[PlaceholderAPI/PlaceholderAPI] Issue opened: #347 Add StaffFacilities to placeholders wiki

Type

What kind of change is it? (Multiple selections possible)

  • [x] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [ ] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Info

My plugin StaffFacilities has PAPI support and also has its own placeholders...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on commit `3e396a9`

Wouldn't it be much simpler/better when you would pass the PlaceholderAPIPlugin instance through the execute method instead of just always having final PlaceholderAPIPlugin plugin = PlaceholderAPIPlugin.getInstance(); since this is used most of the time?
Would help reducing the usage of the same code over and over again. #DRYPrinciple

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] New branch created: gradle
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] New comment on commit `3e396a9`

Wouldn't it be much simpler/better when you would pass the PlaceholderAPIPlugin instance through the execute method instead of just always having final PlaceholderAPIPlugin plugin = PlaceholderAPIPlugin.getInstance(); since this is used most of the time?
Would help reducing the usage of the same code over and over again. #DRYPrinciple

May help with DRY, but it'd be a direct violation of the interface segregation principle. Unfortunately for this specific case, ISP takes precedence o...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
extendedclip
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #340: Performance Improvements

Yes, before the recent changes we made, we did test your method and I think we should move forward with using your method over regex.

We did have a good talk on discord about your method and I do remember someone saying there was an issue with the "closure" detection but either way lets move forward with it for internal parsing of placeholders. just have to keep the regex patterns and other methods ex: containsPlaceholders etc.

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/eCloud] New branch created: master
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #103: Expansions load anomaly

This got my attention. I'm not sure if I understood the main problem but...

Only if Spigot had an event that is fired after the startup process is complete

Are you guys looking for a way to register expansions after the server is completely loaded? Doesn't the current runTaskLater method work? This runs after the server is completely loaded.

We are aware about the ServerLoadEvent if you hint towards that, but the thing is, that this was only added in around 1.13 (I think)...

still narwhalBOT
still narwhalBOT
#
extendedclip
[PlaceholderAPI/PlaceholderAPI] New comment on issue #236: Missing dependency declarations

This is something that can be ignored. md_5 is not willing to do anything Spigot side to take into account plugins like PlaceholderAPI which might have unknown dependencies. There is no way I am going to push PAPI updates to add a new dependency every time a new dependency is created.

If the messages bother anyone feel free to open the plugin jar with winzip and manually add the dependencies yourself based on what dependencies you use.

#
DarkSoul034
[PlaceholderAPI/Statistics-Expansion] Issue opened: #18 Satistic placeholders increasing by double the true value

Issue

What is the issue? Describe it like you would tell a friend.
I'm using the Statistic placeholders to track player stats in my spigot 1.16.1 server. However, when some statistics such as "blocks broken" and "items crafted" are increased, the value increases by double that of what happened. E.g I break 1 block, the scoreboard displaying the placeholder increases by 2 instead. I craft 4 wooden planks, the scoreboard increases by 8

Expected behaviour

What should Placehol...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #236: Missing dependency declarations

It would be much more useful, if Spigot offered a way for dynamic dependency management.
Like where you could have something like this:

// Imagine this being in a onLoad method
PluginManager manager = getServer().getPluginManager();
if(manager.isPluginLoaded("Essentials")){ // Checks if the plugin is loaded by the server (or will be loaded?)
    manager.addSoftDepend(this, "Essentials); // Would add the plugin as softdepend to delay the onEnable
    // Do other stuff
}

B...

#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI3] Issue opened: #23 Import resources aswell as classes when loading an expansion jar

Type

What kind of request is this? (Multiple selections possible)

  • [x] Change to code (Internal).
    Changes to code that won't affect the end-user.

Info

What is the change?
Please provide as much information (including links, images and code-snippeds) as possible.

PlaceholderAPI currently loads expansions by looping through each jar with a JarInputStream, then manually adding any file that ends with .class to the class loader. I propose that this is replaced with...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #316: Suggestion: Move Wiki to GitHub pages

I personally believe, that at least having the files of the wiki in a separate folder in this repository (wiki folder) and then using a GitHub Action to automatically push changes to the main wiki would be a huge benefit, as people could just make PRs to add/edit pages, which then could be merged/closed.
This would give the freedom of allowing others to edit the wiki, without the risk of having trolls messing it up, since the wiki's perm system is basically all or nothing.

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #200: Suggestion: Add the all\-contributors bot to this repo

I don't have access to this repository, so I can't set it up for you (sadly)

A general installation guide on how to add the bot to the repository is found here: https://allcontributors.org/docs/en/bot/installation

In general would the bot work, where you just type "@all-contributors add X for Y" (Y is one or multiple types which you find here) and the bot would make a PR to add the user to wherever you defined contributors to be displayed...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/eCloud] Issue opened: #1 eCloud Updates
  • [x] The hosted eCloud will need to be updated to support different categories, namely the ones defined in the Expansion API issue. This should be the only change needed on the eCloud itself.
  • [ ] Expansions uploaded to the ecloud should have their filenames sanitized to this format: expansionname-version.jar. e.g. player-1.0.0.jar
still narwhalBOT
still narwhalBOT
#
knightzmc
[PlaceholderAPI/eCloud] New branch created: api\-help
still narwhalBOT
still narwhalBOT
#
knightzmc
[PlaceholderAPI/eCloud] branch deleted: api\-help
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #316: Suggestion: Move Wiki to GitHub pages

I actually looked into (and try to use) Orchid and while it sure is interesting, does it have some major flaws.

  • Any {@link} tag will have the text without any space before and after it.
  • {@link} to external resources aren't working yet.
  • Wiki pages are somewhat a mess and would require an entire rewriting of ALL paths, relative or not.
  • Overall structure is weird and configuration isn't that easy. It took me 2 hours and the help of the maintainer/dev of Orchid to get a somewhat...
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New review comment on pull request #352: POM fixes\.

I don't know if you noticed, but the last 1600+ merged PR contained code that referenced classes provided by internal spigot dependencies.

This is why they changed the dependency from spigot-api to spigot, rendering the entire project unworkable unless you had the server jar built in your local repo.

I reverted this, and added a provided dependency for what those classes referenced.

still narwhalBOT
still narwhalBOT
#
axelrindle
[PlaceholderAPI/PlaceholderAPI] Issue opened: #353 Add new plugin to "Plugins using PlaceholderAPI"

Type

  • [x] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [ ] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Info

I recently added support for using existing placeholders to one of my own plugins: [Broadcaster-Plugin](https://github.com/axelrindle/Broad...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #354: replacer api, unit tests, and benchmarks

Benchmark results.

Benchmark                                          Mode  Cnt        Score        Error  Units
ReplacerBenchmarks.measureRegexReplacerLargeText  thrpt   25   517010.404 ±  37117.162  ops/s
ReplacerBenchmarks.measureTestsReplacerLargeText  thrpt   25   576912.638 ±   6980.202  ops/s
ReplacerBenchmarks.measureCharsReplacerLargeText  thrpt   25   938678.884 ±  16682.921  ops/s
ReplacerBenchmarks.measureRegexReplacerSmallText  thrpt   25  1937592.602 ±  86878.971  op...
still narwhalBOT
mint valveBOT
#
PlaceholderAPI #99

Build: 99
Status: failure

Artifacts:

No artifacts saved.

Jenkins v2.222.3, Discord Notifier v1.4.11
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: commands\_rewrite
still narwhalBOT
still narwhalBOT
#
Niall7459
[PlaceholderAPI/eCloud] Issue opened: #5 Platform independent expansions/versions

Requirements:

  • Expansion versions may support one or more, but not all platforms.
  • Separate downloadable expansion files for each platform.

Example

  • An expansion supports platform A on versions 1.0 through to 2.0
  • As of version 2.1 through to 3.0 It supports platform A and B.
  • As of version 3.1 onward it supports B and C, support for A is now dropped.

The user interface would have to provide for:

A, downloading the correct version per platform.
This would m...

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/KotlinScript-Expansion] New branch created: master
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
ThePlay3r
[PlaceholderAPI/PlaceholderAPI] Issue opened: #356 New plugin supporting PlaceholderAPI

Type

What kind of change is it? (Multiple selections possible)

  • [x] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [ ] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Info

Please provide any information that is useful including links and images. ...

still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/Player-Expansion] New branch created: pom\_updates
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/Player-Expansion] New branch created: fixes
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] branch deleted: gradle
mint valveBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] branch deleted: commands\_rewrite
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] branch deleted: development\-2\.10\.6
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] branch deleted: 1\.16
mint valveBOT
still narwhalBOT
mint valveBOT
#
PlaceholderAPI #6

Build: 6
Status: failure

Changes:

No changes.

Artifacts:

No artifacts saved.

Jenkins v2.222.3, Discord Notifier v1.4.11
#
PlaceholderAPI #7

Build: 7
Status: failure

Changes:

No changes.

Artifacts:

No artifacts saved.

Jenkins v2.222.3, Discord Notifier v1.4.11
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
#
PlaceholderAPI #15

Build: 15
Status: success

Changes:

  • 68d17b fixed sxtannas shit - PiggyPiglet
  • 79ae18 Revert "fixed sxtannas shit" - ranald.taylor
  • ac6e56 Revert "Revert "fixed sxtannas shit"" - PiggyPiglet
  • 9327ee Revert "Revert "Revert "fixed sxtannas shit""" - ranald.taylor
  • b50b1b Revert "Revert "Revert "Revert "fixed sxtannas shit"""" - PiggyPiglet
  • 883a61 changed ECloud to eCloud - PiggyPiglet

Artifacts:

Jenkins v2.222.3, Discord Notifier v1.4.11
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: casing
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] branch deleted: casing
mint valveBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: expansions
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: codestyle
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on issue #363: Sort expansions alphabetically

Was talking specifically about the list command not the download command.

I comprehended that, but in your scenario they are "searching" for something by name, in which case simply using the tab completes makes infinite more sense.

outlining to the user how it's currently being sorted

This is agreeable.

I believe it would be a benefit of being able to change how the expansions are sorted

This kinda isn't. More opportunity for user error is annoying.

#
Funny-cube
[PlaceholderAPI/PlaceholderAPI] New comment on issue #363: Sort expansions alphabetically

I comprehended that, but in your scenario they are "searching" for something by name, in which case simply using the tab completes makes infinite more sense.

Personally I feel it's more plausible for a user to search via the list rather than tab completing on the download command. If you wanted to find something, you'd list them, you wouldn't naturally go to the download command if you didn't know what you were downloading.

This is of course unless by "search", you actually meant "b...

still narwhalBOT
mint valveBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] branch deleted: expansions
still narwhalBOT
#
darbyjack
[PlaceholderAPI/PlaceholderAPI] New branch created: fix\-gradle\-resolve
#
darbyjack
[PlaceholderAPI/PlaceholderAPI] branch deleted: fix\-gradle\-resolve
mint valveBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #367: Return previous papi ecloud list output

Spigot 1.16.1
PlaceholderAPI 2.10.7-DEV-20

Now I get a single line contained all expansions.

>papi ecloud list installed
[09:47:25] [Server thread/INFO]: Installed Expansions Page: 1
1. Vault [Author: clip, Verified: ✔, Latest Version: 1.6.0]2. Quests [Author: pikamug, Verified: ✔, Latest Version: 1.1]3. Essentials [Author: clip, Verified: ✔, Latest Version: 1.2.4]4. Player [Author: clip, Verified: ✔, Latest Version: 1.7.2]5. WorldGuard [Author: clip, Verified: ✔, Latest Version: ...
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: managers
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] Pull request opened: #368 Managers Update

This PR completely rewrites how the individual managers (local and cloud) function.

  • Resolving expansion classes from files is now asynchronous.
  • PlaceholderAPI class no longer handles registration and unregistration.
  • PlaceholderAPI class no longer stores expansions. (moved to local manager)
  • CloudExpansionManager no longer indexes by treemap.
  • etc.. etc..
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] branch deleted: managers
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
#
LoSCraft-Network
[PlaceholderAPI/PlaceholderAPI] Issue opened: #372 Error on startup \+ Javascript not working

Issue

What is the issue? Describe it like you would tell a friend.

there is an error on startup and it will prevent javascript placeholders from working

Expected behaviour

What should PlaceholderAPI do?

no error

Actual behaviour

What does PlaceholderAPI actually do?

error
https://hasteb.in/afikiyan.shell

How to reproduce

What steps did you made, to get this bug?

  1. Started when I updated to PlaceholderAPI-2.10.7-DEV-25.jar
#
LoSCraft-Network
[PlaceholderAPI/PlaceholderAPI] New comment on issue #372: Error on startup \+ Javascript not working

Admin360-Reloaded*, AdvancedAchievements, AdvancedPortals, AnimatedScoreboard, AntiCooldown, AuctionHouse, BetterSpawners, BuycraftX, ChatControl, ChatReaction, CheaperPhysicsEvent, ChestShop, Citizens, CitizensCMD, CombatLogX, CoreProtect, CrazyCrates, DeluxeMenus, DeluxeTags*, Duels, Elevator, EntityTrackerFixer, Essentials, EssentialsSpawn, FlightControl, GlowAPI*, GPAccruals*, GriefPrevention, GSit, HeadDatabase*, HolographicDisplays, HolographicExtension, IllegalStack, LeaderHeadsRevampe...

#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on issue #372: Error on startup \+ Javascript not working

Yes, the issue of other plugins breaking will be a thing until they are updated.

PAPI is currently going through a massive rewrite of several facets of the plugin, many of which are being completely removed in favour of a simpler external API.

The best thing you can do at this point is to notify authors of this change, link them to this issue.

I'm going to keep this issue open for the sake of information.

still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New branch created: feature/add\-wiki\-action
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New branch created: update\_command
mint valveBOT
still narwhalBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] branch deleted: update\_command
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #376: NoSuchMethodError on server start\-up

A little more research and isolation. Similar errors are produced by any of the following:

  • DiseasePlaceholders-0.0.1.jar
  • Expansion-Skript-1.0.0.jar
  • Expansion-Thirst-1.1.0.jar
  • PAPI-CoordinateTool-1.1.0.jar

Without them, my other 10 expansions load. These are installed

Expansion-LuckPerms-5.0-R2.jar
Expansion-Multiverse-1.0.1.jar
Expansion-Quests-1.1.jar
Expansion-Statistics-1.7.0.jar
Expansion-ViaVersion.jar
PAPI-Expansion-Essentials-1.2.4.jar
PAPI-Expansion-Player-1...
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI] New comment on issue #376: NoSuchMethodError on server start\-up

I will open issues for them.

What do they need to do? Is it a deprecated method? Do they need to update to later PAPI? Please be specific as possible, and I will open tickets for the identified ones.

Any expansion that handles it's own registration, may not work. i.e. expansions that register themselves, via PlaceholderAPI#registerPlaceholderHook, will cease to function. In general, any expansion or plugin that uses the following methods:

  • PlaceholderAPI#register*
  • Placehold...
mint valveBOT
#
PlaceholderAPI #31

Build: 31
Status: success

Changes:

  • 6eb1ec Implemented general purpose table utility - ranald.taylor
  • 933919 updated ecloud list command to use new table utility, added message for when no expansions can be listed - ranald.taylor
  • 30ead2 fixed spacing - ranald.taylor
  • 07cd34 updated local expansion to not catch potential linkage errors, and report them. closes #377 - ranald.taylor

Artifacts:

Jenkins v2.222.3, Discord Notifier v1.4.11
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

Spigot 1.16.1
PlaceholderAPI 2.10.7-DEV-31

papi ecloud list installed still returns nothing.

[22:23:52] [Server thread/INFO]: [PlaceholderAPI] Loading PlaceholderAPI v2.10.7-DEV-31
[22:24:00] [Server thread/INFO]: [PlaceholderAPI] Successfully registered expansion: aac
[22:24:00] [Server thread/INFO]: [AAC] Hooked into PlaceholderAPI
[22:24:17] [Server thread/INFO]: [PlaceholderAPI] Enabling PlaceholderAPI v2.10.7-DEV-31
[22:24:17] [Server thread/INFO]: [PlaceholderAPI] Fetc...
still narwhalBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

DiseasePlaceholders-0.0.2.jar
Expansion-LuckPerms-5.0-R2.jar
Expansion-Multiverse-1.0.1.jar
Expansion-Quests-1.1.jar
Expansion-Skript-1.0.0.jar
Expansion-Statistics-1.7.0.jar
Expansion-Thirst-1.1.0.jar
Expansion-ViaVersion.jar
PAPI-CoordinateTool-1.1.0.jar
PAPI-Expansion-Essentials-1.2.4.jar
PAPI-Expansion-Player-1.7.2.jar
PAPI-Expansion-Server-2.2.3.jar
PAPI-Expansion-Statistic-1.9.2.jar
PAPI-Expansion-Vault-1.6.0.jar
VanishPlaceholders.jar
WorldGuard-Expansion-1.4.2.jar

still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

The ones with placeholders -- including those with "internal" placeholders -- are listed above. I prefer not to expose the entire list publicly. If you really need it, I'll PM.

>papi list
[22:32:14] [Server thread/INFO]: 28 placeholder hook(s) active: 
  DeluxeMenus, aac, bentobox, biomeremap, bskyblock, challenges, combatlogx, denizen, disease, essentials
  frp, insights, level, luckperms, mobhunting, multiverse, plan, player, quests, residence
  server, spark, stat, statistic, van...
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
#
PiggyPiglet
[PlaceholderAPI/PlaceholderAPI3] branch deleted: wip/commands
still narwhalBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

I do isolation testing slightly differently in these cases: Start at ground zero, and add material until it breaks.

Installed plugins

Multiverse-Core-4.1.1-b785.jar
Multiverse-Inventories-4.0.0-b469.jar
Multiverse-NetherPortals-4.1.1-b764.jar
Multiverse-Portals-4.1.1-b808.jar
PlaceholderAPI-2.10.7-b31.jar
Vault-1.7.3-b21.jar

Installed expansions

Expansion-Multiverse-1.0.1.jar
PAPI-Expansion-Player-1.7.2.jar
PAPI-Expansion-Server-2.2.3.jar
PAPI-Expansion-Vault-1.6...
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #364: Google Codestyle

You should first fix your "IDE" making the code look shit before telling me it has "very little importance for me".

image

The google codestyle isn't any less readable for a human than whatever you want to use.
And I did do my fair share of contributions to this project and using PlaceholderAPI in several plugins and expansions, so I believe it does have importance to me to a certa...

#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #364: Google Codestyle

You should first fix your "IDE" making the code look shit

  1. Why is IDE in quotes?
  2. It is not IntelliJ that does that, it's github/chrome.
    Either GitHub or Chrome is rendering tabs as 8 spaces instead of the more normal 4.

You can see this by appending the tab modifier parameter to the URL of the file:
https://github.com/PlaceholderAPI/PlaceholderAPI/blob/master/src/main/java/me/clip/placeholderapi/commands/PlaceholderCommandRouter.java?ts=4

Tabs are the better spacer as they ...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #364: Google Codestyle

I'm sorry that I didn't had 200 commits to this project, but I did do commits and helped more than once here (The issue templates where mostly made by me f.e.).

All I say is, that you don't need to have many contributions here to have a voice and give your own feedback here. The commit count isn't relevant here.
People will mostly care for Javadocs anyway, so why bother that the indend/position of a { is like 2 spaces of or something?

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New branch created: change/reposition\-urls\-in\-templates
#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on pull request #364: Google Codestyle

All I say is, that you don't need to have many contributions here to have a voice and give your own feedback here. The commit count isn't relevant here.

This is legit not at all what I said, I never implied you have no contributions. I said, and I quote, "You have almost no actual programming contribution to this repo, readability has very little importance to you."

Which is objectively true, 80% of your contributions have been to documentation, the wiki, and github issue/contribu...

#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] branch deleted: codestyle
#
LemmoTresto
[PlaceholderAPI/eCloud-Client] New branch created: master
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

The conflict is in this working set. I'll have it isolated soon, but if this list rings any warning bells, that would save time.

AAC-4.4.0
CommandSigns-2.2.0-beta
ConsoleSpamFix-1.8.3
DelayJoin-1.0
DeluxeMenus-1.13.3-b90
DisableItemFrameRotation-1.2.0
ExtraContexts-1.0-b14
FakeBlock-2.0.0-b8
ForceResourcepacks-1.9.3
ItemJoin-5.0.7-b480
PluginVersions-1.0.4
RegisterCommands-1.3.0
ServerListPlus-3.5.2-bukkit
Shopkeepers-2.10.0
VoidSpawn-1.18.3
WhatPlugin-1.0
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #377: papi ecloud list installed returns nothing

The problem is this line

Messages-To-Hide-Filter:
- '--------------------------------------'

I have removed that line, and all is well. Feature request: papi ecloud list installed could replace the continuous line, with something like (for example)

>papi ecloud list installed
[11:02:29] [Server thread/INFO]: Installed Expansions
     Name        Author   Verified  Version  
---- ----------- -------- --------- ---------
1.   Disease     artitus  Y         0.0.2    
2....
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
Sxtanna
[PlaceholderAPI/Gradient-Expansion] New branch created: master
mint valveBOT
still narwhalBOT
#
ThePlay3r
[PlaceholderAPI/PlaceholderAPI] Issue opened: #380 Plugin Placeholders Update

Type

What kind of change is it? (Multiple selections possible)

  • [ ] New plugin/expansion supporting PlaceholderAPI.
    A new plugin/expansion uses and/or provides placeholders.
  • [x] Placeholder(s) changed.
    An already listed expansion/plugin has new placeholders or old ones have changed.
  • [ ] Plugin/Expansion no longer supports PlaceholderAPI and/or was deleted.
  • [ ] Other: __________

Info

Please provide any information that is useful including links and images.
...

still narwhalBOT
still narwhalBOT
mint valveBOT
#
PlaceholderAPI #37

Build: 37
Status: success

Changes:

  • b63f10 lets pretend no one saw this. - ranald.taylor
  • a160f3 updated to further phase out PlaceholderHook, added contracts to expansion methods - ranald.taylor
  • ee33de removed final from deprecated methods, restored fallback for getRequiredPlugin - ranald.taylor
  • 836051 rewrote discovery and registration code to be composable and higher level - ranald.taylor

Artifacts:

Jenkins v2.222.3, Discord Notifier v1.4.11
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT
#
cj89898
[PlaceholderAPI/Thirst-Expansion] New branch created: master
mint valveBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
still narwhalBOT
#
extendedclip
[PlaceholderAPI/PlaceholderAPI] New branch created: daddy
#
Funny-cube
[PlaceholderAPI/PlaceholderAPI] branch deleted: daddy
#
darbyjack
[PlaceholderAPI/PlaceholderAPI] New branch created: daddy
#
Funny-cube
[PlaceholderAPI/PlaceholderAPI] branch deleted: daddy
#
extendedclip
[PlaceholderAPI/PlaceholderAPI] New branch created: daddy
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
extendedclip
[PlaceholderAPI/PlaceholderAPI] Issue opened: #386 Only build on jenkins when there are code changes

Please read

This template is only for requesting new functions to be added to PlaceholderAPI or for old one to be changed!
Please use the [Request change (Wiki)] template to ask for additions/changes to the wiki.
For reporting bugs of PlaceholderAPI use the [Bug report] template.

Also make sure that the [wiki] doesn't already mention such a function/change and that there isn't a issue about this change on the [issues] page.

We recommend to [open a Pull Request][Pull Requests] ...

#
extendedclip
[PlaceholderAPI/PlaceholderAPI] Issue opened: #387 \[PlaceholderAPI/PlaceholderAPI\] Run failed: Update Wiki \- master

Please read

This template is only for reporting bugs of PlaceholderAPI!
If you want to request changes to the code of PlaceholderAPI, use the [Request change (PlaceholderAPI)] template.
For changes to the wiki use the [Request change (Wiki)] template.

Please also make sure that you use the [latest release][Spigot] or the latest [developement build][Jenkins] and that your bug isn't already reported on the [issues] page.

**Please report issues with expansions/placeholders to the...

still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #387: \[PlaceholderAPI/PlaceholderAPI\] Run failed: Update Wiki \- master

could not read Password for 'https://***@github.com': No such device or address

This confirms my belief that this action requires a PAT (Private Access Token) in order to function.
Someone that has both push access and access to the repo's settings has to create a PAT with repo scope (https://github.com/settings/tokens/new?scopes=repo) and add a new secret (Settings -> Secrets -> Add new) to the repo that contains the generated PAT as value.

You can then give me the name of the sec...

mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #386: Only build on jenkins when there are code changes

I myself only have a setup where I don't build anything if the commit message contains [skip ci] but that doesn't work that well for obvious reasons.
If Jenkins offers options to check for what files in what directories have changed could this perhaps be used.
Alternatively could yet another GitHub Action be created, which listens for pushes towards the src folder and then triggers a workflow run on Jenkins (Maybe doing a POST request to the Jenkin's VCS URL? Idk)

#
SlimeDog
[PlaceholderAPI/PlaceholderAPI] New comment on issue #378: Configuration option to disable colors

At present, PAPI presents us with a set of color choices dictated by the dev team, that cannot be changed. Many other plugins have managed to provide options -- either to disable formatting codes altogether, or to provide color configuration capabilities, or to provide localization of all strings -- to let users choose their own color palettes. It's sad that you view this as "pandering" to your users. I strongly disagree with that characterization. The coding required is minimal to render for...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New comment on issue #386: Only build on jenkins when there are code changes

Could also just drop jenkins and use github actions entirely. 🙂

In what way? GitHub Actions doesn't store the generated files by default (There is some sort of temp/limited storage, but that isn't optimal.)
It setups a workflow based on the repo maybe sets up other stuff based on the used actions (like f.e. a Git Repo when using Checkout Action), performs the actions in order and after that cleans up everything which essentially means, it deletes the stuff afterwards.

The only reliab...

#
Sxtanna
[PlaceholderAPI/PlaceholderAPI] New comment on issue #378: Configuration option to disable colors

Many other plugins have managed to provide options -- either to disable formatting codes altogether, or to provide color configuration capabilities, or to provide localization of all strings

Every aspect of PAPI commands are locked behind permissions, and are really only meant to be ran by administration of a server.

Many plugins do in fact provide these levels of customization, but it just doesn't make sense for PAPI.

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New branch created: change/wiki\-navigation
#
Andre601
[PlaceholderAPI/PlaceholderAPI] Pull request opened: #388 Update navigation

Please read

Please make sure you checked the following:

  • You checked the [Pull requests] page for any upcoming changes.
  • You documented any public code that the end-user might use.
  • You followed the [contributing file]

Type

  • [ ] Internal change (Doesn't affect end-user).
  • [ ] External change (Does affect end-user).
  • [x] Wiki (Changes towards the [Wiki]).
  • [ ] Other: __________

Description

Provide additional information if needed.

[Pull request...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] branch deleted: feature/add\-wiki\-action
#
Andre601
[PlaceholderAPI/PlaceholderAPI] branch deleted: change/reposition\-urls\-in\-templates
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
#
Andre601
[PlaceholderAPI/PlaceholderAPI] Issue opened: #389 Reduce GitHub spam

Please read

This template is only for requesting new information to be added to the wiki or for updating of already existing information!
Please use the [Request change (PlaceholderAPI)] template to ask for additions/changes to the plugin PlaceholderAPI.
For reporting bugs of PlaceholderAPI use the [Bug report] template.

Also make sure that the [wiki] doesn't already have the change and that there isn't a issue about this change on the [issues] page.

Type

What kind of c...

#
Andre601
[PlaceholderAPI/PlaceholderAPI] New branch created: update/request\-change\-wiki
#
Andre601
[PlaceholderAPI/PlaceholderAPI] Pull request opened: #390 Update request change \(wiki\) template

Please read

Please make sure you checked the following:

  • You checked the [Pull requests] page for any upcoming changes.
  • You documented any public code that the end-user might use.
  • You followed the [contributing file]

Type

  • [ ] Internal change (Doesn't affect end-user).
  • [ ] External change (Does affect end-user).
  • [x] Wiki (Changes towards the [Wiki]).
  • [ ] Other: __________

Description

Provide additional information if needed.

The template now ment...

still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
mint valveBOT
still narwhalBOT
still narwhalBOT