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 createTamperstamp has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public byte[] createTamperstamp(User theUser, final long datetime, final String event,
                                    final String itemId, final String userId)

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 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 processResults has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    private List<EventsForDay> processResults(final ResultSet rs, User fetchingUser, Group adminGroup,
                                             final boolean includePersonal, final boolean validateTamperstamp)

Method create has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public void create( final String logLevel, final User theUser, final AccessControledObject item,
                        final String message, final boolean sendEmail )

Method extractCustomFieldsFromRequest has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    Map<String, String> extractCustomFieldsFromRequest(HttpServletRequest request) {
        Map<String,String> customFields = new TreeMap<>();
        int cfCount = -1;

        while( true ) {

• Read up
  Read up

Method equals has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public boolean equals(GenericJDBCConnectionInformation otherConfig) {
        return otherConfig != null && (dbType.equals(otherConfig.dbType)
            && driver.equals(otherConfig.driver) && url.equals(otherConfig.url)
            && username.equals(otherConfig.username) && password.equals(otherConfig.password));
    }

• Read up
  Read up

Method run has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    public void run() {
        if (!smtpEnabled) {
            return;
        }

• Read up
  Read up

Method doPost has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    protected void doPost(final HttpServletRequest request, final HttpServletResponse response)
            throws IOException, ServletException {
        request.setAttribute("error_page", "/system/CreatePassword");
        try {

• Read up
  Read up

Method decrypt has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public static void decrypt(final PasswordBase password, final AccessControl ac,
                                     final byte[] data, final Properties passwordProperties)
        throws IOException, GeneralSecurityException {
        password.decrypt(ac);
        String propertiesString = ac.decrypt(data);

• Read up
  Read up

Method removeEscapeCharacters has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public static String removeEscapeCharacters( String original ) {
        StringBuilder newString = new StringBuilder(original.length());
        for( int i = 0 ; i < original.length() ; i++) {
            char thisChar = original.charAt(i);
            if(thisChar == '\\') {

• Read up
  Read up

Method getRulesForUser has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public final Map<String,String> getRulesForUser( final String id ) throws SQLException {
        Map<String,String> rules = new HashMap<>();
        try(PreparedStatement ps = BOMFactory.getCurrentConntection().prepareStatement(GET_BY_USER_ID_SQL)) {
            ps.setString(1, id);
            try(ResultSet rs = ps.executeQuery()) {

• Read up
  Read up

Method newInstance has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    Password newInstance(ResultSet rs)
            throws SQLException {
        Password password = new Password(rs.getString(1), rs.getBytes(1 + 1));

• Read up
  Read up

Method setRestrictedAccess has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    private void setRestrictedAccess(final HttpServletRequest request, final Password password ) {
        String raEnabled = request.getParameter("ra_enabled");
        if( raEnabled != null && raEnabled.equals("Y") ) {
            password.setRaEnabled(true);
            String approvers = request.getParameter("ra_approvers");

• Read up
  Read up

Method doPost has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    protected void doPost( final HttpServletRequest request, final HttpServletResponse response)
            throws ServletException, IOException {
        String id = request.getParameter(SecurityUtils.USER_ID_PARAMETER);
        String internalpassword = request.getParameter(INTERNAL_PASSWORD_PARAMETER);

• Read up
  Read up

Method doFilter has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    public void doFilter(ServletRequest request, ServletResponse response,
            FilterChain next) throws ServletException, IOException {
        try {
            next.doFilter(request, response);

• Read up
  Read up

Method santiseName has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public static String santiseName(final String originalName) {
        StringBuilder endName = new StringBuilder(originalName.length());
        
        boolean capsNext = false;
        for(char c : originalName.toCharArray()) {

• Read up
  Read up

Avoid too many return statements within this method.

Open

            return ACCESIBILITY_ALLOWED;

Avoid too many return statements within this method.

Open

                return theNode.getName();