Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OneBlock phases intermix #326

Closed
Razorxd1993 opened this issue Jul 17, 2023 · 20 comments · Fixed by #330
Closed

OneBlock phases intermix #326

Razorxd1993 opened this issue Jul 17, 2023 · 20 comments · Fixed by #330
Assignees
Labels
Status: Done Type: Bug Something isn't working under investigation this is being checked out

Comments

@Razorxd1993
Copy link

Expected behavior

Unfortunately, I have the problem that my phases start to blend together from about the middle of the count.
So mobs and blocks from other phases appear that should only come in several next phases or already were.

Observed/Actual behavior

Steps/models to reproduce

Unfortunately it is not possible to reproduce

BentoBox version

[20:16:59 INFO]: Ausführen von PAPER 1.20.1.
[20:16:59 INFO]: BentoBox version: 1.24.0
[20:16:59 INFO]: Datenbank: JSON
[20:16:59 INFO]: Geladene Spielewelten:
[20:16:59 INFO]: oneblock_world (OneBlock): Oberwelt, Nether, End
[20:16:59 INFO]: Geladene Addons:
[20:16:59 INFO]: AOneBlock 1.14.0-SNAPSHOT-b354 (ENABLED)
[20:16:59 INFO]: Biomes 2.2.0-SNAPSHOT-b250 (ENABLED)
[20:16:59 INFO]: Border 4.1.1-SNAPSHOT-b179 (ENABLED)
[20:16:59 INFO]: Chat 1.1.4 (ENABLED)
[20:16:59 INFO]: ControlPanel 1.13.1-SNAPSHOT-b60 (ENABLED)
[20:16:59 INFO]: DimensionalTrees 1.6.1-SNAPSHOT-b87 (ENABLED)
[20:16:59 INFO]: ExtraMobs 1.12 (ENABLED)
[20:16:59 INFO]: Level 2.11.0-SNAPSHOT-b540 (ENABLED)
[20:16:59 INFO]: Likes 2.4.0-SNAPSHOT-b96 (ENABLED)
[20:16:59 INFO]: Limits 1.19.1-SNAPSHOT-b316 (ENABLED)
[20:16:59 INFO]: MagicCobblestoneGenerator 2.5.1 (ENABLED)
[20:16:59 INFO]: Visit 1.7.0-SNAPSHOT (ENABLED)
[20:16:59 INFO]: Warps 1.13.0-SNAPSHOT-b350 (ENABLED)

Plugin list

Paper Plugins:
EconomyShopGUI, WolfyUtilities

Bukkit Plugins:
AdvancedChests, AntiMapCopy, AutomaticBroadcast, Autorank, Baconize, BentoBox, BentoBox-AOneBlock, BentoBox-Biomes, BentoBox-Level, BentoBox-Likes, BentoBox-Limits, BentoBox-MagicCobblestoneGenerator, BentoBox-Visit, BentoBox-Warps, BetterHeads, BetterReports, BlockPiglinsTranformation, BottledExp, Bundle, BungeeTabListPlus
ChatColor2, ChestShop, Citizens, CMILib, ColoredPrefix, CommandBlocker, CommandPanels, CommandToItem, ConsoleSpamFix
CustomCrafting, DecentHolograms, *DisableJoinMessage, eBackup, Elevator, Essentials, EssentialsSpawn, etwSlots, ExcellentCrates, ExcellentEnchants, FarmControl, FilteredHoppers, HiveChecker, HomeGUI, HopperSorter, InventoryRollbackPlus, InvSeePlusPlus, InvSeePlusPlus_Clear, InvSeePlusPlus_Give, *InWartung, ItemEffects, ItemsAdder, JetsAntiAFKPro, KeepItems, KGenerators, LitePlaytimeRewards, LitMinions, LoneLibs, LuckPerms, Multiverse-Core, Multiverse-Portals, MyCommand, MyHomeRules, NBTAPI, NewbsCreative, NexEngine, NHGrinder, ntdLuckyBlock, OtherDrops, PlaceholderAPI, Plan, PlayerControl, PlugManX, ProtocolLib, RealScoreboard, RHSignItem, SetSpawn, Shopkeepers, *SimpleSit, sleep-most, TabCompleteFilter, TabTPS, TempFly, TradeSystem, TreeFeller, Tweakin, UltimateAutoRestart, Vault, VentureChat, ViaBackwards, ViaVersion, VillagerInventory, VoidGen, VoidTotem, Votifier, VotingPlugin, WorldBorderAPI, WorldEdit, worldedit-selection-viewer, WorldGuard, YamipaPlugin

Other

No response

@tastybento
Copy link
Member

tastybento commented Jul 17, 2023

Hmm. That shouldn't happen. Some questions:

  1. Did you customize the phases at all?
  2. Does this happen with all players, or just one (or a few)? If just a few/one who has this issue?
  3. Can you share your AOneBlock folder and BentoBox database folder? This will help me try and replicate the issue you are seeing (zip and drag and drop in this reply).
  4. When did you notice this? Have you always run the snapshot release shown? Or did you update recently?
  5. For a player that has this problem, are they going through the phases for the first time? Or have they repeated them? What is the count and other info for them?

@tastybento tastybento added the under investigation this is being checked out label Jul 17, 2023
@tastybento tastybento self-assigned this Jul 17, 2023
@Razorxd1993
Copy link
Author

  1. yes all phases have been customized by yourself
  2. all players have this
  3. uploaded
  4. the error was reported to me only recently, unfortunately I do not know since when this exists. Yes I have only recently installed the latest update (about 2 weeks ago).
  5. no matter, this occurs the first time you play through, as well as multiple times.

@Razorxd1993
Copy link
Author

AOneBlock+Database.zip

@tastybento
Copy link
Member

