Uploaded image for project: 'Active and Available Inventory'
  1. Active and Available Inventory
  2. AAI-569

Fix Sonar issues for AAI (Active and Available Inventory)

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Done
    • Icon: Lowest Lowest
    • Dublin Release
    • None

      Story created for fixing SONAR defects.

        1.
        		 Sonar Major : Remove useless parentheses. Sub-task Closed aishkashyap
        2.
        		 To either Log or rethrow exception Sub-task Closed aishkashyap
        3.
        		 Make field final static Sub-task Closed aishkashyap
        4.
        		 To either Log or rethrow exception Sub-task Closed aishkashyap
        5.
        		 Remove the declaration of runtime exception Sub-task Closed aishkashyap
        6.
        		 Define a constant instead of duplicating this literal "model-elements" 8 times. Sub-task Closed aishkashyap
        7.
        		 Define a constant instead of duplicating this literal "relationship-list" Sub-task Closed aishkashyap
        8.
        		 Remove the declaration of thrown exception Sub-task Closed aishkashyap
        9.
        		 Use isEmpty() to check whether the collection is empty or not Sub-task Closed aishkashyap
        10.
        		 Remove duplicate literal Sub-task Closed aishkashyap
        11.
        		 Replace the type specification in constructor call with the diamond operator. Sub-task Closed aishkashyap
        12.
        		 Add the "@Override" annotation . Sub-task Closed aishkashyap
        13.
        		 Remove parenthesis Sub-task Closed aishkashyap
        14.
        		 Remove those useless parentheses Sub-task Closed aishkashyap
        15.
        		 Remove redundant "static" qualifier. Sub-task Closed aishkashyap
        16.
        		 Add the "@Override" annotation above method signature Sub-task Closed aishkashyap
        17.
        		 Add the "@Override" annotation. Sub-task Closed aishkashyap
        18.
        		 Add the "@Override" annotation above method . Sub-task Closed aishkashyap
        19.
        		 Make this final field static. Sub-task Closed aishkashyap
        20.
        		 Add the "@Override" annotation above method signature. Sub-task Closed aishkashyap
        21.
        		 Add the "@Override" annotation above method signature. Sub-task Closed aishkashyap
        22.
        		 Add the "@Override" annotation. Sub-task Closed aishkashyap
        23.
        		 Add the "@Override" annotation above method signature. Sub-task Closed aishkashyap
        24.
        		 Define a constant instead of duplicating this literal "Content-Type" 3 times. Sub-task Closed aishkashyap
        25.
        		 Add the "@Override" annotation above this method signature. Sub-task Closed aishkashyap
        26.
        		 Add the "@Override" annotation above this method signature. Sub-task Closed aishkashyap
        27.
        		 Add the "@Override" annotation above this method signature. Sub-task Closed aishkashyap
        28.
        		 Replace the type specification in this constructor call with the diamond operator ("<>"). Sub-task Closed aishkashyap
        29.
        		 Remove the declaration of thrown runtime exception. Sub-task Closed aishkashyap
        30.
        		 Remove the declaration of thrown runtime exception . Sub-task Closed aishkashyap
        31.
        		 Replace the type specification in this constructor call with the diamond operator ("<>"). Sub-task Closed aishkashyap
        32.
        		 Annotate the "IModelParser" interface with the @FunctionalInterface annotation . Sub-task Closed aishkashyap
        33.
        		 Replace the type specification with the diamond operator . Sub-task Closed aishkashyap
        34.
        		 Add the "@Override" annotation. Sub-task Closed aishkashyap
        35.
        		 Merge if statement with the enclosing one. Sub-task Closed aishkashyap
        36.
        		 Add the "@Override" annotation above this method signature. Sub-task Closed aishkashyap
        37.
        		 Replace the type specification with the diamond operator. Sub-task Closed aishkashyap
        38.
        		 Replace the type specification in this constructor call with the diamond operator . Sub-task Closed aishkashyap
        39.
        		 Define a constant instead of duplicating this literal "depth" . Sub-task Closed aishkashyap
        40.
        		 Sonar Define a constant instead of duplicating this literal. Sub-task Closed aishkashyap
        41.
        		 Define a constant instead of literal. Sub-task Closed aishkashyap
        42.
        		 Sonar Define a constant instead of duplicating this literal . Sub-task Closed aishkashyap
        43.
        		 The block of commented-out lines of code should be removed. Sub-task Closed aishkashyap
        44.
        		 Add the "@Override" annotation above method signature. Sub-task Closed aishkashyap
        45.
        		 Replace the type specification in this constructor call with the diamond operator . Sub-task Closed aishkashyap
        46.
        		 Add the "@Override" annotation above this method signature. Sub-task Closed aishkashyap
        47.
        		 Replace the type specification in this constructor call with the diamond operator. Sub-task Closed aishkashyap
        48.
        		 Make this final field static too. Sub-task Closed aishkashyap
        49.
        		 Sonar major:Move the "aai-internal" string literal on the left side of this string comparison for aai-traversal Sub-task Closed swapnalipode
        50.
        		 Replace the type specification in this constructor call with the diamond operator. Sub-task Closed aishkashyap
        51.
        		 Remove this unused import 'java.util.ArrayList' Sub-task Closed swapnalipode
        52.
        		 Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$' Sub-task Closed prasad811
        53.
        		 Move the "true" string literal on the left side of this string comparison. Sub-task Closed swapnalipode
        54.
        		 Sonar Minor : Rename this local variable name to match the regular expression for AAI-esr-server Sub-task Closed prasadnaidu
        55.
        		 Add a private constructor to hide the implicit public one. Sub-task Closed swapnalipode
        56.
        		 Rename this field to match the regular expression Sub-task Closed prasadnaidu
        57.
        		 Either log or rethrow this exception Sub-task Closed swapnalipode
        58.
        		 Remove this useless assignment to local variable "messages". Sub-task Closed swapnalipode
        59.
        		 Add a private constructor Sub-task Closed prasadnaidu
        60.
        		 Add a private constructor to hide the implicit public one Sub-task Closed dthakur
        61.
        		 Remove this useless assignment to local variable result Sub-task Closed swapnalipode
        62.
        		 Renamed the local variable name to match the regular expression '^[a-z][a-zA-Z0-9]*$'. Sub-task Closed dthakur
        63.
        		 Replace "==" with "===" Sub-task Closed prasadnaidu
        64.
        		 Either log or rethrow this exception. Sub-task Closed swapnalipode
        65.
        		 Replace "!=" with "!==" Sub-task Closed prasadnaidu
        66.
        		 Replace "==" with "===" Sub-task Closed prasadnaidu
        67.
        		 Replace "==" with "===" Sub-task Closed prasadnaidu
        68.
        		 Either log or rethrow this exception Sub-task Closed prasadnaidu
        69.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        70.
        		 Add a private constructor to hide the implicit public one Sub-task Closed prasadnaidu
        71.
        		 This block of commented-out lines of code should be removed. Sub-task Closed prasadnaidu
        72.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        73.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        74.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        75.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        76.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        77.
        		 Add the "@Override" annotation above this method signature Sub-task Closed prasadnaidu
        78.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        79.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        80.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        81.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        82.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        83.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        84.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        85.
        		 Reorder the modifiers to comply with the Java Language Specification Sub-task Closed prasadnaidu
        86.
        		 Either log or rethrow this exception Sub-task Closed prasadnaidu
        87.
        		 Either log or rethrow this exception Sub-task Closed prasadnaidu
        88.
        		 Either log or rethrow this exception Sub-task Closed prasadnaidu
        89.
        		 Replace the type specification in this constructor call with the diamond operator Sub-task Closed prasadnaidu
        90.
        		 Replace type specification in constructor call with diamond operator Sub-task Closed prasadnaidu
        91.
        		 Either log or rethrow this exception Sub-task Closed prasadnaidu
        92.
        		 Fix AAIDmaapEventJMSConsumer architecture Sub-task Closed bogumil_zebek
        93.
        		 Refactoring code smells in AbstractValidatingChampGraph Sub-task Closed jczestochowska
        94.
        		 Fix sonar issues in crud/logging/LoggingUtil Sub-task Closed tswiton
        95.
        		 Refactor of code smells in AAIMicroServiceAuthCore, ArtifactGenerator and GenerateArtifactsService Sub-task Closed wiktor.garbarek
        96.
        		 Fix sonar issues in /crud/util/CrudProperties.java Sub-task Closed tswiton
        97.
        		 Fix sonar issues in /crud/service/CrudRestService Sub-task Closed tswiton
        98.
        		 Use a logger to log this exception. Sub-task Closed adamwudzinski
        99.
        		 Reduce code coupling in AAIDmaapEventJMSProducer Sub-task Closed bogumil_zebek
        100.
        		 Refactor of code smells in NoOpTinkerPopTransaction Sub-task Closed wiktor.garbarek
        101.
        		 Refactor of code smells in KafkaEventConsumer Sub-task Closed wiktor.garbarek
        102.
        		 Refactor of code smells in KafkaEventPublisher Sub-task Closed wiktor.garbarek
        103.
        		 Test coverage in GeneratorUtil Babel Sub-task Closed adamwudzinski
        104.
        		 Fix java code convention violation in traversal Sub-task Closed adamwudzinski
        105.
        		 Remove unusued imports in EdgeIngestor Sub-task Closed wiktor.garbarek
        106.
        		 Fix bugs in Rest Client Sub-task Closed wiktor.garbarek
        107.
        		 Fix bugs in Resources Sub-task Closed wiktor.garbarek
        108.
        		 Fix minor clumsy sonar issues in traversal Sub-task Closed adamwudzinski
        109.
        		 Fix handling of InterruptedException in Champ Sub-task Closed wiktor.garbarek
        110.
        		 Fix sonar issues Sub-task Closed pars
        111.
        		 [SONAR] Fix "NullPointerException" violation Sub-task Closed bogumil_zebek
        112.
        		 [SONAR] Fix stream closing Sub-task Closed bogumil_zebek
        113.
        		 Remove broken, duplicated AbstractLoggingChampGraph class Sub-task Closed bogumil_zebek
        114.
        		 OxmModelLoader code refactor + add missing tests Sub-task Closed bogumil_zebek
        115.
        		 [DataRouterProperties] File stream was not closed properly Sub-task Closed bogumil_zebek
        116.
        		 Fix sonar violation -> This class overrides "equals()" and should therefore also override "hashCode()" Sub-task Closed bogumil_zebek
        117.
        		 Fix sonar violation "Make the enclosing method "static" or remove this set." Sub-task Closed bogumil_zebek
        118.
        		 [aai-sparky-be] Fix potential "NullPointerException" cases Sub-task Closed bogumil_zebek
        119.
        		 [aai-spark-be] Restore interrupted state Sub-task Closed bogumil_zebek
        120.
        		 [aai-traversal] Fix potential NullPointerExceptions Sub-task Closed bogumil_zebek
        121.
        		 [aai-champ] Fix sonar violation "condition always evaluate to true" Sub-task Closed bogumil_zebek
        122.
        		 [aai- champ] Fix potential null pointer places Sub-task Closed bogumil_zebek
        123.
        		 [aai-common] refactor of DataGrooming Sub-task Closed jczestochowska
        124.
        		 [aai-common] remove code smells from ManageJanusGraphSchema Sub-task Closed jczestochowska
        125.
        		 [aai-data-router] Fix class serialization Sub-task Closed bogumil_zebek
        126.
        		 [aai-common] Fix sonar issues in SchemaGenerator Sub-task Closed jczestochowska
        127.
        		 [aai-spark-be] Refactor synchronizer classes: remove duplicated performRetrySync Sub-task Closed bogumil_zebek
        128.
        		 [aai-aai-common] Improve code quality Sub-task Closed bogumil_zebek
        129.
        		 [aai-common/aai-core] Fix sonar violation Sub-task Closed arindamm
        130.
        		 Fix Blocker sonar issues in AAIGraph.java Sub-task Closed amareshkumar
        131.
        		 Fix sonar issues in sparky, search service, data-router and router-core Sub-task Closed arul.nambi
        132.
        		 Sonar Critical: Use Logger and log the exception Sub-task Closed shubhada
        133.
        		 Sonar fixes - Remove this useless assignment to local variable Sub-task Closed driptaroopibm
        134.
        		 Sonar: Remove stacktrace and add error logger Sub-task Closed cghosh12
        135.
        		 Sonar AAIAuthCore.java - Define a constant instead of duplicating this literal "AAI_4001" 3 times. Sub-task Closed driptaroopibm
        136.
        		 Sonar - fix the list get call to retrieve correct value Sub-task Closed driptaroopibm
        137.
        		 Sonar Fixes - VnfSearchQueryBuilder.java Sub-task Closed driptaroopibm
        138.
        		 Sonar Remove stacktrace and add error logger Sub-task Closed cghosh12
        139.
        		 Sonar vulnerability reported Sub-task Closed cghosh12
        140.
        		 Sonar - Use try-with-resources or close this "MongoClient" in a "finally" clause. Sub-task Closed driptaroopibm
        141.
        		 Need to add exception log Sub-task Closed cghosh12
        142.
        		 Sonar Need to add nullcheck in graphadmin Sub-task Closed cghosh12
        143.
        		 Method should not have too many parameter AAI-Traversal Sub-task Closed arindamm
        144.
        		 Fix sonar issue in aai-data-router Sub-task Closed arindamm

            jimmydot jimmydot
            aishkashyap aishkashyap
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: