org.openpermis.editor.policy.view.RoleAssignmentRuleEditor
|
Rule |
Line |
It is somewhat confusing to have a field name with the same name as a method
|
AvoidFieldNameMatchingMethodName
|
100
|
Private field 'delegationDepthLabel' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
105
|
Private field 'absolutePeriodCheckBox' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
110
|
Private field 'startDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
115
|
Private field 'endDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
120
|
Private field 'referenceDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
125
|
Private field 'minimumFromCheckBox' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
130
|
Private field 'minimumFromDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
135
|
Private field 'minimumUpToCheckBox' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
145
|
Private field 'minimumUpToDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
150
|
Private field 'maximumUpToCheckBox' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
160
|
Private field 'maximumUpToDatePicker' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
165
|
Private field 'defaultZoneTimeStamp' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
175
|
org.openpermis.editor.policy.view.RoleHierarchyEditor
|
Rule |
Line |
Private field 'tree' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
85
|
Private field 'roleBrowser' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
90
|
Private field 'propertyChangeListener' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
95
|
Prefer StringBuffer over += for concatenating strings
|
UseStringBufferForStringAppends
|
137
|
Avoid reassigning parameters such as 'node'
|
AvoidReassigningParameters
|
342
|
Private field 'dummy4SelectionColors' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
474
|
Private field 'childrenTableModel' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
639
|
Private field 'childrenTable' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
644
|
Private field 'parentsList' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
649
|
Private field 'parentTableModel' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
654
|
Private field 'parentTable' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
659
|
Private field 'roleHierarchyEditor' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
669
|
Private field 'parent' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
749
|
Private field 'role' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
754
|
Private field 'subRoles' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
759
|
Private field 'hierarchy' could be made final; it is only initialized in the declaration or constructor.
|
ImmutableField
|
764
|
A getX() method which returns a boolean should be named isX()
|
BooleanGetMethodName
|
809
|