Sign upLogin

Battle-Cattle/BCUK-Bot-3

View on GitHub
Star

Showing 130 of 140 total issues

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

    private Set<CustomCommand> EnabledTwitchCommands;
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/user/User.java by checkstyle

Checks that instance variable names conform to a specified pattern.

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

First sentence of Javadoc is missing an ending period.
Open

    /**
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/command/Command.java by checkstyle

Checks thatJavadoc summary sentence does not contain phrases that are not recommended to use.Summaries that contain only the {@inheritDoc} tag are skipped. Check alsoviolate Javadoc that does not contain first sentence.

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

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

    public void sendToAllUsers(TwitchChat chat, String Message)
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/twitch/streams/MultiTwitch.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.

Distance between variable 'StreamerLayout' declaration and its first usage is 18, 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

            VerticalLayout StreamerLayout = new VerticalLayout();
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/StreamAnnouncementsView.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.

'+' should be on a new line.
Open

        String s = Emoji.A.getRaw() + parseText(answers.get(0)) + "\n" +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/fun/trivia/TriviaGame.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.

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

    private final Random RNG = new Random();
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/discord/PointsSystem.java by checkstyle

Checks that instance variable names conform to a specified pattern.

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 TRUE = ReactionEmoji.unicode("\u2705");
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.

Distance between variable 'playingHeader' 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

        H2 playingHeader = new H2(playing);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/MusicView.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 'deleteOld' declaration and its first usage is 12, 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 deleteOld = new Checkbox("Delete Old Messages?");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/StreamAnnouncementsView.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 'addButton' 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

        Button addButton = new Button("Add", e -> editForm.open(new Alias()));
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/commands/AliasView.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 'multiTwitch' declaration and its first usage is 7, 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 multiTwitch = new Checkbox("Send To All Multi Twitch Channels");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/commands/CommandsView.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.

'+' should be on a new line.
Open

        String s = Emoji.TRUE.getRaw() + "True" + "\n" +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/fun/trivia/TriviaGame.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 'sendButton' declaration and its first usage is 9, 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

        Button sendButton = new Button("Send Message", e -> sendBroadcast(input.getValue()));
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/admin/StreamBroadcasterView.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 'trigger' declaration and its first usage is 4, 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 trigger = new TextField("Trigger");
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/commands/CommandsView.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 'QuoteId' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
Open

    void deleteById(@NotNull Integer QuoteId);
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/collection/quote/QuoteRepository.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

            "from(select discord_user_id, rank() over (order by points desc) as rankByPoints from user_points) as q1 " +
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/sql/discord/points/UserPointsRepository.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.

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

    void delete(@NotNull Streamer Streamer);
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.

First sentence of Javadoc is missing an ending period.
Open

    /**
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/command/Command.java by checkstyle

Checks thatJavadoc summary sentence does not contain phrases that are not recommended to use.Summaries that contain only the {@inheritDoc} tag are skipped. Check alsoviolate Javadoc that does not contain first sentence.

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

First sentence of Javadoc is missing an ending period.
Open

    /**
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/command/Command.java by checkstyle

Checks thatJavadoc summary sentence does not contain phrases that are not recommended to use.Summaries that contain only the {@inheritDoc} tag are skipped. Check alsoviolate Javadoc that does not contain first sentence.

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

Distance between variable 'infoLayout' declaration and its first usage is 17, 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

            VerticalLayout infoLayout = new VerticalLayout();
Severity: Minor
Found in src/main/java/com/expiredminotaur/bcukbot/web/view/settings/StreamAnnouncementsView.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