Showing 189 of 206 total issues
Method validatePasswordFields
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
private void validatePasswordFields(final HttpServletRequest request)
throws EPSUIException, SQLException {
String password1 = request.getParameter("password1");
String password2 = request.getParameter("password2");
if ((password1 == null || password1.isEmpty())
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method doPost
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
@Override
protected void doPost(final HttpServletRequest request, final HttpServletResponse response)
throws ServletException, IOException {
request.setAttribute("error_page", "/admin/PasswordRestrictionsAddStage1");
int specialCount;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method meetsCharacterBasedRequirements
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
private boolean meetsCharacterBasedRequirements(final String password) {
int special = 0, numeric = 0, upper = 0, lower = 0;
for (int i = 0; i < password.length(); i++) {
char thisChar = password.charAt(i);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method applyPermissions
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
private void applyPermissions(final User remoteUser, final HierarchyNodeDAO hnDAO, final HierarchyNode node,
final Map<String, PasswordPermission> uPerms,
final Map<String, PasswordPermission> gPerms,
final ChangePermissionsAction action)
Method doPost
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
@Override
protected void doPost(final HttpServletRequest request, final HttpServletResponse response)
throws IOException, ServletException {
String csrfToken = request.getParameter("token");
if(csrfToken == null
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method doGet
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
@Override
protected void doGet(final HttpServletRequest request, final HttpServletResponse response)
throws IOException {
String id = ServletUtils.getInstance().getParameterValue(request, SharedParameterNames.PASSWORD_ID_PARAMETER);
String dt = request.getParameter(BaseServlet.DATE_TIME_PARAMETER);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method getIPv6String
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
private String getIPv6String(HttpServletRequest request, String field) {
String value = request.getParameter(field);
if(!value.contains("::")) {
return mergeSections(value);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method doFilter
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
@Override
public void doFilter(ServletRequest request, ServletResponse response,
FilterChain next) throws IOException {
try {
next.doFilter(request, response);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method transferSettingToDatabase
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
private void transferSettingToDatabase(final HttpServletRequest request,
final ConfigurationDAO cDAO, TamperproofEventLogDAO telDAO,
final User thisUser, final String parameterName) throws SQLException,
GeneralSecurityException, UnsupportedEncodingException {
String value = request.getParameter(parameterName);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method doPost
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
@Override
protected void doPost(final HttpServletRequest request, final HttpServletResponse response)
throws ServletException, IOException {
try {
String moduleId = (String) request.
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method create
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
public void create( final User theUser, final AccessControledObject item,
final String message, final boolean createTamperstamp,
final String logLevel, final boolean sendEmail)
Method processResult
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
private void processResult(ResultSet rs, List<ExpandedTamperproofEventLogEntry> daysEvents, User fetchingUser,
Group adminGroup, final boolean includePersonal, final boolean validateTamperstamp)
Method updateEncryptionOnKeys
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
public void updateEncryptionOnKeys(final User user, final Encrypter encrypter)
throws SQLException, GeneralSecurityException {
if (user == null) {
return;
}
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method hasExpiringPasswords
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
public boolean hasExpiringPasswords(final User user)
throws SQLException, GeneralSecurityException, IOException {
Calendar expiryCal = Calendar.getInstance();
String warningPeriod = ConfigurationDAO.getValue(ConfigurationOption.DAYS_BEFORE_EXPIRY_TO_WARN);
if (warningPeriod != null && warningPeriod.length() > 0) {
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method getChildrenValidForUser
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
public HierarchyNodeChildren getChildrenValidForUser(final HierarchyNode node, final User theUser, boolean includeEmpty,
final Comparator<HierarchyNode> nodeComparator, final Comparator<Password> objectComparator)
Method performPostCreationActions
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
private void performPostCreationActions(final User theImporter, final Group adminGroup,
final User createdUser, UserType userType, boolean hasGeneratedPassword)
Method transferSettingToDatabase
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
private void transferSettingToDatabase(final HttpServletRequest request,
final ConfigurationDAO cDAO, TamperproofEventLogDAO telDAO,
final User thisUser, final String parameterName) throws SQLException,
Method submitChildObjectsToExecutor
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
private void submitChildObjectsToExecutor(final ExecutorService service, final HierarchyNodeDAO hnDAO,
final HierarchyNode node, final User theUser, final NodeObjectAction action) {
Method submitNodeChildrenToExecutor
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
private void submitNodeChildrenToExecutor(final ExecutorService service, final HierarchyNodeDAO hnDAO,
final HierarchyNode node, final User theUser, final NodeObjectAction action) {
Method fixTimeAt
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
private void fixTimeAt(Calendar date, int hour, int minute, int second, int millisecond) {