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

Error with File Generation from challenges, and Iron Age Challenge not completing #252

Closed
Weymouth87 opened this issue Jul 25, 2020 · 6 comments
Labels
Status: Pending Waiting for a developer to start working on this issue. Type: Bug A bug in code.

Comments

@Weymouth87
Copy link

Description

Describe the bug

Iron age challenge saying that there are items missing when there is not.
Tried to use /[admin_command] challenges defaults generate and received an error.

Steps to reproduce the behavior

Expected behavior

Screenshots and videos (Optional)

https://prnt.sc/tohng3 https://gyazo.com/3c846e3bdd1ccc21e627ba348d54c501
Archive.zip

Environment

Output of /bbox version (Mandatory)
Warps 1.10.0 (ENABLED)
25.07 22:06:48 [Server] INFO TwerkingForTrees 1.4.1-SNAPSHOT-b46 (ENABLED)
25.07 22:06:48 [Server] INFO MagicCobblestoneGenerator 2.1.0-SNAPSHOT-b154 (ENABLED)
25.07 22:06:48 [Server] INFO Limits 1.14.0 (ENABLED)
25.07 22:06:48 [Server] INFO Level 2.3.4-SNAPSHOT-b403 (ENABLED)
25.07 22:06:48 [Server] INFO Greenhouses 1.2.0-SNAPSHOT-b242 (ENABLED)
25.07 22:06:48 [Server] INFO DimensionalTrees 1.6.1-SNAPSHOT-b81 (ENABLED)
25.07 22:06:48 [Server] INFO ControlPanel 1.13.0-SNAPSHOT-b38 (ENABLED)
25.07 22:06:48 [Server] INFO CheckMeOut 1.0.2 (ENABLED)
25.07 22:06:48 [Server] INFO Chat 1.1.0-SNAPSHOT-b30 (ENABLED)
25.07 22:06:48 [Server] INFO Challenges 0.8.4-SNAPSHOT-b428 (ENABLED)
25.07 22:06:48 [Server] INFO BSkyBlock 1.14.1-SNAPSHOT-b670 (ENABLED)
25.07 22:06:48 [Server] INFO Biomes 1.13.0-SNAPSHOT-b180 (ENABLED)
25.07 22:06:48 [Server] INFO Loaded Addons:
25.07 22:06:48 [Server] INFO bskyblock_world (BSkyBlock): Overworld, Nether, The End
25.07 22:06:48 [Server] INFO Loaded Game Worlds:
25.07 22:06:48 [Server] INFO Database: JSON
25.07 22:06:48 [Server] INFO BentoBox version: 1.15.0-SNAPSHOT-b1845
25.07 22:06:48 [Server] INFO Running PAPER 1.15.2.

Plugins (Optional)

Plugins (72): ActionHealth*, Advanced_Drop_Party, AdvancedAnnouncement*, AdvancedEnchantments, AdvancedPortals, ArmorStandTool, AsyncWorldEdit, AutoPickup, BannerMaker, BentoBox, BestTools, BookShelf, BottledExp, BuycraftX, ChatColor, ClearLag, CMI, CMIPaperLib, CosmicFunds, CrazyAuctions, CrazyCrates, CrazyEnvoy, CropClick*, CustomHeads*, DeluxeChat, DeluxeMenus, DeluxeTags*, EditableSign, Elevator, Essentials, EssentialsChat, EssentialsSpawn, EternalLight, FlightControl, Graves, HolographicDisplays, HolographicExtension*, HolographicScoreboard*, HubParkour, InteractionVisualizer, Jackpot, LeaderHeadsRevamped, LightAPI, LuckPerms, Multiverse-Core, Multiverse-NetherPortals, Multiverse-Portals, NerfPhantoms, PlaceholderAPI, PlugMan, Prism, ProtocolLib, PvPManager, ShopChest, ShopGUIPlus, Statz, StreamingDrops, TabList, TouchscreenHolograms, TreeFeller, VarLight, Vault, ViaVersion, VoidGenerator, VoidTeleport, WildStacker, WildStackerMobsPatch, WolfyUtilities, WorldEdit, WorldGuard, XPBoost, zDiscord*

Additional context (Optional)

@tastybento tastybento transferred this issue from BentoBoxWorld/BSkyBlock Jul 25, 2020
@tastybento
Copy link
Member

What error do you get?

@tastybento tastybento added Status: Cannot replicate Could not replicate the issue on a fresh install. More details are needed. Status: Need answer Waiting for more information to be provided by the issue's author. labels Jul 25, 2020
@BONNe BONNe removed the Status: Cannot replicate Could not replicate the issue on a fresh install. More details are needed. label Jul 26, 2020
@BONNe
Copy link
Member

