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

fix: Support RegionPlugin UltimateClaims v2 and v1 #200

Merged
merged 2 commits into from
Mar 27, 2024
Merged

fix: Support RegionPlugin UltimateClaims v2 and v1 #200

merged 2 commits into from
Mar 27, 2024

Conversation

SpraxDev
Copy link
Contributor

  • This patch has been tested by a user affected by this bug
  • The URL to the plugin changed, you might want to update the link on your Spigot-Plugin-Page (https://craftaro.com/marketplace/product/65)
  • I tried keeping the code style similar to the existing files

With UltimateClaims v2 the package changed because Songoda renamed to Craftaro at one point.

This fixes a ClassNotFoundException when using v2 of the plugin. The commit also refactors the code a bit as the UltimateClaims#getInstance() method is marked as deprecated and will be removed in the future.

With v2 the package changed because Songoda renamed to Craftaro at one point in time.

This fixes a `ClassNotFoundException` when using v2 of the plugin.
The commit also refactors the code a bit as the `UltimateClaims#getInstance()` method is marked as deprecated
and will be removed in the future.
// NOT TESTED (3.1.0)
// UltimateClaims (v1.6.1) - Abandoned
// https://songoda.com/marketplace/product/ultimateclaims-the-ultimate-claiming-plugin.65
// TESTED (v2.2.0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be 3.6.12

// https://songoda.com/marketplace/product/ultimateclaims-the-ultimate-claiming-plugin.65
// TESTED (v2.2.0)
// UltimateClaims (v2.2.0 + v1.10.4)
// https://craftaro.com/marketplace/product/65
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave both links for where to find each version of the plugin. If the resource no longer exists, please leave a link to the last commit before v2. And ensure that you specify which link is for which version.

Comment on lines +35 to +37
return Class.forName("com.craftaro.ultimateclaims.UltimateClaims"); // v2
} catch (ClassNotFoundException ignore) {
return Class.forName("com.songoda.ultimateclaims.UltimateClaims"); // v1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ARR4NN ARR4NN merged commit 37e4bb1 into RonanPlugins:master Mar 27, 2024
2 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.

None yet

3 participants