v7.3.54 - fix a diagonal loop determination issue resulting in false …

…negative tree validity checks

doc/changelog.md

@@ -2,6 +2,7 @@
 
 ## v7.3 - API Expansion
 
+- v7.3.54 - fix a diagonal loop determination issue resulting in false negative tree validity checks
 - v7.3.53 - properly reset Toggle varibale when toggling functionality back on - affects "Remember Toggle Status"
 - v7.3.52 - [core update] - Support servers without Spigot base unless they want to use hex codes
 - v7.3.51 - update for 1.20.4, also make updating minecraft-version-dependent to not break servers by introducing unknown blocks or tree definitions

readme.md

@@ -74,7 +74,7 @@ It also will take down an entire tree when it is enabled in the config.
 
 ## Changelog
 
-- v7.3.53 - properly reset Toggle varibale when toggling functionality back on - affects "Remember Toggle Status"
+- v7.3.54 - fix a diagonal loop determination issue resulting in false negative tree validity checks
 - [read more](doc/changelog.md)
 
 ***

src/main/java/net/slipcor/treeassist/discovery/TreeStructure.java

@@ -1060,7 +1060,7 @@ private List<Block> findTrunk() {
                                 debug.i("It's an extra l block: " + BlockUtils.printBlock(checkBlock));
                             } else if (naturalBlocks.contains(innerCheck)){
                                 debug.i("It's a natural l block " + BlockUtils.printBlock(checkBlock));
-                            } else if (!allTrunks.contains(checkMaterial) && !allExtras.contains(checkMaterial)) {
+                            } else if (!allTrunks.contains(innerCheck) && !allExtras.contains(innerCheck)) {
                                 debug.i("Unexpected l block! Not a valid tree!");
 
                                 discoveryResult = new DiscoveryResult(config, this, FailReason.INVALID_TRUNK_BLOCK, "block C: " + BlockUtils.printBlock(checkBlock));