BONNe commented Jul 26, 2020

From Discord
https://pastebin.com/Ph9XMWHJ

@BONNe
Copy link
Member

BONNe commented Jul 26, 2020

For some reason, in the database, you provided some challenges contains NULL items.
That is the reason for crashes.

I latest build I add protection methods, that will not allow loading these challenges.

I tried to do the IronAge challenge, and I was successful.
However, there is a way how to make that challenge is not working:

  • if an item has a custom name it will not be considered as a valid item
  • if an item has an enchantment, it will not be considered as a valid item
  • if an item has been damaged, it will not be considered as a valid item

@Weymouth87
Copy link
Author

Thank you :)

Would the items be accepted if the gear was previously damaged but repaired fully via an anvil?

@Weymouth87
Copy link
Author

Just tested with brand new armour and it works, thank you. The player was using repaired armour instead :)

@BONNe
Copy link
Member

BONNe commented Jul 26, 2020

I will try to fix that in next release.

@BONNe BONNe added Status: Pending Waiting for a developer to start working on this issue. Type: Bug A bug in code. and removed Status: Need answer Waiting for more information to be provided by the issue's author. labels Jul 12, 2021
@BONNe BONNe closed this as completed in 08d1561 Sep 24, 2021
BONNe added a commit that referenced this issue May 6, 2022
* Version 0.8.5

