The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 48–58 |
| ClassNamingConventions | The utility class name 'AbstractAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 48–58 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 47–57 |
| ClassNamingConventions | The utility class name 'AbstractAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 47–57 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 45–55 |
| ClassNamingConventions | The utility class name 'AbstractAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 45–55 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 49 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| ClassNamingConventions | The class name 'CategoryAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'CategoryAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| ClassNamingConventions | The class name 'CategoryAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'OfferAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'OfferAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 43–46 |
| ClassNamingConventions | The class name 'OfferAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 43–46 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| ClassNamingConventions | The class name 'OfferGenreAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| ClassNamingConventions | The class name 'OfferGenreAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'OfferGenreAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–47 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'ProductAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'ProductAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'ProductAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'ProviderAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| ClassNamingConventions | The class name 'ProviderAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| ClassNamingConventions | The class name 'ProviderAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| ClassNamingConventions | The class name 'PurchaseAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| ClassNamingConventions | The class name 'PurchaseAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The class name 'PurchaseAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 48–63 |
| ClassNamingConventions | The utility class name 'Category_' doesn't match '[A-Z][a-zA-Z0-9]*' | 48–63 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The static field name 'hashCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The static field name 'parent' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The static field name 'provider' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The static field name 'childrenList' doesn't match '_[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 60 |
| FieldNamingConventions | The static field name 'productSet' doesn't match '_[a-z][a-zA-Z0-9]*' | 61 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'OfferGenre_' doesn't match '[A-Z][a-zA-Z0-9]*' | 44–54 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 44–54 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 47 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 49 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'offersList' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 46–62 |
| ClassNamingConventions | The utility class name 'Offer_' doesn't match '[A-Z][a-zA-Z0-9]*' | 46–62 |
| FieldNamingConventions | The static field name 'product' doesn't match '_[a-z][a-zA-Z0-9]*' | 49 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'statut' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The static field name 'quantity' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The static field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The static field name 'minTickets' doesn't match '_[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The static field name 'maxTickets' doesn't match '_[a-z][a-zA-Z0-9]*' | 60 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 47–60 |
| ClassNamingConventions | The utility class name 'Product_' doesn't match '[A-Z][a-zA-Z0-9]*' | 47–60 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'category' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'price' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The static field name 'provider' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Line |
|---|---|---|
| ConstructorCallsOverridableMethod | Overridable method 'setId' called during object construction | 111 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Provider_' doesn't match '[A-Z][a-zA-Z0-9]*' | 47–62 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 47–62 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'mail' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'phoneNumber' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'address' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The static field name 'contactName' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The static field name 'comment' doesn't match '_[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The static field name 'products' doesn't match '_[a-z][a-zA-Z0-9]*' | 60 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Purchase_' doesn't match '[A-Z][a-zA-Z0-9]*' | 46–57 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 46–57 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 49 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The static field name 'userName' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The static field name 'quantity' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The static field name 'offer' doesn't match '_[a-z][a-zA-Z0-9]*' | 54 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_purchaseId' doesn't match '[a-z][a-zA-Z0-9]*' | 45 |
| Rule | Violation | Line |
|---|---|---|
| ConstructorCallsOverridableMethod | Overridable method 'setQuantityRequested' called during object construction | 79 |
| ConstructorCallsOverridableMethod | Overridable method 'setQuantityAvailable' called during object construction | 80 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_format' doesn't match '[a-z][a-zA-Z0-9]*' | 67 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'StockConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–75 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 56 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 48 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 49 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 59 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 60 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 61 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 47 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 48 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 49 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 49 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 59 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 60 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 58 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 59 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 60 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 49 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 55 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 289–290 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–69 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeMap' | 62 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeDateMap' | 65 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeNumMap' | 68 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 64–67 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 60–63 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 48–58 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 54 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 64–67 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 47–57 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 45–55 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 94–97 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 77–82 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 98–101 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 74–116 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 91 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 99–102 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 76–81 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 97–100 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 74–79 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 95–98 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 74–114 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 94–97 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 74–79 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 95–98 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 75–119 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 94–97 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 75–81 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 97–100 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 75–119 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 72–77 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 93–96 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 75–80 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 75–119 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 94–97 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _owner field if you want a default access modifier | 62 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 62 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 75–80 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 96–99 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 74–118 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 82–87 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 95–98 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'Category' is suspected to be a Data Class (WOC=25.000%, NOPA=0, NOAM=7, WMC=12) | 68–369 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 128–136 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 147–150 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 158–161 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 306–309 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 330–333 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 354–357 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 61–412 |
| MissingOverride | The method 'selectAllFirstLevelWithChildrenWithProduct()' is missing an @Override annotation. | 78–95 |
| ShortVariable | Avoid variables with short names like em | 80 |
| ShortVariable | Avoid variables with short names like cb | 81 |
| ShortVariable | Avoid variables with short names like cq | 83 |
| MissingOverride | The method 'selectAllChildrenWithChildrenWithProduct(Integer)' is missing an @Override annotation. | 101–118 |
| ShortVariable | Avoid variables with short names like em | 103 |
| ShortVariable | Avoid variables with short names like cb | 104 |
| ShortVariable | Avoid variables with short names like cq | 106 |
| MissingOverride | The method 'findByIdWithChildren(Integer)' is missing an @Override annotation. | 124–147 |
| ShortVariable | Avoid variables with short names like em | 126 |
| ShortVariable | Avoid variables with short names like cb | 127 |
| ShortVariable | Avoid variables with short names like cq | 129 |
| MissingOverride | The method 'findByIdWithProduct(Integer)' is missing an @Override annotation. | 153–176 |
| ShortVariable | Avoid variables with short names like em | 155 |
| ShortVariable | Avoid variables with short names like cb | 156 |
| ShortVariable | Avoid variables with short names like cq | 158 |
| MissingOverride | The method 'findByIdWithParent(Integer)' is missing an @Override annotation. | 182–195 |
| MissingOverride | The method 'findByFilter(CategoryFilter, PaginationProperties)' is missing an @Override annotation. | 200–213 |
| ShortVariable | Avoid variables with short names like em | 202 |
| ShortVariable | Avoid variables with short names like cb | 203 |
| ShortVariable | Avoid variables with short names like cq | 205 |
| MissingOverride | The method 'findByFilterWithChildren(CategoryFilter)' is missing an @Override annotation. | 222–240 |
| ShortVariable | Avoid variables with short names like em | 224 |
| ShortVariable | Avoid variables with short names like cb | 225 |
| ShortVariable | Avoid variables with short names like cq | 227 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 289 |
| MissingOverride | The method 'findByFilter(CategoryFilter)' is missing an @Override annotation. | 339–355 |
| ShortVariable | Avoid variables with short names like em | 341 |
| ShortVariable | Avoid variables with short names like cb | 342 |
| ShortVariable | Avoid variables with short names like cq | 344 |
| MissingOverride | The method 'getAllByName(String)' is missing an @Override annotation. | 360–386 |
| ShortVariable | Avoid variables with short names like em | 362 |
| ShortVariable | Avoid variables with short names like cb | 363 |
| ShortVariable | Avoid variables with short names like cq | 365 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 379 |
| MissingOverride | The method 'findAll(List)' is missing an @Override annotation. | 395–411 |
| ShortVariable | Avoid variables with short names like em | 397 |
| ShortVariable | Avoid variables with short names like cb | 398 |
| ShortVariable | Avoid variables with short names like cq | 400 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 84–87 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 95–98 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 105–108 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 116–119 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 48–63 |
| CommentRequired | Header comments are required | 48–63 |
| ShortVariable | Avoid variables with short names like id | 51 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 50 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 48 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 134–136 |
| UncommentedEmptyConstructor | Document empty constructor | 134–136 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'category'. | 134 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 147–150 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 158–161 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 327–330 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 351–354 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 375–378 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findByFilter(OfferFilter, PaginationProperties)' is missing an @Override annotation. | 89–102 |
| ShortVariable | Avoid variables with short names like em | 91 |
| ShortVariable | Avoid variables with short names like cb | 92 |
| ShortVariable | Avoid variables with short names like cq | 94 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 146 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 178 |
| AvoidDuplicateLiterals | The String literal "name" appears 4 times in this file; the first occurrence is on line 180 | 180 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 183 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 198 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 203 |
| MissingOverride | The method 'findByProduct(Integer, OfferFilter)' is missing an @Override annotation. | 225–242 |
| ShortVariable | Avoid variables with short names like em | 228 |
| ShortVariable | Avoid variables with short names like cb | 229 |
| ShortVariable | Avoid variables with short names like cq | 231 |
| MissingOverride | The method 'getQuantity(Integer)' is missing an @Override annotation. | 252–258 |
| ShortVariable | Avoid variables with short names like em | 254 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'OfferFilter' is suspected to be a Data Class (WOC=12.500%, NOPA=0, NOAM=14, WMC=16) | 44–268 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 111–114 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 122–125 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 160–163 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 187–190 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 214–217 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findWithOffersLinked(int)' is missing an @Override annotation. | 66–82 |
| ShortVariable | Avoid variables with short names like em | 68 |
| ShortVariable | Avoid variables with short names like cb | 69 |
| ShortVariable | Avoid variables with short names like cq | 71 |
| MissingOverride | The method 'findAllReferenceList()' is missing an @Override annotation. | 87–101 |
| MissingOverride | The method 'findByName(String)' is missing an @Override annotation. | 106–122 |
| ShortVariable | Avoid variables with short names like em | 108 |
| ShortVariable | Avoid variables with short names like cb | 109 |
| ShortVariable | Avoid variables with short names like cq | 111 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 44–54 |
| CommentRequired | Header comments are required | 44–54 |
| ShortVariable | Avoid variables with short names like id | 47 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–62 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 46–62 |
| ShortVariable | Avoid variables with short names like id | 50 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 113 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'Product' is suspected to be a Data Class (WOC=27.273%, NOPA=0, NOAM=6, WMC=11) | 68–346 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 128–135 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 146–149 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 157–160 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 282–285 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 306–309 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 330–333 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findByFilter(ProductFilter)' is missing an @Override annotation. | 95–111 |
| ShortVariable | Avoid variables with short names like em | 97 |
| ShortVariable | Avoid variables with short names like cb | 98 |
| ShortVariable | Avoid variables with short names like cq | 100 |
| MissingOverride | The method 'findByFilter(ProductFilter, PaginationProperties)' is missing an @Override annotation. | 122–135 |
| ShortVariable | Avoid variables with short names like em | 124 |
| ShortVariable | Avoid variables with short names like cb | 125 |
| ShortVariable | Avoid variables with short names like cq | 127 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 196 |
| MissingOverride | The method 'getAllByName(String)' is missing an @Override annotation. | 247–273 |
| ShortVariable | Avoid variables with short names like em | 249 |
| ShortVariable | Avoid variables with short names like cb | 250 |
| ShortVariable | Avoid variables with short names like cq | 252 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 266 |
| MissingOverride | The method 'getCountProductALAfficheByDate(String)' is missing an @Override annotation. | 278–308 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 508 characters appended. | 281 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 283 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 283 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 284 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 7 consecutive times with literals. Use a single append with a single combined String. | 285 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 285 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 286 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 287 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 288 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 289 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 290 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 291 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 291 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 292 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 298 |
| MissingOverride | The method 'getCountProductAVenirByDate(String)' is missing an @Override annotation. | 313–336 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 229 characters appended. | 316 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 318 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 318 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 319 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 320 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 320 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 326 |
| MissingOverride | The method 'isFull(Integer)' is missing an @Override annotation. | 346–362 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 382 characters appended. | 349 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 351 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 351 |
| InefficientStringBuffering | Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 352 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 352 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 4 consecutive times with literals. Use a single append with a single combined String. | 353 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 353 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 354 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 355 |
| MissingOverride | The method 'isType(Integer, Integer)' is missing an @Override annotation. | 371–379 |
| ShortVariable | Avoid variables with short names like em | 373 |
| CommentRequired | Public method and constructor comments are required | 381–392 |
| MissingOverride | The method 'isTypeOffer(Integer, Integer, String, Timestamp, String)' is missing an @Override annotation. | 381–392 |
| ShortVariable | Avoid variables with short names like em | 383 |
| MissingOverride | The method 'getProductsIdsForTaskTimed(String, Timestamp, Timestamp)' is missing an @Override annotation. | 405–414 |
| ShortVariable | Avoid variables with short names like em | 407 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'ProductFilter' is suspected to be a Data Class (WOC=23.810%, NOPA=0, NOAM=16, WMC=21) | 46–262 |
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 127–130 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 138–141 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 148–151 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 159–162 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getImage(Integer)' is missing an @Override annotation. | 92–97 |
| MissingOverride | The method 'getTbImage(Integer)' is missing an @Override annotation. | 102–107 |
| MissingOverride | The method 'remove(Integer)' is missing an @Override annotation. | 120–125 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–60 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 47–60 |
| ShortVariable | Avoid variables with short names like id | 50 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 93–98 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 106–112 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 141–144 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 152–155 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 317–320 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 335–338 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 359–362 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findAllWithProducts(PaginationProperties)' is missing an @Override annotation. | 77–91 |
| ShortVariable | Avoid variables with short names like em | 79 |
| ShortVariable | Avoid variables with short names like cb | 80 |
| ShortVariable | Avoid variables with short names like cq | 82 |
| MissingOverride | The method 'findByIdWithProducts(int)' is missing an @Override annotation. | 100–115 |
| ShortVariable | Avoid variables with short names like em | 102 |
| ShortVariable | Avoid variables with short names like cb | 103 |
| ShortVariable | Avoid variables with short names like cq | 105 |
| MissingOverride | The method 'findByFilter(ProviderFilter, PaginationProperties)' is missing an @Override annotation. | 126–139 |
| ShortVariable | Avoid variables with short names like em | 128 |
| ShortVariable | Avoid variables with short names like cb | 129 |
| ShortVariable | Avoid variables with short names like cq | 131 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 191 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 191–194 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 202–205 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 212–215 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 223–226 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–62 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 47–62 |
| ShortVariable | Avoid variables with short names like id | 50 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 68 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 109–112 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 120–123 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 200–203 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 224–227 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 248–251 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findByFilter(PurchaseFilter, PaginationProperties)' is missing an @Override annotation. | 92–105 |
| ShortVariable | Avoid variables with short names like em | 94 |
| ShortVariable | Avoid variables with short names like cb | 95 |
| ShortVariable | Avoid variables with short names like cq | 97 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 151 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 184 |
| AvoidDuplicateLiterals | The String literal "name" appears 4 times in this file; the first occurrence is on line 186 | 186 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 189 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 204 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 209 |
| MissingOverride | The method 'getQuantityPurchasedByIdProductAndUserName(Integer, Integer, String)' is missing an @Override annotation. | 226–249 |
| ShortVariable | Avoid variables with short names like em | 228 |
| ShortVariable | Avoid variables with short names like cb | 229 |
| ShortVariable | Avoid variables with short names like cq | 231 |
| MissingOverride | The method 'getCountPurchaseByBeginDateAndLastDate(String, String)' is missing an @Override annotation. | 259–283 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 277 characters appended. | 262 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 264 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 264 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 265 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 266 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 266 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 267 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 267 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 273 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'PurchaseFilter' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=20, WMC=24) | 45–327 |
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 155–158 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 166–169 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 176–179 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 187–190 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–57 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 46–57 |
| ShortVariable | Avoid variables with short names like id | 49 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'PurchaseOutOfStock' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) | 39–124 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–54 |
| ShortVariable | Avoid variables with short names like id | 49 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 51 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 78 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'findByProduct(Integer)' is missing an @Override annotation. | 65–68 |
| MissingOverride | The method 'getQuantity(Integer)' is missing an @Override annotation. | 78–81 |
| MissingOverride | The method 'findById(Integer)' is missing an @Override annotation. | 91–94 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getAllProduct()' is missing an @Override annotation. | 63–66 |
| MissingOverride | The method 'isFull(Integer)' is missing an @Override annotation. | 75–78 |
| MissingOverride | The method 'isType(Integer, Integer)' is missing an @Override annotation. | 87–90 |
| MissingOverride | The method 'isTypeOffer(Integer, Integer, String, Timestamp, String)' is missing an @Override annotation. | 92–95 |
| CommentRequired | Public method and constructor comments are required | 92–95 |
| MissingOverride | The method 'findByFilter(ProductFilter)' is missing an @Override annotation. | 104–107 |
| MissingOverride | The method 'getProductsIdsForTaskTimed(String, Timestamp, Timestamp)' is missing an @Override annotation. | 120–123 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'checkBeforePurchase(IPurchaseDTO, String)' is missing an @Override annotation. | 72–88 |
| Rule | Violation | Line |
|---|---|---|
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 67 |
| ImmutableField | Private field '_idleQuantity' could be made final; it is only initialized in the declaration or constructor. | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 81 |
| ImmutableField | Private field '_activePurchaseBySession' could be made final; it is only initialized in the declaration or constructor. | 81 |
| MissingOverride | The method 'reserve(String, IPurchaseDTO)' is missing an @Override annotation. | 91–121 |
| MissingOverride | The method 'checkReserved(String, IPurchaseDTO)' is missing an @Override annotation. | 131–153 |
| MissingOverride | The method 'release(String, IPurchaseDTO)' is missing an @Override annotation. | 163–183 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 163–183 |
| MissingOverride | The method 'releaseAll(String)' is missing an @Override annotation. | 193–210 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 193–210 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'register()' is missing an @Override annotation. | 79–118 |
| ShortVariable | Avoid variables with short names like rt | 81 |
| ShortVariable | Avoid variables with short names like p | 86 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 127–130 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 141–144 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 127–143 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 131–138 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 133 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 157–164 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'sessionDestroyed(HttpSessionEvent)' is missing an @Override annotation. | 57–67 |
| ShortVariable | Avoid variables with short names like se | 57 |
| MissingOverride | The method 'sessionCreated(HttpSessionEvent)' is missing an @Override annotation. | 75–78 |
| ShortVariable | Avoid variables with short names like se | 75 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.BooleanUtils' | 39 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–69 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeMap' | 3 | 62 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeDateMap' | 3 | 65 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_attributeNumMap' | 3 | 68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 64–67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 60–63 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 48–58 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 48–58 |
| ClassNamingConventions | The utility class name 'AbstractAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 48–58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 56 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 54 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 64–67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 47–57 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 47–57 |
| ClassNamingConventions | The utility class name 'AbstractAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 47–57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 45–55 |
| ClassNamingConventions | The utility class name 'AbstractAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 45–55 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 45–55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 48 |
| FieldNamingConventions | The static field name 'owner' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| FieldNamingConventions | The static field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 49 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 49 |
| FieldNamingConventions | The static field name 'key' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 94–97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 77–82 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 98–101 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| ClassNamingConventions | The class name 'CategoryAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 74–116 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 91 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 99–102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'CategoryAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| ClassNamingConventions | The class name 'CategoryAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 76–81 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 97–100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 74–79 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 95–98 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'OfferAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 74–114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'OfferAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 43–46 |
| ClassNamingConventions | The class name 'OfferAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 43–46 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 94–97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 74–79 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 95–98 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| ClassNamingConventions | The class name 'OfferGenreAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 75–119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| ClassNamingConventions | The class name 'OfferGenreAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'OfferGenreAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 94–97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 75–81 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 97–100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'ProductAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| CommentRequired | Header comments are required | 3 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 75–119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| ClassNamingConventions | The class name 'ProductAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The class name 'ProductAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| CommentRequired | Header comments are required | 3 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 72–77 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 93–96 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 75–80 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| ClassNamingConventions | The class name 'ProviderAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 75–119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 83–88 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| ClassNamingConventions | The class name 'ProviderAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| CommentRequired | Header comments are required | 3 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| ClassNamingConventions | The class name 'ProviderAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| CommentRequired | Header comments are required | 3 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 55 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–78 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 94–97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _owner field if you want a default access modifier | 3 | 62 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 62 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 75–80 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 96–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| ClassNamingConventions | The class name 'PurchaseAttributeDate_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'equals(Object)' has a cyclomatic complexity of 10. | 3 | 74–118 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 57 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 82–87 |
| MissingOverride | The method 'getOwner()' is missing an @Override annotation. | 3 | 95–98 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| ClassNamingConventions | The class name 'PurchaseAttributeNum_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| ClassNamingConventions | The class name 'PurchaseAttribute_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 39–42 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'Category' is suspected to be a Data Class (WOC=25.000%, NOPA=0, NOAM=7, WMC=12) | 3 | 68–369 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 128–136 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 147–150 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 158–161 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 306–309 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 330–333 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 354–357 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 61–412 |
| MissingOverride | The method 'selectAllFirstLevelWithChildrenWithProduct()' is missing an @Override annotation. | 3 | 78–95 |
| ShortVariable | Avoid variables with short names like em | 3 | 80 |
| ShortVariable | Avoid variables with short names like cb | 3 | 81 |
| ShortVariable | Avoid variables with short names like cq | 3 | 83 |
| MissingOverride | The method 'selectAllChildrenWithChildrenWithProduct(Integer)' is missing an @Override annotation. | 3 | 101–118 |
| ShortVariable | Avoid variables with short names like em | 3 | 103 |
| ShortVariable | Avoid variables with short names like cb | 3 | 104 |
| ShortVariable | Avoid variables with short names like cq | 3 | 106 |
| MissingOverride | The method 'findByIdWithChildren(Integer)' is missing an @Override annotation. | 3 | 124–147 |
| ShortVariable | Avoid variables with short names like em | 3 | 126 |
| ShortVariable | Avoid variables with short names like cb | 3 | 127 |
| ShortVariable | Avoid variables with short names like cq | 3 | 129 |
| MissingOverride | The method 'findByIdWithProduct(Integer)' is missing an @Override annotation. | 3 | 153–176 |
| ShortVariable | Avoid variables with short names like em | 3 | 155 |
| ShortVariable | Avoid variables with short names like cb | 3 | 156 |
| ShortVariable | Avoid variables with short names like cq | 3 | 158 |
| MissingOverride | The method 'findByIdWithParent(Integer)' is missing an @Override annotation. | 3 | 182–195 |
| MissingOverride | The method 'findByFilter(CategoryFilter, PaginationProperties)' is missing an @Override annotation. | 3 | 200–213 |
| ShortVariable | Avoid variables with short names like em | 3 | 202 |
| ShortVariable | Avoid variables with short names like cb | 3 | 203 |
| ShortVariable | Avoid variables with short names like cq | 3 | 205 |
| MissingOverride | The method 'findByFilterWithChildren(CategoryFilter)' is missing an @Override annotation. | 3 | 222–240 |
| ShortVariable | Avoid variables with short names like em | 3 | 224 |
| ShortVariable | Avoid variables with short names like cb | 3 | 225 |
| ShortVariable | Avoid variables with short names like cq | 3 | 227 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 289 |
| MissingOverride | The method 'findByFilter(CategoryFilter)' is missing an @Override annotation. | 3 | 339–355 |
| ShortVariable | Avoid variables with short names like em | 3 | 341 |
| ShortVariable | Avoid variables with short names like cb | 3 | 342 |
| ShortVariable | Avoid variables with short names like cq | 3 | 344 |
| MissingOverride | The method 'getAllByName(String)' is missing an @Override annotation. | 3 | 360–386 |
| ShortVariable | Avoid variables with short names like em | 3 | 362 |
| ShortVariable | Avoid variables with short names like cb | 3 | 363 |
| ShortVariable | Avoid variables with short names like cq | 3 | 365 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 379 |
| MissingOverride | The method 'findAll(List)' is missing an @Override annotation. | 3 | 395–411 |
| ShortVariable | Avoid variables with short names like em | 3 | 397 |
| ShortVariable | Avoid variables with short names like cb | 3 | 398 |
| ShortVariable | Avoid variables with short names like cq | 3 | 400 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.BooleanUtils' | 4 | 39 |
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 3 | 84–87 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 3 | 95–98 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 3 | 105–108 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 3 | 116–119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 48–63 |
| CommentRequired | Header comments are required | 3 | 48–63 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 48–63 |
| ClassNamingConventions | The utility class name 'Category_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 48–63 |
| ShortVariable | Avoid variables with short names like id | 3 | 51 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 54 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 56 |
| FieldNamingConventions | The static field name 'hashCode' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 57 |
| FieldNamingConventions | The static field name 'parent' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 58 |
| FieldNamingConventions | The static field name 'provider' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 59 |
| FieldNamingConventions | The static field name 'childrenList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 60 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 60 |
| FieldNamingConventions | The static field name 'productSet' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 61 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 61 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 50 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 134–136 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 134–136 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'category'. | 3 | 134 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 147–150 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 158–161 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 327–330 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 351–354 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 375–378 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findByFilter(OfferFilter, PaginationProperties)' is missing an @Override annotation. | 3 | 89–102 |
| ShortVariable | Avoid variables with short names like em | 3 | 91 |
| ShortVariable | Avoid variables with short names like cb | 3 | 92 |
| ShortVariable | Avoid variables with short names like cq | 3 | 94 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 146 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 178 |
| AvoidDuplicateLiterals | The String literal "name" appears 4 times in this file; the first occurrence is on line 180 | 3 | 180 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 183 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 198 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 203 |
| MissingOverride | The method 'findByProduct(Integer, OfferFilter)' is missing an @Override annotation. | 3 | 225–242 |
| ShortVariable | Avoid variables with short names like em | 3 | 228 |
| ShortVariable | Avoid variables with short names like cb | 3 | 229 |
| ShortVariable | Avoid variables with short names like cq | 3 | 231 |
| MissingOverride | The method 'getQuantity(Integer)' is missing an @Override annotation. | 3 | 252–258 |
| ShortVariable | Avoid variables with short names like em | 3 | 254 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'OfferFilter' is suspected to be a Data Class (WOC=12.500%, NOPA=0, NOAM=14, WMC=16) | 3 | 44–268 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 111–114 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 122–125 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 160–163 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 187–190 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 214–217 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findWithOffersLinked(int)' is missing an @Override annotation. | 3 | 66–82 |
| ShortVariable | Avoid variables with short names like em | 3 | 68 |
| ShortVariable | Avoid variables with short names like cb | 3 | 69 |
| ShortVariable | Avoid variables with short names like cq | 3 | 71 |
| MissingOverride | The method 'findAllReferenceList()' is missing an @Override annotation. | 3 | 87–101 |
| MissingOverride | The method 'findByName(String)' is missing an @Override annotation. | 3 | 106–122 |
| ShortVariable | Avoid variables with short names like em | 3 | 108 |
| ShortVariable | Avoid variables with short names like cb | 3 | 109 |
| ShortVariable | Avoid variables with short names like cq | 3 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 44–54 |
| CommentRequired | Header comments are required | 3 | 44–54 |
| ClassNamingConventions | The utility class name 'OfferGenre_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 44–54 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 44–54 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 47 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 47 |
| ShortVariable | Avoid variables with short names like id | 3 | 47 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 48 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 49 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 49 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'offersList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 46–62 |
| CommentRequired | Header comments are required | 3 | 46–62 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 46–62 |
| ClassNamingConventions | The utility class name 'Offer_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 46–62 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 49 |
| FieldNamingConventions | The static field name 'product' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 49 |
| ShortVariable | Avoid variables with short names like id | 3 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'statut' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 54 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 56 |
| FieldNamingConventions | The static field name 'quantity' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 57 |
| FieldNamingConventions | The static field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 59 |
| FieldNamingConventions | The static field name 'minTickets' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FieldNamingConventions | The static field name 'maxTickets' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 60 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 60 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 113 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'Product' is suspected to be a Data Class (WOC=27.273%, NOPA=0, NOAM=6, WMC=11) | 3 | 68–346 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 128–135 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 146–149 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 157–160 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 282–285 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 306–309 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 330–333 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findByFilter(ProductFilter)' is missing an @Override annotation. | 3 | 95–111 |
| ShortVariable | Avoid variables with short names like em | 3 | 97 |
| ShortVariable | Avoid variables with short names like cb | 3 | 98 |
| ShortVariable | Avoid variables with short names like cq | 3 | 100 |
| MissingOverride | The method 'findByFilter(ProductFilter, PaginationProperties)' is missing an @Override annotation. | 3 | 122–135 |
| ShortVariable | Avoid variables with short names like em | 3 | 124 |
| ShortVariable | Avoid variables with short names like cb | 3 | 125 |
| ShortVariable | Avoid variables with short names like cq | 3 | 127 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 196 |
| MissingOverride | The method 'getAllByName(String)' is missing an @Override annotation. | 3 | 247–273 |
| ShortVariable | Avoid variables with short names like em | 3 | 249 |
| ShortVariable | Avoid variables with short names like cb | 3 | 250 |
| ShortVariable | Avoid variables with short names like cq | 3 | 252 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 266 |
| MissingOverride | The method 'getCountProductALAfficheByDate(String)' is missing an @Override annotation. | 3 | 278–308 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 508 characters appended. | 3 | 281 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 283 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 283 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 284 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 7 consecutive times with literals. Use a single append with a single combined String. | 3 | 285 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 285 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 286 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 287 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 288 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 289 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 290 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 291 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 291 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 292 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 298 |
| MissingOverride | The method 'getCountProductAVenirByDate(String)' is missing an @Override annotation. | 3 | 313–336 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 229 characters appended. | 3 | 316 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 318 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 318 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 319 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 320 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 320 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 326 |
| MissingOverride | The method 'isFull(Integer)' is missing an @Override annotation. | 3 | 346–362 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 382 characters appended. | 3 | 349 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 351 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 351 |
| InefficientStringBuffering | Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 3 | 352 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 352 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 4 consecutive times with literals. Use a single append with a single combined String. | 3 | 353 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 353 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 354 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 355 |
| MissingOverride | The method 'isType(Integer, Integer)' is missing an @Override annotation. | 3 | 371–379 |
| ShortVariable | Avoid variables with short names like em | 3 | 373 |
| CommentRequired | Public method and constructor comments are required | 3 | 381–392 |
| MissingOverride | The method 'isTypeOffer(Integer, Integer, String, Timestamp, String)' is missing an @Override annotation. | 3 | 381–392 |
| ShortVariable | Avoid variables with short names like em | 3 | 383 |
| MissingOverride | The method 'getProductsIdsForTaskTimed(String, Timestamp, Timestamp)' is missing an @Override annotation. | 3 | 405–414 |
| ShortVariable | Avoid variables with short names like em | 3 | 407 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'ProductFilter' is suspected to be a Data Class (WOC=23.810%, NOPA=0, NOAM=16, WMC=21) | 3 | 46–262 |
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 3 | 127–130 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 3 | 138–141 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 3 | 148–151 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 3 | 159–162 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getImage(Integer)' is missing an @Override annotation. | 3 | 92–97 |
| MissingOverride | The method 'getTbImage(Integer)' is missing an @Override annotation. | 3 | 102–107 |
| MissingOverride | The method 'remove(Integer)' is missing an @Override annotation. | 3 | 120–125 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–60 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 47–60 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 47–60 |
| ClassNamingConventions | The utility class name 'Product_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 47–60 |
| ShortVariable | Avoid variables with short names like id | 3 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'category' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| FieldNamingConventions | The static field name 'price' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 54 |
| FieldNamingConventions | The static field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 56 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 57 |
| FieldNamingConventions | The static field name 'provider' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 58 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 93–98 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 106–112 |
| ConstructorCallsOverridableMethod | Overridable method 'setId' called during object construction | 1 | 111 |
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 141–144 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 152–155 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 317–320 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 335–338 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 359–362 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findAllWithProducts(PaginationProperties)' is missing an @Override annotation. | 3 | 77–91 |
| ShortVariable | Avoid variables with short names like em | 3 | 79 |
| ShortVariable | Avoid variables with short names like cb | 3 | 80 |
| ShortVariable | Avoid variables with short names like cq | 3 | 82 |
| MissingOverride | The method 'findByIdWithProducts(int)' is missing an @Override annotation. | 3 | 100–115 |
| ShortVariable | Avoid variables with short names like em | 3 | 102 |
| ShortVariable | Avoid variables with short names like cb | 3 | 103 |
| ShortVariable | Avoid variables with short names like cq | 3 | 105 |
| MissingOverride | The method 'findByFilter(ProviderFilter, PaginationProperties)' is missing an @Override annotation. | 3 | 126–139 |
| ShortVariable | Avoid variables with short names like em | 3 | 128 |
| ShortVariable | Avoid variables with short names like cb | 3 | 129 |
| ShortVariable | Avoid variables with short names like cq | 3 | 131 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 191 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 3 | 191–194 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 3 | 202–205 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 3 | 212–215 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 3 | 223–226 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–62 |
| ClassNamingConventions | The utility class name 'Provider_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 47–62 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 47–62 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 47–62 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| ShortVariable | Avoid variables with short names like id | 3 | 50 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| FieldNamingConventions | The static field name 'mail' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 54 |
| FieldNamingConventions | The static field name 'phoneNumber' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| FieldNamingConventions | The static field name 'address' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| FieldNamingConventions | The static field name 'contactName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 56 |
| FieldNamingConventions | The static field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 58 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 59 |
| FieldNamingConventions | The static field name 'comment' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FieldNamingConventions | The static field name 'products' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 60 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 60 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 109–112 |
| MissingOverride | The method 'setId(Integer)' is missing an @Override annotation. | 3 | 120–123 |
| MissingOverride | The method 'getAttributeList()' is missing an @Override annotation. | 3 | 200–203 |
| MissingOverride | The method 'getAttributeDateList()' is missing an @Override annotation. | 3 | 224–227 |
| MissingOverride | The method 'getAttributeNumList()' is missing an @Override annotation. | 3 | 248–251 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findByFilter(PurchaseFilter, PaginationProperties)' is missing an @Override annotation. | 3 | 92–105 |
| ShortVariable | Avoid variables with short names like em | 3 | 94 |
| ShortVariable | Avoid variables with short names like cb | 3 | 95 |
| ShortVariable | Avoid variables with short names like cq | 3 | 97 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 151 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 184 |
| AvoidDuplicateLiterals | The String literal "name" appears 4 times in this file; the first occurrence is on line 186 | 3 | 186 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 189 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 204 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 209 |
| MissingOverride | The method 'getQuantityPurchasedByIdProductAndUserName(Integer, Integer, String)' is missing an @Override annotation. | 3 | 226–249 |
| ShortVariable | Avoid variables with short names like em | 3 | 228 |
| ShortVariable | Avoid variables with short names like cb | 3 | 229 |
| ShortVariable | Avoid variables with short names like cq | 3 | 231 |
| MissingOverride | The method 'getCountPurchaseByBeginDateAndLastDate(String, String)' is missing an @Override annotation. | 3 | 259–283 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 277 characters appended. | 3 | 262 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 264 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 264 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 265 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 266 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 266 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 267 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 267 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 273 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'PurchaseFilter' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=20, WMC=24) | 3 | 45–327 |
| MissingOverride | The method 'isOrderAsc()' is missing an @Override annotation. | 3 | 155–158 |
| MissingOverride | The method 'setOrderAsc(boolean)' is missing an @Override annotation. | 3 | 166–169 |
| MissingOverride | The method 'getOrders()' is missing an @Override annotation. | 3 | 176–179 |
| MissingOverride | The method 'setOrders(List)' is missing an @Override annotation. | 3 | 187–190 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Purchase_' doesn't match '[A-Z][a-zA-Z0-9]*' | 1 | 46–57 |
| CommentRequired | Header comments are required | 3 | 46–57 |
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 46–57 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 46–57 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 49 |
| ShortVariable | Avoid variables with short names like id | 3 | 49 |
| FieldNamingConventions | The static field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 49 |
| FieldNamingConventions | The static field name 'attributeNumList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 50 |
| FieldNamingConventions | The static field name 'attributeList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 51 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 52 |
| FieldNamingConventions | The static field name 'userName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| FieldNamingConventions | The static field name 'quantity' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 53 |
| FieldNamingConventions | The static field name 'offer' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 54 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 54 |
| FieldNamingConventions | The static field name 'attributeDateList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_purchaseId' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'PurchaseOutOfStock' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) | 3 | 39–124 |
| ConstructorCallsOverridableMethod | Overridable method 'setQuantityRequested' called during object construction | 1 | 79 |
| ConstructorCallsOverridableMethod | Overridable method 'setQuantityAvailable' called during object construction | 1 | 80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–54 |
| ShortVariable | Avoid variables with short names like id | 3 | 49 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 51 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 78 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'findByProduct(Integer)' is missing an @Override annotation. | 3 | 65–68 |
| MissingOverride | The method 'getQuantity(Integer)' is missing an @Override annotation. | 3 | 78–81 |
| MissingOverride | The method 'findById(Integer)' is missing an @Override annotation. | 3 | 91–94 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getAllProduct()' is missing an @Override annotation. | 3 | 63–66 |
| MissingOverride | The method 'isFull(Integer)' is missing an @Override annotation. | 3 | 75–78 |
| MissingOverride | The method 'isType(Integer, Integer)' is missing an @Override annotation. | 3 | 87–90 |
| MissingOverride | The method 'isTypeOffer(Integer, Integer, String, Timestamp, String)' is missing an @Override annotation. | 3 | 92–95 |
| CommentRequired | Public method and constructor comments are required | 3 | 92–95 |
| MissingOverride | The method 'findByFilter(ProductFilter)' is missing an @Override annotation. | 3 | 104–107 |
| MissingOverride | The method 'getProductsIdsForTaskTimed(String, Timestamp, Timestamp)' is missing an @Override annotation. | 3 | 120–123 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'checkBeforePurchase(IPurchaseDTO, String)' is missing an @Override annotation. | 3 | 72–88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 67 |
| FieldNamingConventions | The final field name '_format' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 67 |
| ImmutableField | Private field '_idleQuantity' could be made final; it is only initialized in the declaration or constructor. | 3 | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 76 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 81 |
| ImmutableField | Private field '_activePurchaseBySession' could be made final; it is only initialized in the declaration or constructor. | 3 | 81 |
| MissingOverride | The method 'reserve(String, IPurchaseDTO)' is missing an @Override annotation. | 3 | 91–121 |
| MissingOverride | The method 'checkReserved(String, IPurchaseDTO)' is missing an @Override annotation. | 3 | 131–153 |
| MissingOverride | The method 'release(String, IPurchaseDTO)' is missing an @Override annotation. | 3 | 163–183 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 163–183 |
| MissingOverride | The method 'releaseAll(String)' is missing an @Override annotation. | 3 | 193–210 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 193–210 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 289–290 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 79–118 |
| ShortVariable | Avoid variables with short names like rt | 3 | 81 |
| ShortVariable | Avoid variables with short names like p | 3 | 86 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 127–130 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 141–144 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 127–143 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 131–138 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 133 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 157–164 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'StockConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–75 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'sessionDestroyed(HttpSessionEvent)' is missing an @Override annotation. | 3 | 57–67 |
| ShortVariable | Avoid variables with short names like se | 3 | 57 |
| MissingOverride | The method 'sessionCreated(HttpSessionEvent)' is missing an @Override annotation. | 3 | 75–78 |
| ShortVariable | Avoid variables with short names like se | 3 | 75 |
