Sign upLogin

Battle-Cattle/BCUK-Bot-3

View on GitHub
Star
src/main/java/com/expiredminotaur/bcukbot/web/view/settings/JustGivingView.java

Summary

Maintainability
A
0 mins
Test Coverage

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

        Binder<JustGivingSettings> binder = new Binder<>(JustGivingSettings.class);
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 '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 '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.

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.

There are no issues that match your filters.

Category
Status