Revize ba8b306a

re #7886 Assembly validator in progress (validating assembly SQLQuery + locations for existing assembly, not tested for new assembly).
Added comments.
Removing unused imports.

src/main/java/vldc/aswi/AswiJpa.java

@@ -7 +7 @@
 import vldc.aswi.configuration.AppConfig;
 import vldc.aswi.configuration.DbConfig;
 
+/**
+ * Main class.
+ */
 @SpringBootApplication
 @Import({
 	AppConfig.class,
@@ -14 +17 @@
 })
 public class AswiJpa {
 
+	/**
+	 * Main method.
+	 * @param args - Input arguments.
+	 */
 	public static void main(String[] args) {
 		SpringApplication.run(AswiJpa.class, args);
 	}

src/main/java/vldc/aswi/configuration/AppConfig.java

@@ -6 +6 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.ComponentScan;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.http.HttpMethod;
 import org.springframework.security.config.annotation.authentication.builders.AuthenticationManagerBuilder;
 import org.springframework.security.config.annotation.web.builders.HttpSecurity;
 import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
@@ -14 +13 @@
 import org.springframework.security.core.userdetails.UserDetailsService;
 import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
 import org.springframework.security.crypto.password.PasswordEncoder;
-import org.springframework.security.web.util.matcher.RegexRequestMatcher;
 import org.springframework.web.servlet.config.annotation.EnableWebMvc;
 import org.springframework.web.servlet.config.annotation.ResourceHandlerRegistry;
 import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
@@ -24 +22 @@
 import org.thymeleaf.spring5.view.ThymeleafViewResolver;
 import org.thymeleaf.templatemode.TemplateMode;
 
+/**
+ * Application configuration.
+ */
 @Configuration
 @ComponentScan
 @EnableWebMvc

src/main/java/vldc/aswi/configuration/DbConfig.java

@@ -5 +5 @@
 import com.sun.istack.NotNull;
 import oracle.jdbc.pool.OracleDataSource;
 import org.springframework.boot.context.properties.ConfigurationProperties;
-import org.springframework.boot.jdbc.DataSourceBuilder;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.transaction.annotation.EnableTransactionManagement;
 
 import java.sql.SQLException;
 
+/**
+ * Connection configuration for database.
+ */
 @Configuration
 @EnableTransactionManagement
 @ConfigurationProperties(prefix = "oracle")
 public class DbConfig {
 
+	/** Username of logging user. */
 	@NotNull
 	private String username;
 
+	/** Password of logging user. */
 	@NotNull
 	private String password;
 
+	/** TNS url for connection into database. */
 	@NotNull
 	private String url;
 
+	/**
+	 * Set new username.
+	 * @param username - User's name.
+	 */
 	public void setUsername(String username) {
 		this.username = username;
 	}
 
+	/**
+	 * Set new password.
+	 * @param password - User's password.
+	 */
 	public void setPassword(String password) {
 		this.password = password;
 	}
 
+	/**
+	 * Set new url.
+	 * @param url - TNS url for connection into database.
+	 */
 	public void setUrl(String url) {
 		this.url = url;
 	}
 
+	/**
+	 * Create new datasource.
+	 * @return new Datasource of currently logged database.
+	 * @throws SQLException
+	 */
 	@Bean
 	DataSource dataSource() throws SQLException {
 		OracleDataSource dataSource = new OracleDataSource();

src/main/java/vldc/aswi/dao/AssemblyRepository.java

@@ -1 +1 @@
 package vldc.aswi.dao;
 
-import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.data.repository.CrudRepository;
 import org.springframework.stereotype.Repository;
 import vldc.aswi.domain.Assembly;
@@ -18 +17 @@
      */
     Assembly getById(Long Id);
 
-    // TODO
+
+    /**
+     * Find record with highest order.
+     * @return Assembly with highest order.
+     */
+    // TODO: test it.
     Assembly findFirst1ByOrderByAssemblyOrder();
 }

src/main/java/vldc/aswi/dao/parameter/ParameterTypeRepository.java

@@ -17 +17 @@
      */
     ParameterType getById(Long Id);
 
+    /**
+     * Find ParameterType by its name.
+     * @param name - ParameterType name
+     * @return ParameterType if name is present in database.
+     */
     ParameterType findByName(String name);
 }

src/main/java/vldc/aswi/database/DatabaseInterface.java

@@ -1 +1 @@
 package vldc.aswi.database;
 
-import org.springframework.context.annotation.Bean;
 import org.springframework.dao.DataAccessException;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.core.ResultSetExtractor;
-import org.springframework.jdbc.core.RowMapper;
 import org.springframework.stereotype.Component;
 import vldc.aswi.model.table.TableColumn;
 
@@ -16 +14 @@
 import java.util.List;
 import java.util.Map;
 
+/**
+ * Class for manipulating (selecting, updating, creating, ...) database data.
+ */
 @Component
 public class DatabaseInterface {
 
+    /** JdbcTemplate for executing queries. */
     private JdbcTemplate jdbcTemplate = null;
 
+    /**
+     * Constructor.
+     * @param jdbcTemplate - JdbcTemplate for executing queries.
+     */
     public DatabaseInterface(JdbcTemplate jdbcTemplate) {
         this.jdbcTemplate = jdbcTemplate;
     }
 
+    /**
+     * Validate given SQLQuery.
+     * @param sqlQuery - Tested SQL Query.
+     * @return true if SQL query is valid, false if not.
+     */
+    public boolean validateSQLQuery(String sqlQuery) {
+        try
+        {
+            jdbcTemplate.execute("EXPLAIN PLAN FOR " + sqlQuery);
+            return true;
+        }
+        catch (DataAccessException e)
+        {
+            return false;
+        }
+    }
+
+    /**
+     * Execute given SQL Query and return results as list of TableColumn class.
+     * @param sqlQuery - SQL query.
+     * @return List of results saved as TableColumn.
+     */
     public Map<String, TableColumn> executeQueryAndReturnTableColumns(String sqlQuery) {
         return jdbcTemplate.query(
                 sqlQuery,

src/main/java/vldc/aswi/domain/Assembly.java

@@ -65 +65 @@
         this.setIsPublic(isPublic);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/Configuration.java

@@ -49 +49 @@
         this.setTableName(tableName);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/EntityParent.java

@@ -3 +3 @@
 import javax.persistence.*;
 
 import lombok.Data;
-import org.apache.poi.ss.formula.functions.T;
-import vldc.aswi.domain.parameter.Parameter;
-
-import java.util.LinkedList;
-import java.util.List;
 
 /**
  * Class which is extended for all domain entities containing common attributes.
@@ -22 +17 @@
     @GeneratedValue(strategy=GenerationType.IDENTITY)
     private Long id;
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj)
     {

src/main/java/vldc/aswi/domain/Function.java

@@ -41 +41 @@
         this.setName(name);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/Location.java

@@ -38 +38 @@
         this.setName(name);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/Operator.java

@@ -38 +38 @@
         this.setName(name);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/Role.java

@@ -36 +36 @@
         this.setName(name);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/User.java

@@ -45 +45 @@
         this.setPassword(password);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/parameter/Parameter.java

@@ -1 +1 @@
 package vldc.aswi.domain.parameter;
 
-import com.fasterxml.jackson.annotation.JsonBackReference;
-import com.fasterxml.jackson.annotation.JsonIdentityInfo;
-import com.fasterxml.jackson.annotation.JsonManagedReference;
-import com.fasterxml.jackson.annotation.ObjectIdGenerators;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-import lombok.ToString;
 import vldc.aswi.domain.*;
 import javax.persistence.*;
 import java.util.List;
@@ -99 +94 @@
         this.setDefaultValue(defaultValue);
     }
 
+    /**
+     * Overriding toString method to prevent StackOverflowError when parsing values from assembly form.
+     * @return Parameter info.
+     */
     @Override
     public String toString() {
         return "Parameter[name=" + name + ", nameOfSelect=" + nameOfSelect + ", defaultHodnota=" + defaultValue + "]";
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/parameter/ParameterInConfiguration.java

@@ -66 +66 @@
         this.setColumnName(columnName);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/parameter/ParameterType.java

@@ -36 +36 @@
         this.setName(name);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/domain/parameter/ParameterValue.java

@@ -34 +34 @@
         this.setValue(value);
     }
 
+    /**
+     * Equal method for comparing two objects.
+     * @param obj - Comparing object.
+     * @return true if objects are same.
+     */
     @Override
     public boolean equals(Object obj) {
         return super.equals(obj);

src/main/java/vldc/aswi/model/table/TableColumn.java

@@ -3 +3 @@
 import java.util.ArrayList;
 import java.util.List;
 
+/**
+ * Class representing one table column.
+ */
 public class TableColumn {
 
+    /** Name of table column. */
     private String name;
 
+    /** List of values in current column. */
     private List<String> values;
 
+    /**
+     * Constructor.
+     * @param name - Name of table column.
+     */
     public TableColumn(String name) {
         this.name = name;
         values = new ArrayList<>();
     }
 
+    /**
+     * Add new value into list of values.
+     * @param value - New value.
+     */
     public void addValue(String value) {
         values.add(value);
     }
 
+    /**
+     * Get name of table column.
+     * @return Name of table column.
+     */
     public String getName() {
         return name;
     }
 
+    /**
+     * Get all values in current table column.
+     * @return List of all values.
+     */
     public List<String> getValues() {
         return values;
     }

src/main/java/vldc/aswi/model/table/contingencyTable/ContingencyTableRow.java

@@ -3 +3 @@
 import java.util.ArrayList;
 import java.util.List;
 
+/**
+ * Class representing one row in Contingency table. Rows are represented as tree with given level.
+ */
 public class ContingencyTableRow {
 
+    /** True if current row is header, false if not. */
     private boolean isHeader;
 
+    /** Level of current row. */
     private int level;
 
+    /** List of cells in current row. */
     private List<ContingencyTableRowCell> cells;
 
+    /**
+     * Constructor.
+     * @param isHeader - True if current row is header, false if not.
+     * @param level - Level of current row.
+     */
     public ContingencyTableRow(boolean isHeader, int level) {
         this.isHeader = isHeader;
         this.level = level;
@@ -18 +29 @@
         cells = new ArrayList<>();
     }
 
+    /**
+     * Add new cell into current row.
+     * @param cell - New cell.
+     */
     public void addTableRowCell(ContingencyTableRowCell cell) {
         cells.add(cell);
     }
 
+    /**
+     * Check if current row is header or not.
+     * @return True if current row is header, false if not.
+     */
     public boolean isHeader() {
         return isHeader;
     }
 
+    /**
+     * Get level of current cell.
+     * @return
+     */
     public int getLevel() {
         return level;
     }
 
+    /**
+     * Get all cells in current row.
+     * @return List of cells.
+     */
     public List<ContingencyTableRowCell> getCells() {
         return cells;
     }
 
+    /**
+     * Overriding toString method for debugging.
+     * @return String.
+     */
     @Override
     public String toString() {
         String str = "";

src/main/java/vldc/aswi/model/table/contingencyTable/ContingencyTableRowCell.java

@@ -1 +1 @@
 package vldc.aswi.model.table.contingencyTable;
 
+/**
+ * Class representing one cell in Contingency table row.
+ */
 public class ContingencyTableRowCell {
 
+    /** Colspan of current cell. */
     private int colSpan;
 
+    /** Value in current cell. */
     private String value;
 
+    /**
+     * Constructor.
+     * @param value - Value in current cell.
+     * @param colSpan - Colspan of current cell.
+     */
     public ContingencyTableRowCell(String value, int colSpan) {
         this.value = value;
         this.colSpan = colSpan;
     }
 
+    /**
+     * Get value of current cell.
+     * @return Value of cell.
+     */
     public String getValue() {
         return value;
     }
 
+    /**
+     * Get colspan of current cell.
+     * @return Colspan of cell.
+     */
     public int getColSpan() {
         return colSpan;
     }
 
+    /**
+     * Overrided toString method for debugging.
+     * @return String.
+     */
     @Override
     public String toString() {
         return "(" + value + ", " + colSpan + ")";

src/main/java/vldc/aswi/service/AssemblyManager.java

@@ -17 +17 @@
      */
     List<Assembly> getAssemblies();
 
+    /**
+     * Get assembly by id.
+     * @param id - ID of assembly.
+     * @return Assembly.
+     */
     Assembly getAssemblyById(Long id);
 
+    /**
+     * Update assembly.
+     * @param updatedAssemblyValues - Assembly values.
+     * @return ID of assembly.
+     */
     Long updateAssembly(Assembly updatedAssemblyValues);
 
+    /**
+     * Update assembly's order.
+     * @param id - ID of assembly.
+     * @param order - Assembly's order.
+     */
     void updateAssemblyOrder(Long id, int order);
 
+    /**
+     * Create new assembly.
+     * @param assemblyValues - Assembly values.
+     * @return ID of assembly.
+     */
     Long createAssembly(Assembly assemblyValues);
 }

src/main/java/vldc/aswi/service/AssemblyManagerImpl.java

@@ -15 +15 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Assembly.
+ */
 @Service
 @Slf4j
 public class AssemblyManagerImpl implements AssemblyManager {
@@ -53 +56 @@
         return retVal;
     }
 
+    /**
+     * Get assembly by id.
+     * @param id - ID of assembly.
+     * @return Assembly.
+     */
     @Override
     public Assembly getAssemblyById(Long id) {
         return assemblyRepository.getById(id);
     }
 
+    /**
+     * Update assembly's order.
+     * @param id - ID of assembly.
+     * @param order - Assembly's order.
+     */
     @Override
     public void updateAssemblyOrder(Long id, int order) {
         Assembly assembly = getAssemblyById(id);
@@ -65 +78 @@
         this.assemblyRepository.save(assembly);
     }
 
+    /**
+     * Update assembly.
+     * @param updatedAssemblyValues - Assembly values.
+     * @return ID of assembly.
+     */
     @Override
     public Long updateAssembly(Assembly updatedAssemblyValues) {
         // TODO: test it.
@@ -80 +98 @@
         return assembly.getId();
     }
 
+    /**
+     * Update assembly's roles.
+     * @param id - ID of assembly.
+     * @param updatedRoles - List of roles.
+     */
     private void updateAssemblyRoles(Long id, List<Role> updatedRoles) {
         // TODO: test it.
         Assembly assembly = getAssemblyById(id);
@@ -108 +131 @@
         this.assemblyRepository.save(assembly);
     }
 
+    /**
+     * Create new assembly.
+     * @param assemblyValues - Assembly values.
+     * @return ID of assembly.
+     */
     @Override
     public Long createAssembly(Assembly assemblyValues) {
         // TODO: test it.
+        // Create new assembly and assign values.
         Assembly assembly = new Assembly();
         assembly.setName(assemblyValues.getName());
         assembly.setSQLQuery(assemblyValues.getSQLQuery());
@@ -118 +147 @@
         assembly.setIsPublic(assemblyValues.getIsPublic());
         List<Role> roles = new LinkedList<>();
 
+        // Add access for roles.
         if (assemblyValues.getRoles() != null && assemblyValues.getRoles().size() > 0) {
             for (Role newRole : assemblyValues.getRoles()) {
                 roles.add(this.roleRepository.getById(newRole.getId()));
@@ -125 +155 @@
         }
 
         assembly.setRoles(roles);
+
+        // Create new record in database.
         this.assemblyRepository.save(assembly);
 
         return assembly.getId();

src/main/java/vldc/aswi/service/FunctionManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Function.
+ */
 @Service
 @Slf4j
 public class FunctionManagerImpl implements  FunctionManager {

src/main/java/vldc/aswi/service/LocationManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Location.
+ */
 @Service
 @Slf4j
 public class LocationManagerImpl implements LocationManager {

src/main/java/vldc/aswi/service/OperatorManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Operator.
+ */
 @Service
 @Slf4j
 public class OperatorManagerImpl implements OperatorManager {

src/main/java/vldc/aswi/service/RoleManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Role.
+ */
 @Service
 @Slf4j
 public class RoleManagerImpl implements RoleManager {

src/main/java/vldc/aswi/service/SqlQueryManager.java

@@ -4 +4 @@
 
 import java.util.List;
 
+/**
+ * Interface for SqlQuery manager.
+ */
 public interface SqlQueryManager {
 
-    List<ContingencyTableRow> getContingencyTableRow(String sql);
+    /**
+     * Get list of  contingencyTableRow.
+     * @param sqlQuery - SQL query.
+     * @return List of contingencyTableRow.
+     */
+    List<ContingencyTableRow> getContingencyTableRow(String sqlQuery);
+
+    /**
+     * Validate given SQL query.
+     * @param sqlQuery - Validated SQL query.
+     * @return true if SQL query is ok, false if not.
+     */
+    boolean validateSQLQuery(String sqlQuery);
 }

src/main/java/vldc/aswi/service/SqlQueryManagerImpl.java

@@ -12 +12 @@
 import java.util.List;
 import java.util.Map;
 
+/**
+ * Manager for SqlQuery.
+ */
 @Service
 @Slf4j
 public class SqlQueryManagerImpl implements SqlQueryManager{
 
-    @Autowired
-    private DataSource dataSource;
-
+    /** Autowired databaseInterface component. */
     @Autowired
     private DatabaseInterface databaseInterface;
 
+    /**
+     * Get list of  contingencyTableRow.
+     * @param sqlQuery - SQL query.
+     * @return List of contingencyTableRow.
+     */
     @Override
     public List<ContingencyTableRow> getContingencyTableRow(String sqlQuery) {
         Map<String, TableColumn> tableColumns = databaseInterface.executeQueryAndReturnTableColumns(sqlQuery);
         return Converter.convertTableColumnsToContingencyTableRows(tableColumns);
     }
+
+    /**
+     * Validate given SQL query.
+     * @param sqlQuery - Validated SQL query.
+     * @return true if SQL query is ok, false if not.
+     */
+    @Override
+    public boolean validateSQLQuery(String sqlQuery) {
+        return databaseInterface.validateSQLQuery(sqlQuery);
+    }
 }

src/main/java/vldc/aswi/service/UserManagerImpl.java

@@ -14 +14 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for User.
+ */
 @Service
 @Slf4j
 public class UserManagerImpl implements UserManager {
@@ -46 +49 @@
             User referentka = this.userRepository.getById(idReferentka);
             User prorektor = this.userRepository.getById(idProrektor);
 
-            // TODO: set roles for every default user.
             //admin.setRole();
         }
     }

src/main/java/vldc/aswi/service/parameter/ParameterInConfigurationManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for ParameterInConfiguration.
+ */
 @Service
 @Slf4j
 public class ParameterInConfigurationManagerImpl implements ParameterInConfigurationManager {

src/main/java/vldc/aswi/service/parameter/ParameterManager.java

@@ -15 +15 @@
      */
     List<Parameter> getParameters();
 
+    /**
+     * Add new parameter into database.
+     * @param assemblyId - Assembly in which parameter is presented.
+     * @param parameterValuesList - Parameter values.
+     */
     void addParameters(Long assemblyId, List<Parameter> parameterValuesList);
 
+    /**
+     * Update parameter with given values.
+     * @param assemblyId - Assembly in which parameter is presented.
+     * @param newAssemblyParameters - List of parameters.
+     */
     void updateParameters(Long assemblyId, List<Parameter> newAssemblyParameters);
 }

src/main/java/vldc/aswi/service/parameter/ParameterManagerImpl.java

@@ -15 +15 @@
 import vldc.aswi.dao.parameter.ParameterValueRepository;
 import vldc.aswi.domain.*;
 import vldc.aswi.domain.parameter.Parameter;
-import vldc.aswi.domain.parameter.ParameterType;
 import vldc.aswi.domain.parameter.ParameterValue;
-import vldc.aswi.service.FunctionManager;
-import vldc.aswi.service.LocationManager;
-import vldc.aswi.service.OperatorManager;
 
 import javax.transaction.Transactional;
-import java.util.ArrayList;
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for Parameter.
+ */
 @Service
 @Slf4j
 public class ParameterManagerImpl implements ParameterManager {
@@ -34 +32 @@
     @Autowired
     private ParameterRepository parameterRepository;
 
+    /** Autowired assembly repository. */
     @Autowired
     private AssemblyRepository assemblyRepository;
 
+    /** Autowired parameterType repository. */
     @Autowired
     private ParameterTypeRepository parameterTypeRepository;
 
+    /** Autowired location repository. */
     @Autowired
     private LocationRepository locationRepository;
 
+    /** Autowired function repository. */
     @Autowired
     private FunctionRepository functionRepository;
 
+    /** Autowired operator repository. */
     @Autowired
     private OperatorRepository operatorRepository;
 
+    /** Autowired parameterValue repository. */
     @Autowired
     private ParameterValueRepository parameterValueRepository;
 
@@ -79 +83 @@
         return retVal;
     }
 
+    /**
+     * Add new parameter into database.
+     * @param assemblyId - Assembly in which parameter is presented.
+     * @param parameterValuesList - Parameter values.
+     */
     @Override
     public void addParameters(Long assemblyId, List<Parameter> parameterValuesList) {
         Assembly assembly = this.assemblyRepository.getById(assemblyId);
@@ -86 +95 @@
         if (parameterValuesList == null) return;
 
         for (Parameter parameterValues : parameterValuesList) {
+            // Create new parameter and assign him all values.
             Parameter parameter = new Parameter();
             parameter.setName(parameterValues.getNameOfSelect());
             parameter.setNameOfSelect(parameterValues.getNameOfSelect());
@@ -97 +107 @@
             List<Function> functionList = new LinkedList<>();
             List<Operator> operatorList = new LinkedList<>();
 
+            // Try to put all locations into list.
             if (parameterValues.getLocations() != null && parameterValues.getLocations().size() > 0) {
                 for (Location location : parameterValues.getLocations()) {
                     locationList.add(this.locationRepository.getById(location.getId()));
                 }
             }
 
+            // Try to put all functions into list.
             if (parameterValues.getFunctions() != null && parameterValues.getFunctions().size() > 0) {
                 for (Function function : parameterValues.getFunctions()) {
                     functionList.add(this.functionRepository.getById(function.getId()));
                 }
             }
 
+            // Try to put all operators into list.
             if (parameterValues.getOperators() != null && parameterValues.getOperators().size() > 0) {
                 for (Operator operator : parameterValues.getOperators()) {
                     operatorList.add(this.operatorRepository.getById(operator.getId()));
                 }
             }
 
+            // Set lists of values into parameter.
             parameter.setLocations(locationList);
             parameter.setFunctions(functionList);
             parameter.setOperators(operatorList);
 
+            // Insert parameter into database.
             this.parameterRepository.save(parameter);
         }
     }
 
+    /**
+     * Update parameter - remove deleted parameters, add new parameters, update updated parameters.
+     * @param assemblyId - Assembly in which parameter is presented.
+     * @param newAssemblyParameters - List of parameters.
+     */
     @Override
     public void updateParameters(Long assemblyId, List<Parameter> newAssemblyParameters) {
         // TODO: test it.
         Assembly assembly = this.assemblyRepository.getById(assemblyId);
         List<Parameter> currentAssemblyParameters = assembly.getParameters();
 
-        // Find new parameters;
+        // Find new parameters.
         List<Parameter> currentAssemblyParametersNew = new LinkedList<>(currentAssemblyParameters);
         List<Parameter> newAssemblyParametersNew = new LinkedList<>(newAssemblyParameters);
         newAssemblyParametersNew.removeAll(currentAssemblyParametersNew);
 
-        // Find deleted parameters
+        // Find deleted parameters.
         List<Parameter> currentAssemblyParametersDeleted = new LinkedList<>(currentAssemblyParameters);
         List<Parameter> newAssemblyParametersDeleted = new LinkedList<>(newAssemblyParameters);
         currentAssemblyParametersDeleted.removeAll(newAssemblyParametersDeleted);
@@ -150 +170 @@
         // Remove deleted parameters.
         for (Parameter parameter : currentAssemblyParametersDeleted) {
             for (ParameterValue parameterValue : parameter.getParameterValues()) {
+                // Remove parameters values.
                 this.parameterValueRepository.delete(parameterValue);
             }
 
+            // Remove parameter.
             this.parameterRepository.delete(parameter);
         }
 
@@ -163 +185 @@
         addParameters(assemblyId, newAssemblyParametersNew);
     }
 
+    /**
+     * Update updated parameters.
+     * @param parameterValuesList - List of updated parameters.
+     */
     private void updateParameters(List<Parameter> parameterValuesList) {
         // TODO: test it.
         for (Parameter parameterValues : parameterValuesList) {
+            // Get current parameter from database.
             Parameter parameter = this.parameterRepository.getById(parameterValues.getId());
+            // Update current parameter values.
             parameter.setName(parameterValues.getName());
             parameter.setNameOfSelect(parameterValues.getNameOfSelect());
             parameter.setAssembly(this.assemblyRepository.getById(parameterValues.getAssembly().getId()));
             parameter.setParameterType(this.parameterTypeRepository.getById(parameterValues.getParameterType().getId()));
             parameter.setDefaultValue(parameterValues.getDefaultValue());
+
+            // Insert changes into database.
             this.parameterRepository.save(parameter);
 
+            // Update parameter values, locations, functions and operators.
             updateParameterValues(parameterValues.getId(), parameterValues.getParameterValues());
             updateParameterLocations(parameterValues.getId(), parameterValues.getLocations());
             updateParameterFunctions(parameterValues.getId(), parameterValues.getFunctions());
@@ -181 +212 @@
         }
     }
 
+    /**
+     * Update current parameter locations.
+     * @param id - Id of parameter.
+     * @param updatedLocations - List of locations.
+     */
     private void updateParameterLocations(Long id, List<Location> updatedLocations) {
         // TODO: test it.
         Parameter parameter = this.parameterRepository.getById(id);
         List<Location> currentLocations = parameter.getLocations();
 
+        // Get new locations.
         List<Location> currentLocationsNew = new LinkedList<>(currentLocations);
         List<Location> updatedLocationsNew = new LinkedList<>(updatedLocations);
         updatedLocationsNew.removeAll(currentLocationsNew);
 
+        // Get deleted locations.
         List<Location> currentLocationsDeleted = new LinkedList<>(currentLocations);
         List<Location> updatedLocationsDeleted = new LinkedList<>(updatedLocations);
         currentLocationsDeleted.removeAll(updatedLocationsDeleted);
 
+        // If there is no change, do not update.
         if (updatedLocationsNew.size() == 0 && currentLocationsDeleted.size() == 0) return;
 
+        // Add new locations.
         for (Location newLocation : updatedLocationsNew) {
             currentLocations.add(this.locationRepository.getById(newLocation.getId()));
         }
+
+        // Remove deleted locations.
         currentLocations.removeAll(currentLocationsDeleted);
 
+        // Insert changes into database.
         this.parameterRepository.save(parameter);
     }
 
+    /**
+     * Update current parameter functions.
+     * @param id - Id of parameter.
+     * @param updatedFunctions - List of functions.
+     */
     private void updateParameterFunctions(Long id, List<Function> updatedFunctions) {
         // TODO: test it.
         Parameter parameter = this.parameterRepository.getById(id);
         List<Function> currentFunctions = parameter.getFunctions();
 
+        // Get new functions.
         List<Function> currentFunctionsNew = new LinkedList<>(currentFunctions);
         List<Function> updatedFunctionsNew = new LinkedList<>(updatedFunctions);
         updatedFunctionsNew.removeAll(currentFunctionsNew);
 
+        // Get deleted functions.
         List<Function> currentFunctionsDeleted = new LinkedList<>(currentFunctions);
         List<Function> updatedFunctionsDeleted = new LinkedList<>(updatedFunctions);
         currentFunctionsDeleted.removeAll(updatedFunctionsDeleted);
 
+        // If there is no change, do not update.
         if (updatedFunctionsNew.size() == 0 && currentFunctionsDeleted.size() == 0) return;
 
+        // Add new functions.
         for (Function newFunction : updatedFunctionsNew) {
             currentFunctions.add(this.functionRepository.getById(newFunction.getId()));
         }
+
+        // Delete removed functions.
         currentFunctions.removeAll(currentFunctionsDeleted);
 
+        // Insert changes into database.
         this.parameterRepository.save(parameter);
     }
 
+    /**
+     * Update current parameter operators.
+     * @param id - Id of parameter.
+     * @param updatedOperators - List of operators.
+     */
     private void updateParameterOperators(Long id, List<Operator> updatedOperators) {
         // TODO: test it.
         Parameter parameter = this.parameterRepository.getById(id);
         List<Operator> currentOperators = parameter.getOperators();
 
+        // Get new operators.
         List<Operator> currentOperatorsNew = new LinkedList<>(currentOperators);
         List<Operator> updatedOperatorsNew = new LinkedList<>(updatedOperators);
         updatedOperatorsNew.removeAll(currentOperatorsNew);
 
+        // Get deleted operators.
         List<Operator> currentOperatorsDeleted = new LinkedList<>(currentOperators);
         List<Operator> updatedOperatorsDeleted = new LinkedList<>(updatedOperators);
         currentOperatorsDeleted.removeAll(updatedOperatorsDeleted);
 
+        // If there is no changes, do not update.
         if (updatedOperatorsNew.size() == 0 && currentOperatorsDeleted.size() == 0) return;
 
+        // Insert new operators.
         for (Operator newOperator : updatedOperatorsNew) {
             currentOperators.add(this.operatorRepository.getById(newOperator.getId()));
         }
+
+        // Delete removed operators.
         currentOperators.removeAll(currentOperatorsDeleted);
 
+        // Insert changes into database.
         this.parameterRepository.save(parameter);
     }
 
+    /**
+     * Update current parameter values.
+     * @param parameterId - ID of parameter.
+     * @param newParameterValues - List of parameter values.
+     */
     private void updateParameterValues(Long parameterId, List<ParameterValue> newParameterValues) {
         // TODO: test it.
         Parameter parameter = this.parameterRepository.getById(parameterId);
         List<ParameterValue> currentParameterValues = parameter.getParameterValues();
 
+        // Get new parameter values.
         List<ParameterValue> currentParameterValuesNew = new LinkedList<>(currentParameterValues);
         List<ParameterValue> newParameterValuesNew = new LinkedList<>(newParameterValues);
         newParameterValuesNew.removeAll(currentParameterValuesNew);
 
+        // Get deleted parameter values.
         List<ParameterValue> currentParameterValuesDeleted = new LinkedList<>(currentParameterValues);
         List<ParameterValue> newParameterValuesDeleted = new LinkedList<>(newParameterValues);
         currentParameterValuesDeleted.removeAll(newParameterValuesDeleted);
 
+        // Get updated parameter values.
         List<ParameterValue> currentParameterValuesUpdated = new LinkedList<>(currentParameterValues);
         List<ParameterValue> newParameterValuesUpdated = new LinkedList<>(newParameterValues);
         newParameterValuesUpdated.retainAll(currentParameterValuesUpdated);
 
+        // If there is no changes, do not update.
         if (newParameterValuesNew.size() == 0 && currentParameterValuesDeleted.size() == 0 && newParameterValuesUpdated.size() == currentParameterValues.size()) return;
 
+        // Delete removed parameter values.
         for (ParameterValue parameterValue : currentParameterValuesDeleted) {
             this.parameterValueRepository.delete(parameterValue);
         }
 
+        // Update parameter values.
         updateParameterValueValues(newParameterValuesUpdated);
 
+        // Add new parameter values.
         addParameterValueValues(parameterId, newParameterValuesNew);
     }
 
+    /**
+     * Update all updated parameter values.
+     * @param updatedParameterValuesList - List of all updated parameter values.
+     */
     private void updateParameterValueValues(List<ParameterValue> updatedParameterValuesList) {
         // TODO: test it.
         for (ParameterValue parameterValueValues : updatedParameterValuesList) {
             ParameterValue parameterValue = this.parameterValueRepository.getById(parameterValueValues.getId());
             parameterValue.setValue(parameterValueValues.getValue());
 
+            // Insert changes into database.
             this.parameterValueRepository.save(parameterValue);
         }
     }
 
+    /**
+     * Add new parameter values.
+     * @param parameterId - ID of parameter.
+     * @param newParameterValuesNew - List of new parameter values.
+     */
     private void addParameterValueValues(Long parameterId, List<ParameterValue> newParameterValuesNew) {
         // TODO: test it.
         Parameter parameter = this.parameterRepository.getById(parameterId);
@@ -297 +386 @@
             parameterValue.setValue(parameterValueValues.getValue());
             parameterValue.setParameter(parameter);
 
+            // Insert new parameterValue into database.
             this.parameterValueRepository.save(parameterValue);
         }
     }

src/main/java/vldc/aswi/service/parameter/ParameterTypeManagerImpl.java

@@ -13 +13 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for ParameterType.
+ */
 @Service
 @Slf4j
 public class ParameterTypeManagerImpl implements ParameterTypeManager {

src/main/java/vldc/aswi/service/parameter/ParameterValueManager.java

@@ -14 +14 @@
      * @return List of parameter values.
      */
     List<ParameterValue> getParameterValues();
-
-    void createParameterValues(Long parameterId, List<ParameterValue> parameterValues);
-
-    void updateParameterValues(Long parameterId, List<ParameterValue> parameterValues);
-
-    void addParameterValues(Long parameterId, List<ParameterValue> newParameterValuesNew);
 }

src/main/java/vldc/aswi/service/parameter/ParameterValueManagerImpl.java

@@ -15 +15 @@
 import java.util.LinkedList;
 import java.util.List;
 
+/**
+ * Manager for ParameterValue.
+ */
 @Service
 @Slf4j
 public class ParameterValueManagerImpl implements ParameterValueManager {
@@ -52 +55 @@
         this.parameterValueRepository.findAll().forEach(retVal::add);
         return retVal;
     }
-
-
-    @Override
-    public void createParameterValues(Long parameterId, List<ParameterValue> parameterValues) {
-        Parameter parameter = this.parameterRepository.getById(parameterId);
-
-        for (ParameterValue newParameterValue : parameterValues) {
-            ParameterValue parameterValue = new ParameterValue();
-            parameterValue.setValue(newParameterValue.getValue());
-            parameterValue.setParameter(parameter);
-
-            this.parameterValueRepository.save(parameterValue);
-        }
-    }
-
-    @Override
-    public void updateParameterValues(Long parameterId, List<ParameterValue> newParameterValues) {
-        Parameter parameter = this.parameterRepository.getById(parameterId);
-        List<ParameterValue> currentParameterValues = parameter.getParameterValues();
-
-        List<ParameterValue> currentParameterValuesNew = new LinkedList<>(currentParameterValues);
-        List<ParameterValue> newParameterValuesNew = new LinkedList<>(newParameterValues);
-        newParameterValuesNew.removeAll(currentParameterValuesNew);
-
-        List<ParameterValue> currentParameterValuesDeleted = new LinkedList<>(currentParameterValues);
-        List<ParameterValue> newParameterValuesDeleted = new LinkedList<>(newParameterValues);
-        currentParameterValuesDeleted.removeAll(newParameterValuesDeleted);
-
-        List<ParameterValue> currentParameterValuesUpdated = new LinkedList<>(currentParameterValues);
-        List<ParameterValue> newParameterValuesUpdated = new LinkedList<>(newParameterValues);
-        newParameterValuesUpdated.retainAll(currentParameterValuesUpdated);
-
-        if (newParameterValuesNew.size() == 0 && currentParameterValuesDeleted.size() == 0 && newParameterValuesUpdated.size() == currentParameterValues.size()) return;
-
-        for (ParameterValue parameterValue : currentParameterValuesDeleted) {
-            this.parameterValueRepository.delete(parameterValue);
-        }
-
-        updateParameterValues(newParameterValuesUpdated);
-
-        addParameterValues(parameterId, newParameterValuesNew);
-    }
-
-    private void updateParameterValues(List<ParameterValue> updatedParameterValuesList) {
-        for (ParameterValue parameterValueValues : updatedParameterValuesList) {
-            ParameterValue parameterValue = this.parameterValueRepository.getById(parameterValueValues.getId());
-            parameterValue.setValue(parameterValueValues.getValue());
-
-            this.parameterValueRepository.save(parameterValue);
-        }
-    }
-
-    @Override
-    public void addParameterValues(Long parameterId, List<ParameterValue> newParameterValuesNew) {
-        Parameter parameter = this.parameterRepository.getById(parameterId);
-
-        for (ParameterValue parameterValueValues : newParameterValuesNew) {
-            ParameterValue parameterValue = new ParameterValue();
-            parameterValue.setValue(parameterValueValues.getValue());
-            parameterValue.setParameter(parameter);
-
-            this.parameterValueRepository.save(parameterValue);
-        }
-    }
-
 }

src/main/java/vldc/aswi/utils/Converter.java

@@ -8 +8 @@
 import java.util.List;
 import java.util.Map;
 
+/**
+ * Class for converting classes.
+ */
 public class Converter {
 
+    /**
+     * Convert Map of TableColumns into list of ContingencyTableRow.
+     * @param tableColumns - Map of TableColumn.
+     * @return List of ContingencyTableRow.
+     */
     public static List<ContingencyTableRow> convertTableColumnsToContingencyTableRows(Map<String, TableColumn> tableColumns) {
         List<ContingencyTableRow> contingencyTableRows = new ArrayList<>();
 
+        // Create header row with column names.
         ContingencyTableRow contingencyTableRowHeader = new ContingencyTableRow(true, 0);
         for (String columnName : tableColumns.keySet()) {
             contingencyTableRowHeader.addTableRowCell(new ContingencyTableRowCell(columnName, 0));
@@ -21 +30 @@
 
         boolean listExpanded = false;
 
+        // Create contingency table row.
         for (TableColumn tableColumn : tableColumns.values()) {
             List<String> values = tableColumn.getValues();
 
+            // If list is not created, then create it.
             if (!listExpanded) {
                 for (int j = 0; j < values.size(); j++) {
                     ContingencyTableRow contingencyTableRow = new ContingencyTableRow(false, 0);
@@ -32 +43 @@
                 listExpanded = true;
             }
 
+            // Iterate through all values and assign one value per one row.
             for (int j = 0; j < values.size(); j++) {
                 ContingencyTableRowCell contingencyTableRowCell = new ContingencyTableRowCell(values.get(j), 0);
                 contingencyTableRows.get(j + 1).addTableRowCell(contingencyTableRowCell);

src/main/java/vldc/aswi/validators/AssemblyValidator.java

@@ +1 @@
+package vldc.aswi.validators;
+
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Component;
+import org.springframework.validation.Errors;
+import org.springframework.validation.Validator;
+import vldc.aswi.domain.Assembly;
+import vldc.aswi.domain.parameter.Parameter;
+import vldc.aswi.service.SqlQueryManager;
+
+/**
+ * Validator for assembly.
+ */
+@Component
+public class AssemblyValidator implements Validator {
+
+    /** Autowired sql query manager. */
+    @Autowired
+    private SqlQueryManager sqlQueryManager;
+
+    /**
+     * Validator for supporting this class.
+     * @param aClass - Class.
+     * @return true if sent class match Assembly class.
+     */
+    @Override
+    public boolean supports(Class<?> aClass) {
+        return Assembly.class.equals(aClass);
+    }
+
+    /**
+     * Validate assembly values.
+     * @param obj - Validated object.
+     * @param errors - Errors sended to output.
+     */
+    @Override
+    public void validate(Object obj, Errors errors) {
+        // TODO: VALIDATING   - https://docs.spring.io/spring/docs/4.1.x/spring-framework-reference/html/validation.html
+        // TODO: ERROR STYLES - https://mkyong.com/spring-mvc/spring-mvc-form-errors-tag-example/
+
+        // Get assembly values.
+        Assembly assembly = (Assembly) obj;