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

Wildcard support #18

Merged
merged 10 commits into from
Aug 10, 2024
Merged

Wildcard support #18

merged 10 commits into from
Aug 10, 2024

Conversation

NonSwag
Copy link
Member

@NonSwag NonSwag commented Aug 9, 2024

  • paper
    • command permission
    • command hide
    • command reset
    • command unregister
  • velocity
    • command permission
    • command hide
    • command reset
    • command unregister

NonSwag added 5 commits August 9, 2024 22:29
Adjusted the indentation for the getSuggestions method to improve readability. This change does not alter the functionality but ensures consistent code formatting.
Utilize `findCommands` method to streamline command lookup and matching. This enhances maintainability and ensure consistent command filtering across multiple operations like hide, register, reveal, and unregister. Introduces regex pattern matching to support more flexible command searches.
Refactored `findCommands` methods to be static for better accessibility and potential use without needing an instance of `PaperCommandRegistry`. This change will also help in simplifying unit tests related to command finding functionality.
Replace direct permission and reset operations with bulk command handling using 'findCommands'. This enhancement ensures that all relevant commands are consistently updated or reset, streamlining the internal logic and improving maintainability.
@NonSwag NonSwag self-assigned this Aug 9, 2024
@NonSwag NonSwag linked an issue Aug 10, 2024 that may be closed by this pull request
Introduce the CommandFinder interface to streamline command search functionality. Added implementations for both Paper and Velocity platforms. Also updated Commander and related plugins to integrate the CommandFinder feature.
Refactor PaperCommandRegistry and PaperPermissionOverride to use the plugin's commandFinder utility for command searching. This enhances code maintainability by centralizing command lookup logic and removes redundant static methods.
Update `ProxyCommandRegistry` and `ProxyPermissionOverride` to improve command lookup and manipulation. Ensure hidden, unregistered, and overridden commands are accurately processed by integrating enhanced command finding logic.
Bump version from 4.0.0 to 4.1.0 in CommanderPlugin.java and build.gradle.kts. This reflects the latest changes and improvements in the project.
@NonSwag NonSwag merged commit 3e1cb11 into master Aug 10, 2024
2 checks passed
@NonSwag NonSwag deleted the wildcard branch August 10, 2024 15:14
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.

Wildcard support
1 participant