Sign upLogin

Battle-Cattle/BCUK-Bot-3

View on GitHub
Star

Showing 130 of 140 total issues

Distance between variable 'enabled' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        Checkbox enabled = new Checkbox("Enabled");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/JustGivingView.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Distance between variable 'campaignLink' declaration and its first usage is 10, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        TextField campaignLink = new TextField("Campaign Link");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/JustGivingView.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Distance between variable 'userID' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        long userID = userTools.getCurrentUsersID();
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/MainView.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Distance between variable 'logo' declaration and its first usage is 11, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        Image logo = new Image(resolvedImage, "logo");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/layout/MainLayout.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Parameter name 'UserId' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    Optional<User> findById(@NotNull Long UserId);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/user/UserRepository.java by checkstyle

Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

To validate catch parameters please useCatchParameterName.

To validate lambda parameters please useLambdaParameterName.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Parameter name 'UserId' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    void deleteById(@NotNull Long UserId);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/user/UserRepository.java by checkstyle

Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

To validate catch parameters please useCatchParameterName.

To validate lambda parameters please useLambdaParameterName.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Unicode escape(s) usage should be avoided.
Open

    public static final ReactionEmoji.Unicode B = ReactionEmoji.unicode("\uD83C\uDDE7");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/discord/Emoji.java by checkstyle

Restricts usingUnicode escapes (such as \u221e).It is possible to allow using escapes fornon-printable, control characters.Also, this check can be configured to allow using escapesif trail comment is present. By the option it is possible toallow using escapes if literal contains only them.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Unicode escape(s) usage should be avoided.
Open

    public static final ReactionEmoji.Unicode D = ReactionEmoji.unicode("\uD83C\uDDE9");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/discord/Emoji.java by checkstyle

Restricts usingUnicode escapes (such as \u221e).It is possible to allow using escapes fornon-printable, control characters.Also, this check can be configured to allow using escapesif trail comment is present. By the option it is possible toallow using escapes if literal contains only them.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Empty catch block.
Open

            {
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/collection/CollectionUtil.java by checkstyle

Checks for empty catch blocks.By default check allows empty catch block with any comment inside.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Parameter name 'Clip' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    @NotNull <S extends Clip> S save(@NotNull S Clip);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/collection/clip/ClipRepository.java by checkstyle

Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

To validate catch parameters please useCatchParameterName.

To validate lambda parameters please useLambdaParameterName.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Parameter name 'Joke' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    @NotNull <S extends Joke> S save(@NotNull S Joke);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/collection/joke/JokeRepository.java by checkstyle

Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

To validate catch parameters please useCatchParameterName.

To validate lambda parameters please useLambdaParameterName.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Parameter name 'StreamerId' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    void deleteById(@NotNull Integer StreamerId);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/twitch/streams/streamer/StreamerRepository.java by checkstyle

Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

To validate catch parameters please useCatchParameterName.

To validate lambda parameters please useLambdaParameterName.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

'+' should be on a new line.
Open

        String url = "https://id.twitch.tv/oauth2/token?" +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/twitch/TwitchBot.java by checkstyle

Checks the policy on how to wrap lines on operators.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Method name 'ButtonEvent' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
Open

    private void ButtonEvent(Runnable f)
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/bot/BotView.java by checkstyle

Checks that method names conform to a specified pattern.

Also, checks if a method name has the same name as the residing class.The default is false (it is not allowed). It is legal in Java to havemethod with the same name as a class. As long as a return type is specifiedit is a method and not a constructor which it could be easily confused as.Does not check-style the name of an overridden methods because the developerdoes not have a choice in renaming such methods.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Unicode escape(s) usage should be avoided.
Open

    private static final String[] emojis = {"\uD83C\uDF47", "\uD83C\uDF4A", "\uD83C\uDF52", "\uD83C\uDF53"};
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/fun/slot/SlotGame.java by checkstyle

Restricts usingUnicode escapes (such as \u221e).It is possible to allow using escapes fornon-printable, control characters.Also, this check can be configured to allow using escapesif trail comment is present. By the option it is possible toallow using escapes if literal contains only them.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

'+' should be on a new line.
Open

                File.separator +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/discord/music/SFXHandler.java by checkstyle

Checks the policy on how to wrap lines on operators.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Distance between variable 'appId' declaration and its first usage is 8, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        TextField appId = new TextField("App ID");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/JustGivingView.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Unicode escape(s) usage should be avoided.
Open

        Arrays.fill(display, "\u2753");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/fun/slot/SlotGame.java by checkstyle

Restricts usingUnicode escapes (such as \u221e).It is possible to allow using escapes fornon-printable, control characters.Also, this check can be configured to allow using escapesif trail comment is present. By the option it is possible toallow using escapes if literal contains only them.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

'+' should be on a new line.
Open

        return "." +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/discord/music/SFXHandler.java by checkstyle

Checks the policy on how to wrap lines on operators.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Distance between variable 'channels' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
Open

        MultiSelectComboBox<String> channels = new MultiSelectComboBox<>("Twitch Channels");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/JustGivingView.java by checkstyle

Checks the distance between declaration of variable and its first usage.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Severity
Category
Status
Source
Language