I can't see any issues with the phase files. Unfortunately, short of playing through a whole game myself, I cannot see a way to replicate this. What I need is the name/UUID of the player that is having an issue with this right now so I can "become" him and try and see if happens.

From the code though, I cannot see how this could happen. Each phase is loaded separately into phase objects that don't share any data. If a user's block count were to change to another phase then they would get a popup showing the new phase's name when they broke a block. This phase is then used to select a mob or block. Do players see a pop up showing the phase as changed?

If you could get a video of this happening it would be useful.

@Razorxd1993
Copy link
Author

Good morning :)
So mainly it's noticed in at these UUID's that have reported it:

6c510ad0-5f33-4635-b473-2cc0ce5ff813
5115ea28-9467-43c1-8048-bc7864d30144
e3f23229-2258-41d2-b1a9-a9c4789fe958

Yes the players often see a pop-up/title that the phase has changed, although they are still in a phase, a video I submit later.

Kind regards

@tastybento
Copy link
Member

This information helps. Thanks.

@tastybento
Copy link
Member

I can't find anything in the code around block breaking that would cause this to happen. All the data structures are loaded correctly and programatic tests show that they output the correct phase for the block count number. So, my current theory is that the block count itself is changing somehow. If this happens again, I'd like the user to run the /ob count command to see what it says. Then share what phase they were on and then what it became. Is the count increasing or decreasing?
Other than that, I can only think that maybe some mod or admin is messing with the count using the admin setcount command! (Joke).

@Razorxd1993
Copy link
Author

Razorxd1993 commented Jul 19, 2023

Hi, the /ob setcount command is not used in our system.
I copied all files to a test server and was able to replicate the problem directly in the "Berge" phase.
There appear from about 400-500 mined blocks snow blocks and powdersnow blocks, which are not entered in the phase, but appear one phase later in the "winter".

Hi, the /ob setcount command is not used in our system.
I copied all files to a test server and was able to replicate the problem directly in the Berge phase.
There appear from about 400-500 mined blocks snow blocks and powdersnow blocks, which are not entered in the phase, but appear one phase later in the "winter".

Also, the mobs of the next phase "Strays" spawn in the "Berge" phase.

image

Also you can see now under /ob phases that 2 phase are active at the same time.
image

tastybento added a commit to BentoBoxWorld/BentoBox that referenced this issue Jul 20, 2023
If glow was applied to a panel item, but then the glow setting was set
to false, the glow was not removed from the panel item.

Relates to BentoBoxWorld/AOneBlock#326
@tastybento
Copy link
Member

I was able to replicate the glow on two phases at the same time, and it was caused by a bug in the PanelItem class where the glow was not removed properly if you click different phases to jump to them. It does not actually indicate that 2 phases are active at the same time, it was just a bug where the glow was not removed. I've fixed it in the latest BentoBox, so thanks for finding that one!

For the snow blocks, I couldn't make any generate when I ran through the Berge phase. How did you get to that phases? Did you jump to it somehow? Were you breaking blocks in Creative mode?

I did see a number of calcite blocks, which are white - could those be them?
Screenshot 2023-07-20 at 12 02 41 AM

If you can replicate the snow, please do a screenshot of the F3 page showing the count if you can. Then maybe share the database and tell me the UUID of the player and I'll investigate more.

@Razorxd1993
Copy link
Author

Hi yes, I played up to the stage, some blocks also in creative to get there faster.
But my players also have the problem and they can not use gamemode.

No it was not cold time, it was powder snow and also normal snow block. (see screenshot)
image

UUID: 630073f2-1577-4b9e-a751-d1de22b84767

This is the DataBase folder:
Databases.tar.gz

I have played yesterday all new updates, there were also again 2 phases displayed in the gui.

Kind regards

@tastybento
Copy link
Member

This is totally weird. I can't replicate at all. Also, I am very surprised that you see two phases glowing in the GUI. Do you click on the phase icons to change phases at all?

I think the only way to see this is to try a copy of your server with the world. I thought about giving you a build with debugging on, but I can't think of what I would try to check for because I cannot see how this can be happening at all. I'm totally stumped, sorry.

@Razorxd1993
Copy link
Author

Is it possible to contact you directly somehow? I don't want to upload my whole server publicly to GitHub.
Kind regards

@tastybento
Copy link
Member

Yes, absolutely. Email is tastybento@bentobox.world.

@Razorxd1993
Copy link
Author

Hi I have just sent you the EMail.

Kind regards :)

@tastybento
Copy link
Member

Thanks for the files. When I start the server, I can see in the log that AOneBlock phases are loaded twice. When I removed all the plugins except for BentoBox they are only loaded once. I don't know yet which plugin is causing this, but this look suspicious. I am going to investigate further.

@tastybento
Copy link
Member

Found it, it's ItemsAdder. I'm not sure if this is what is causing the issue, but it might be. I need to check into it. There was a PR that added support for it, so this may be the cause.

tastybento added a commit that referenced this issue Jul 23, 2023
@tastybento tastybento linked a pull request Jul 23, 2023 that will close this issue
@tastybento tastybento added Type: Bug Something isn't working done task has been completed Status: Done and removed done task has been completed labels Jul 23, 2023
@tastybento
Copy link
Member

I'm pretty sure I found the problem. I'll release a new version. Please try it and confirm it's fixed 🤞

@Razorxd1993
Copy link
Author

Thank you for your efforts, I'll test it tomorrow directly :)

@Razorxd1993
Copy link
Author

with the new update all settings in the Bentobox & the OneBlock config were "excluded".
But the phases work now.

@tastybento
Copy link
Member

Can you explain more what you mean about settings "excluded"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Done Type: Bug Something isn't working under investigation this is being checked out
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants