Skip to content

fix: solved argument error for Purpur - #140

Merged
ez-plugins merged 1 commit into
v3.xfrom
fix/folia-reflection-arguments
Jul 27, 2026
Merged

fix: solved argument error for Purpur#140
ez-plugins merged 1 commit into
v3.xfrom
fix/folia-reflection-arguments

Conversation

@ez-plugins

@ez-plugins ez-plugins commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Fixed

  • Fixed /rtp intermittently failing (observed on Purpur) with CompletionException - java.lang.IllegalArgumentException: argument type mismatch during async location search. The Folia region/global-scheduler reflection bridges in PaperPlatformScheduler and BukkitPlatformScheduler only caught ReflectiveOperationException around Method.invoke() calls; they now also catch RuntimeException and fall back to the standard Bukkit scheduler instead of letting the exception propagate. This can occur even on non-Folia servers whose classpath carries the Folia regionized-runtime marker class.
  • BukkitPlatformScheduler#executeRegion/executeGlobal/executeGlobalDelayed/executeRegionDelayed now consistently fall back to the standard Bukkit scheduler when the reflective Folia scheduler call fails, instead of silently dropping the task (matching existing PaperPlatformScheduler behavior).

Added

  • Regression test coverage for the scheduler reflection-fallback fix:
    • PaperPlatformSchedulerReflectionFallbackTest and BukkitPlatformSchedulerReflectionFallbackTest simulate a broken region/global-scheduler reflection call and verify the submitted task still runs via the standard scheduler fallback.
    • PlatformRuntimeCapabilities.PURPUR_REGIONIZED constant documenting the exact capability combination that triggered the bug.

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 63.82979% with 17 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ockexp/ezrtp/platform/BukkitPlatformScheduler.java 62.96% 6 Missing and 4 partials ⚠️
...lockexp/ezrtp/platform/PaperPlatformScheduler.java 75.00% 4 Missing ⚠️
.../skyblockexp/ezrtp/util/compat/ParticleCompat.java 0.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@ez-plugins
ez-plugins merged commit ce6540f into v3.x Jul 27, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants