| Violation |
Line |
| A high number of imports can indicate a high degree of coupling within an object. |
34–1007 |
| Possible God class (WMC=128, ATFD=258, TCC=0.0) |
34–1007 |
| The class 'NotifyDirectoryService' has a Modified Cyclomatic Complexity of 4 (Highest = 14). |
108–1007 |
| The class 'NotifyDirectoryService' has a Cyclomatic Complexity of 5 (Highest = 25). |
108–1007 |
| The class 'NotifyDirectoryService' has a Standard Cyclomatic Complexity of 4 (Highest = 14). |
108–1007 |
| Private field '_listAcceptedEntryTypesEmailSMS' could be made final; it is only initialized in the declaration or constructor. |
127 |
| Private field '_listAcceptedEntryTypesUserGuid' could be made final; it is only initialized in the declaration or constructor. |
128 |
| Private field '_listRefusedEntryTypes' could be made final; it is only initialized in the declaration or constructor. |
129 |
| Private field '_listAcceptedEntryTypesFile' could be made final; it is only initialized in the declaration or constructor. |
130 |
| The method 'sendMessage' has a Cyclomatic Complexity of 16. |
515–579 |
| Rather than using a lot of String arguments, consider using a container object for those values. |
515–516 |
| The method sendMessage() has an NPath complexity of 450 |
515–579 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
521 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
547 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
568 |
| Avoid really long methods. |
585–754 |
| The method 'fillModel' has a Cyclomatic Complexity of 25. |
585–754 |
| The method 'fillModel' has a Modified Cyclomatic Complexity of 14. |
585–754 |
| The method fillModel() has an NPath complexity of 4704 |
585–754 |
| The method 'fillModel' has a Standard Cyclomatic Complexity of 14. |
585–754 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
588 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' |
611 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' |
619 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
682 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
688 |
| Unnecessary use of fully qualified name 'java.util.UUID.randomUUID' due to existing import 'java.util.UUID' |
702 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
727 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
735 |
| The method 'getFiles' has a Cyclomatic Complexity of 12. |
838–892 |
| The method fillModelWithUserAttributes() has an NPath complexity of 627 |
935–954 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
968 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
969 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
970 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
971–972 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
973 |
