boiscljo / SBA

An addon to ScreamingSandals bedwars plugin to hypixelify it!
GNU General Public License v3.0
27 stars 15 forks source link

Unrealized stuff and bugs #34

Open EastRane opened 1 year ago

EastRane commented 1 year ago

Using this plugin for about 2 months now, thanks for it! Some problems have appeared during this time:

All of this was in SBW 0.2.29 and SBA 1.5.9 and all of this remained in 0.2.30 & 1.5.11.

Paper 1.20.1. Bukkit Plugins: Authy, BedWars, Citizens, DecentHolograms, Essentials, FastAsyncWorldEdit, LuckPerms, MdMotd, Multiverse-Core, NoOffhand, OldCombatMechanics, PlaceholderAPI, ProtocolLib, RegenManager, SBA, SkinsRestorer, spark, SuperLobby, TabList, Vault, WhitelistByTime, YamipaPlugin SBA config: https://pastebin.com/hye06mJz SBW config: https://pastebin.com/TMk6ffuj

boiscljo commented 1 year ago

When ugrading internal plugin's files (after plugin's update), it deletes my changes in languages files.

You are using /sba upgrade ,which reset everything, most of the time doing /sba cancel is better and old config will work just fine.

boiscljo commented 1 year ago

I'm not quite sure I'm getting this right, but it seems like the values for fireballs and TNT are synchronized? In terms of knockback and such. Seems to me they should be set up separately. And it's pretty hard to configure fireballs for me, maybe it would be much better if you comment some parameters in config etc, because it doesn't look like hypixel's

The way the configs are generated I cannot put comments in there, but it will be added on the FAQ. It won't be like that server. I'm not even sure how the parameters affect the jump. But in short:

I'm not sure why they are together, it was like the when @pronze owned the plugin, I just took over.

boiscljo commented 1 year ago

When I translate the internal NPC strings (games-inventory) and then do /sba generate, my string translation resets.

SBA generate completely reset games-inventory with the new games, I guess that also affect the npc somehow, where were those strings located

boiscljo commented 1 year ago

Also I'm not sure if this is to you or the developer of the bedwars plugin itself, but the villagers don't exist in spectator mode (to spectators of arena).

It's made on purpose like this because shops don't really exists they are created only for people in survival mode. (Packet based, not a real entity)

EastRane commented 1 year ago

SBA generate completely reset games-inventory with the new games, I guess that also affect the npc somehow, where were those strings located

Yes, it completely overwrites the file in the games-inventory folder when I use /sba generate. And there are lines of names/lores of inventory items and items with the join to the arena. If it's worth the coding time, imo it would make more sense to update only the arena list (or maybe have a separate command specifically for updating arenas within games-inventory, it makes even more sense), and leave all other lines as they are (to prevent user's custom non-join-arena-items inventory settings from reseting)