* Retranslated zh-CN.yml (#273)

* 50% completed.

* 60% completed.

* 63% completed.

* Completed.

* Change the indentation, some improvements.

Co-authored-by: zhangYi <apachezy@hotmail.com>

* Updated german language file (#278)

fixed double & and double whitespace

* Fixes a mistaken permission for completing multiple challenges at once.

* Fixes translated placeholders in PL translation.

Note: translation looks bad.

* Update CompleteChallengeCommandTest.java

* Upgrade to BentoBox 1.17 API changes.

Implement Pladdon functionality.
Compile against java 16 and Spigot 1.17

* Fix Java 16 compilation.

* Use BentoBox 1.17.0

* Update pom.xml

* Create Statistic Requirement for Challenges addon.

Statistic requirement is a new type of challenge that is based on Statistic page for clients.

* Switch to annotations instead of plugin.yml file.

* Move managers to a separate directory.

* Add mojang authLib instead of NMS.

* Rename classes to Selectors.

Split single and multiple item selectors for easier implementation.
Update proper locales.

* Implement customizable user panels.

Server owners can customize 3 panels:
- main panel
- gamemode selector
- multiple completions

Panel functions will be explained in docs later.

* Update all admin panels.

Admin panels will not contain better locales codding and easier-to-improve design.

Remove old and unused GUIs.

* Remove unused adapters.

Updates Challenges and ChallengesLevel objects.
Add TypeMigrationAdapter that will fix issue with renamed challenge type.

* Update commands.

Commands will now call correct GUI.

* Update Settings file.

Remove unused parts.

* Fixes ChallengesManager and Completer.

* Adds panel saving to the `/challenges/panels` directory.

* Updates locales file.

Complete rework of the locales file.
Very sorry translators :( no migration.

* Updates pom.xml

* Updates tests.

ChallengesGUITest is removed because GUI is removed.

* Update default.json

Split text into multiple lines.

* Create template YAML file.

This file format is for people who has an alergy with ingame GUI.

* Implements Template reading.

Add template loading via Admin Panel.
Improve LibraryPanel so it could find json and yml files.

* Improve coloring scheme a bit.

* Change settings file.

Add ability to change commands for addon.
Change default mode from player challenges to island challenges.

* Update Main addon class.

Move vault and level detection after everything is loaded.
Update command names.

* Update all commands.

Commands now will have an option to change their call values.

* Update default config value.

* Fixes #264

Challenges Menu will be opened only if player is in correct world.

* Changes User#sendMessage to Utils#sendMessage

This allows add "prefix" to all messages send from Challenges addon.

* Separate singe and multiple listings.

* Clean up Constants a bit.

* Add meta for items translations.

* Fix permission link.

* Translates color codes for database texts.

* Fixes a bug when global commands does not displays in tab-complete.

Remove DefaultsCommand.java as it is not used anymore.

* Fixes small bugs in translation.

* Remove unnecessary "admin" tag.

* Update default locale.

* Update latvian locale to the latest version.

* Implement multi-linguistic server support.

Now server owners can specify different name, description and reward text for each challenge and level via locales file.

Add showcase example.

* Comment out showcase translation.

* Update BentoBox version

* Update missing icons for blocks.

Some blocks cannot be displayed in GUI's, and were leaving empty spaces.
This replaces their icon with a close representative.

Fixes #286

* Add missing mob heads.

* Fixes illegal stack issues in default challenges.

#249

* Change from click-to-select to a proper next/previous page tooltip

* Add search field to the PagedSelectors.

Add missing tooltips.

* Change download icon from hopper to cobweb.

* Add missing tooltips to the CommonPagedPanel

* Add search button to the CommonPagedPanel.

Search button will allow to search elements if there are more than displayed elements.

* Add missing strings into locale.

* Reorder dependencies

The Mojang dependency was blocking out the needed Google common
packages.

* Prevent errors in TryToCompleteTest

Note - tests still fail.

* Fixed errors and tests for CompleteChallengeCommandTest

* Fixed ChallengesCommandTest tests

* Fixes tests

* Fix JavaDoc, Shade plugin settings

* Updated .gitignore

* Try different spigot API version

* Remove Vault repo because it is not needed.

* Excluded unnecessary files from shading.

* Fixes #253

Adds TeamKick and TeamLeave events to the reset check.
Do not reset challenges if data is stored per island. As in that case, they will already lose their data.

* Fixes #187

Add a new method that updates unlocked level list without changing active level. This method returns if last unlocked level was changed, and in that case it triggers whole gui rebuilding.

* Fixes #269

Disable waiver amount message for last challenge level.

* Add timeout for repeatable challenges.

Relates #71

* Implement timeout respecting in challenges completion.

Implement timeout in GUI's.

Relates #71

* Implement changing Timeout in the Challenge Edit GUI.

Relates #71

* Implement an option to set which item type will ignore metadata per challenge.

Fixes #261
Fixes #252

* Fixes failing unit-test

* Removed shade plugin from POM

* Replace GuiUtils and HeadLib to the PanelUtils library.

* Link templates to the docs.

* Remove unnecessary NMS dependency.

NMS code was used for Player Heads, but instead of NMS now it uses public mojang lib.

* Address some code quality reports from SonarCloud.

Most of the errors are just sanity checks, as the most of null-pointers were already checked in other ways.

* Fixes incorrect NEXT and PREVIOUS button descriptions.

Fixes #289

* Implement MetaData ignoring for rewards.

While required items had a metadata grouping, reward items did not have it. This will fix that.

Fixes #289

* Fix an issue when edit menu did not display item amount.

* Update lv translation.

* Fixes some small bugs with translation potion base effect.

There was an issue that it tried to translate extra effects and ignored main one.

Relates to #290

* Fix a bug with completion broadcasting

Reported via Discord.

* Update pom.xml

* Fixes a bug with `-1` repeat-times

There was a bug that prevented the challenge to be completed if negative numbers were set in the "max-repeats" value.

* Improve equal item listing.

Change when items should be grouped.
Instead of relaying strictly from ignoreMetaData set, now try to group equal elements without durability check, and use set only if that fails.

* Update German translation (#295)

* Translate de.yml via GitLocalize

* Translate de.yml via GitLocalize

* Translate de.yml via GitLocalize

Co-authored-by: Patrick <patrick.wassmuth@gmx.de>
Co-authored-by: Michael F <unhappyangel83@googlemail.com>
Co-authored-by: DAge030 <dage030@web.de>

* Fix NPEs when running tests.

Note that there are still test failures, but these are assertions and
not errors.

* Fix error in test class.

Note this does not fix the failing assertion.

* Fix failing test. Make player default to being on island.

* Fixed test failures.

* Avoid potential call with a null parameter to User.getInstance

* Check for null world

* Null check

* Added null check

* Require non-nulls.

getInventory never returns null.

* Remove various code smells.

Co-authored-by: tastybento <tastybento@wasteofplastic.com>
Co-authored-by: apachezy <50116371+apachezy@users.noreply.github.com>
Co-authored-by: zhangYi <apachezy@hotmail.com>
Co-authored-by: Qumoo <76853697+Qumoo@users.noreply.github.com>
Co-authored-by: tastybento <tastybento@users.noreply.github.com>
Co-authored-by: gitlocalize-app[bot] <55277160+gitlocalize-app[bot]@users.noreply.github.com>
Co-authored-by: Patrick <patrick.wassmuth@gmx.de>
Co-authored-by: Michael F <unhappyangel83@googlemail.com>
Co-authored-by: DAge030 <dage030@web.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Pending Waiting for a developer to start working on this issue. Type: Bug A bug in code.
Projects
None yet
Development

No branches or pull requests

3 participants