Skip to content
analyze and review code

remove the flag firstArrivingMeterValueIfMultiple and its usage #800

Sign in to view logs

remove the flag firstArrivingMeterValueIfMultiple and its usage

remove the flag firstArrivingMeterValueIfMultiple and its usage #800

Triggered via pull request March 26, 2024 11:53
@goekaygoekay
synchronize #1423
1209-remove-firstArrivingMeterValueIfMultiple
Status Success
Total duration 2m 27s
Artifacts 1

review.yml

on: pull_request
license-check
28s
license-check
pmd
13s
pmd
checkstyle
19s
checkstyle
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Too many static imports may lead to messy code: src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L19
If you overuse the static import feature, it can make your program unreadable and unmaintainable, polluting its namespace with all the static members you import. Readers of your code (including you, a few months after you wrote it) will not know which class a static member comes from (Sun 1.5 Language Guide). TooManyStaticImports (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#toomanystaticimports
Unused static import 'de.rwth.idsg.steve.utils.CustomDSL.date': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L45
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.ChargeBox.CHARGE_BOX': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L46
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.Connector.CONNECTOR': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L47
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.ConnectorMeterValue.CONNECTOR_METER_VALUE': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L48
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.OcppTag.OCPP_TAG': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L49
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.Transaction.TRANSACTION': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L50
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'jooq.steve.db.tables.TransactionStart.TRANSACTION_START': src/main/java/de/rwth/idsg/steve/repository/impl/TransactionRepositoryImpl.java#L51
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'de.rwth.idsg.steve.utils.TransactionStopServiceHelper.floatingStringToIntString': src/main/java/de/rwth/idsg/steve/service/TransactionStopService.java#L40
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport
Unused static import 'de.rwth.idsg.steve.utils.TransactionStopServiceHelper.kWhStringToWhString': src/main/java/de/rwth/idsg/steve/service/TransactionStopService.java#L41
Reports import statements that can be removed. They are either unused, duplicated, or the members they import are already implicitly in scope, because they're in java.lang, or the current package. UnnecessaryImport (Priority: 4, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.0.0/pmd_rules_java_codestyle.html#unnecessaryimport

Artifacts

Produced during runtime
Name Size
PMD Report Expired
1.45 KB