Revize cd3f21e5

Configuration renamed to Threshold

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/controller/AppController.java

@@ -219 +219 @@
      * Method of storing new configurations for individual AP.
      *
      * @param model object for passing data to the UI
-     * @param configValues changed configuration values
-     * @param configNames changed configuration names
+     * @param thresholdValues changed configuration values
+     * @param thresholdNames changed configuration names
      * @return html file name for thymeleaf template
      */
     @PostMapping("/configuration")
     public String configurationPost(Model model,
-                                    @RequestParam(value = "configValues", required = false) String[] configValues,
-                                    @RequestParam(value = "configNames", required = false) String[] configNames) {
+                                    @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
+                                    @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames) {
 
         List<AntiPattern> antiPatterns = antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns());
-        List<String> wrongParameters = antiPatternService.saveNewConfiguration(configNames, configValues);
+        List<String> wrongParameters = antiPatternService.saveNewConfiguration(thresholdNames, thresholdValues);
 
         if (wrongParameters.isEmpty()) {
             model.addAttribute("successMessage", "All configuration values has been successfully saved.");
@@ -247 +247 @@
      *
      * @param model object for passing data to the UI
      * @param id id of AP
-     * @param configValues new config values
-     * @param configNames configuration names
+     * @param thresholdValues new config values
+     * @param thresholdNames configuration names
      * @param redirectAttrs attributes for redirection
      * @return redirected html file name for thymeleaf template
      */
     @PostMapping("/anti-patterns/{id}")
     public String antiPatternsPost(Model model,
                                    @PathVariable Long id,
-                                   @RequestParam(value = "configValues", required = false) String[] configValues,
-                                   @RequestParam(value = "configNames", required = false) String[] configNames,
+                                   @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
+                                   @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames,
                                    RedirectAttributes redirectAttrs) {
 
         AntiPattern antiPattern = antiPatternService.antiPatternToModel(antiPatternService.getAntiPatternById(id));
-        List<String> wrongParameters = antiPatternService.saveNewConfiguration(configNames, configValues);
+        List<String> wrongParameters = antiPatternService.saveNewConfiguration(thresholdNames, thresholdValues);
 
         if (wrongParameters.isEmpty()) {
             redirectAttrs.addFlashAttribute("successMessage", "All threshold values has been successfully saved.");

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/AntiPattern.java

@@ -18 +18 @@
     private String printName;
     private String name;
     private String description;
-    private Map<String, Configuration> configurations;
+    private Map<String, Threshold> thresholds;
     private String catalogueFileName;
 
     public AntiPattern(Long id, String printName, String name, String description) {
@@ -28 +28 @@
         this.description = description;
     }
 
-    public AntiPattern(Long id, String printName, String name, String description, Map<String, Configuration> configurations) {
+    public AntiPattern(Long id, String printName, String name, String description, Map<String, Threshold> thresholds) {
         this.id = id;
         this.printName = printName;
         this.name = name;
         this.description = description;
-        this.configurations = configurations;
+        this.thresholds = thresholds;
     }
 
-    public AntiPattern(Long id, String printName, String name, String description, Map<String, Configuration> configurations, String catalogueFileName) {
+    public AntiPattern(Long id, String printName, String name, String description, Map<String, Threshold> thresholds, String catalogueFileName) {
         this.id = id;
         this.printName = printName;
         this.name = name;
         this.description = description;
-        this.configurations = configurations;
+        this.thresholds = thresholds;
         this.catalogueFileName = catalogueFileName;
     }
 
@@ -77 +77 @@
         this.description = description;
     }
 
-    public Map<String, Configuration> getConfigurations() {
-        return configurations;
+    public Map<String, Threshold> getThresholds() {
+        return thresholds;
     }
 
-    public void setConfigurations(Map<String, Configuration> configurations) {
-        this.configurations = configurations;
+    public void setThresholds(Map<String, Threshold> thresholds) {
+        this.thresholds = thresholds;
     }
 
     public String getUrl() {
@@ -156 +156 @@
                 ", printName='" + printName + '\'' +
                 ", name='" + name + '\'' +
                 ", description='" + description + '\'' +
-                ", configurations=" + configurations +
+                ", thresholds=" + thresholds +
                 ", catalogueFileName='" + catalogueFileName + '\'' +
                 '}';
     }

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/Configuration.java

@@ -1 @@
-package cz.zcu.fav.kiv.antipatterndetectionapp.model;
-
-/**
- * Model class for configuration.
- * @param <T> configuration can have different data types
- */
-public class Configuration<T> {
-    private String name;
-    private String printName;
-    private String description;
-    private String errorMessage;
-    private boolean isErrorMessageShown;
-    private T value;
-
-    public Configuration(String name, String printName, String description, T value) {
-        this.name = name;
-        this.printName = printName;
-        this.description = description;
-        this.value = value;
-        this.isErrorMessageShown = false;
-    }
-
-    public Configuration(String name, String printName, String description, String errorMessage, T value) {
-        this.name = name;
-        this.printName = printName;
-        this.description = description;
-        this.errorMessage = errorMessage;
-        this.value = value;
-        this.isErrorMessageShown = false;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public String getPrintName() {
-        return printName;
-    }
-
-    public void setPrintName(String printName) {
-        this.printName = printName;
-    }
-
-    public String getDescription() {
-        return description;
-    }
-
-    public void setDescription(String description) {
-        this.description = description;
-    }
-
-    public T getValue() {
-        return value;
-    }
-
-    public void setValue(T value) {
-        this.value = value;
-    }
-
-    public String getErrorMessage() {
-        return errorMessage;
-    }
-
-    public void setErrorMessage(String errorMessage) {
-        this.errorMessage = errorMessage;
-    }
-
-    public boolean isErrorMessageShown() {
-        return isErrorMessageShown;
-    }
-
-    public void setErrorMessageShown(boolean errorMessageShown) {
-        isErrorMessageShown = errorMessageShown;
-    }
-}

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/Threshold.java

@@ +1 @@
+package cz.zcu.fav.kiv.antipatterndetectionapp.model;
+
+/**
+ * Model class for threshold.
+ * @param <T> threshold can have different data types
+ */
+public class Threshold<T> {
+    private String name;
+    private String printName;
+    private String description;
+    private String errorMessage;
+    private boolean isErrorMessageShown;
+    private T value;
+
+    public Threshold(String name, String printName, String description, T value) {
+        this.name = name;
+        this.printName = printName;
+        this.description = description;
+        this.value = value;
+        this.isErrorMessageShown = false;
+    }
+
+    public Threshold(String name, String printName, String description, String errorMessage, T value) {
+        this.name = name;
+        this.printName = printName;
+        this.description = description;
+        this.errorMessage = errorMessage;
+        this.value = value;
+        this.isErrorMessageShown = false;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public String getPrintName() {
+        return printName;
+    }
+
+    public void setPrintName(String printName) {
+        this.printName = printName;
+    }
+
+    public String getDescription() {
+        return description;
+    }
+
+    public void setDescription(String description) {
+        this.description = description;
+    }
+
+    public T getValue() {
+        return value;
+    }
+
+    public void setValue(T value) {
+        this.value = value;
+    }
+
+    public String getErrorMessage() {
+        return errorMessage;
+    }
+
+    public void setErrorMessage(String errorMessage) {
+        this.errorMessage = errorMessage;
+    }
+
+    public boolean isErrorMessageShown() {
+        return isErrorMessageShown;
+    }
+
+    public void setErrorMessageShown(boolean errorMessageShown) {
+        isErrorMessageShown = errorMessageShown;
+    }
+}

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/service/AntiPatternServiceImpl.java

@@ -4 +4 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.detecting.detectors.AntiPatternDetector;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.AntiPattern;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.CacheablesValues;
-import cz.zcu.fav.kiv.antipatterndetectionapp.model.Configuration;
+import cz.zcu.fav.kiv.antipatterndetectionapp.model.Threshold;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.QueryResult;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.types.Percentage;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.types.PositiveFloat;
@@ -48 +48 @@
             AntiPattern antiPattern = antiPatternDetector.getAntiPatternModel();
 
             // set to default value
-            for (Map.Entry<String, Configuration> config : antiPattern.getConfigurations().entrySet()) {
-                config.getValue().setErrorMessageShown(false);
+            for (Map.Entry<String, Threshold> threshold : antiPattern.getThresholds().entrySet()) {
+                threshold.getValue().setErrorMessageShown(false);
             }
             antiPatterns.add(antiPattern);
         }
@@ -60 +60 @@
     public AntiPattern antiPatternToModel(AntiPatternDetector antiPatternDetector) {
         AntiPattern antiPattern = antiPatternDetector.getAntiPatternModel();
         // set to default value
-        for (Map.Entry<String, Configuration> config : antiPattern.getConfigurations().entrySet()) {
-            config.getValue().setErrorMessageShown(false);
+        for (Map.Entry<String, Threshold> threshold : antiPattern.getThresholds().entrySet()) {
+            threshold.getValue().setErrorMessageShown(false);
         }
         return antiPattern;
     }
@@ -76 +76 @@
         return antiPatternDetectors;
     }
 
-    public List<String> saveNewConfiguration(String[] configNames, String[] configValues) {
+    public List<String> saveNewConfiguration(String[] thresholdNames, String[] thresholdValues) {
         List<AntiPatternDetector> antiPatternDetectors = antiPatternRepository.getAllAntiPatterns();
         List<String> incorrectParameters = new ArrayList<>();
 
         for (AntiPatternDetector antiPatternDetector : antiPatternDetectors) {
-            // not every anti-pattern should have configuration
-            if (antiPatternDetector.getAntiPatternModel().getConfigurations() == null) {
+            // not every anti-pattern should have thresholds
+            if (antiPatternDetector.getAntiPatternModel().getThresholds() == null) {
                 continue;
             }
-            for (int i = 0; i < configNames.length; i++) {
-                if (antiPatternDetector.getAntiPatternModel().getConfigurations().containsKey(configNames[i])) {
+            for (int i = 0; i < thresholdNames.length; i++) {
+                if (antiPatternDetector.getAntiPatternModel().getThresholds().containsKey(thresholdNames[i])) {
 
-                    if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == Integer.class) {
+                    if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == Integer.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((Integer.parseInt(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((Integer.parseInt(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
 
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == Percentage.class) {
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == Percentage.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((Percentage.parsePercentage(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((Percentage.parsePercentage(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == PositiveInteger.class) {
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == PositiveInteger.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((PositiveInteger.parsePositiveInteger(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((PositiveInteger.parsePositiveInteger(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == PositiveFloat.class) {
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == PositiveFloat.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((PositiveFloat.parsePositiveFloat(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((PositiveFloat.parsePositiveFloat(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == Float.class) {
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == Float.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((Float.parseFloat(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((Float.parseFloat(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == Double.class) {
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == Double.class) {
                         try {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((Double.parseDouble(configValues[i])));
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((Double.parseDouble(thresholdValues[i])));
                             setConfigurationChanged(true);
                         } catch (NumberFormatException e) {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
-                    } else if (antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).getValue().getClass() == String.class) {
-                        if (Utils.checkStringSubstrings(configValues[i])) {
-                            antiPatternDetector.getAntiPatternModel().getConfigurations().get(configNames[i]).setValue((configValues[i]));
+                    } else if (antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).getValue().getClass() == String.class) {
+                        if (Utils.checkStringSubstrings(thresholdValues[i])) {
+                            antiPatternDetector.getAntiPatternModel().getThresholds().get(thresholdNames[i]).setValue((thresholdValues[i]));
                             setConfigurationChanged(true);
 
                         } else {
-                            incorrectParameters.add(configNames[i]);
+                            incorrectParameters.add(thresholdNames[i]);
                         }
                     }
                 }
@@ -185 +185 @@
     @Override
     public List<AntiPattern> setErrorMessages(List<AntiPattern> antiPatterns, List<String> wrongParameters) {
         for (AntiPattern antiPattern : antiPatterns) {
-            for (Map.Entry<String, Configuration> config : antiPattern.getConfigurations().entrySet()) {
-                if (wrongParameters.contains(config.getKey())) {
-                    config.getValue().setErrorMessageShown(true);
+            for (Map.Entry<String, Threshold> threshold : antiPattern.getThresholds().entrySet()) {
+                if (wrongParameters.contains(threshold.getKey())) {
+                    threshold.getValue().setErrorMessageShown(true);
                 } else {
-                    config.getValue().setErrorMessageShown(false);
+                    threshold.getValue().setErrorMessageShown(false);
                 }
             }
         }
@@ -220 +220 @@
         String APDescription = node.get("description").asText();
         String APCatalogueFileName = node.get("catalogueFileName") != null ? node.get("catalogueFileName").asText() : null;
 
-        Map<String, Configuration> APMap = new HashMap<>();
+        Map<String, Threshold> APMap = new HashMap<>();
 
-        JsonNode array = node.get("configurations");
-        Configuration<?> tmpConfig = null;
+        JsonNode array = node.get("thresholds");
+        Threshold<?> tmpThreshold = null;
 
         for(int i = 0; i < array.size(); i++){
             JsonNode tmpNode = array.get(i);
 
-            String configName = tmpNode.get("configName").asText();
-            String configType = tmpNode.get("configType").asText();
+            String thresholdName = tmpNode.get("thresholdName").asText();
+            String thresholdType = tmpNode.get("thresholdType").asText();
 
-            JsonNode configNode = tmpNode.get("configuration");
+            JsonNode thresholdNode = tmpNode.get("threshold");
 
-            String CConfigName = configNode.get("configName").asText();
-            String CConfigPrintName = configNode.get("configPrintName").asText();
-            String CConfigDescription = configNode.get("configDescription").asText();
-            String CConfigErrorMess = configNode.get("configErrorMess").asText();
-            String CConfigValue = configNode.get("configValue").asText();
+            String tThresholdName = thresholdNode.get("thresholdName").asText();
+            String tThresholdPrintName = thresholdNode.get("thresholdPrintName").asText();
+            String tThresholdDescription = thresholdNode.get("thresholdDescription").asText();
+            String tThresholdErrorMess = thresholdNode.get("thresholdErrorMess").asText();
+            String tThresholdValue = thresholdNode.get("thresholdValue").asText();
 
-            tmpConfig = getConfiguration(configType, CConfigName, CConfigPrintName, CConfigDescription, CConfigErrorMess, CConfigValue);
+            tmpThreshold = getThreshold(thresholdType, tThresholdName, tThresholdPrintName, tThresholdDescription, tThresholdErrorMess, tThresholdValue);
 
-            APMap.put(configName, tmpConfig);
+            APMap.put(thresholdName, tmpThreshold);
         }
 
         AntiPattern newAP = new AntiPattern(APid, APPrintName, APName, APDescription, APMap, APCatalogueFileName);
@@ -249 +249 @@
         return newAP;
     }
 
-    private Configuration<?> getConfiguration(String configType, String name, String printName, String description, String errorMessage, String value){
+    private Threshold<?> getThreshold(String thresholdType, String name, String printName, String description, String errorMessage, String value){
 
-        if(configType.equals("Percentage")){
-            return new Configuration<>(name, printName, description, errorMessage, new Percentage(Float.parseFloat(value)));
+        if(thresholdType.equals("Percentage")){
+            return new Threshold<>(name, printName, description, errorMessage, new Percentage(Float.parseFloat(value)));
         }
-        else if(configType.equals("PositiveFloat")){
-            return new Configuration<>(name, printName, description, errorMessage, new PositiveFloat(Float.parseFloat(value)));
+        else if(thresholdType.equals("PositiveFloat")){
+            return new Threshold<>(name, printName, description, errorMessage, new PositiveFloat(Float.parseFloat(value)));
         }
-        else if(configType.equals("PositiveInteger")){
-            return new Configuration<>(name, printName, description, errorMessage, new PositiveInteger(Integer.parseInt(value)));
+        else if(thresholdType.equals("PositiveInteger")){
+            return new Threshold<>(name, printName, description, errorMessage, new PositiveInteger(Integer.parseInt(value)));
         }
-        else if(configType.equals("String")){
-            return new Configuration<>(name, printName, description, errorMessage, value);
+        else if(thresholdType.equals("String")){
+            return new Threshold<>(name, printName, description, errorMessage, value);
         }
 
         return null;

src/main/webapp/WEB-INF/templates/anti-pattern.html

@@ -147 +147 @@
 
         <!-- Form for configuration values -->
         <form action="#" th:action="@{/anti-patterns/} + ${antiPattern.id}" th:object="${antiPattern}" method="post">
-            <div th:each="config : ${antiPattern.configurations}" class="form-group">
-                <label th:text="${config.value.printName} + ':'"
-                       th:for="${config.value.name}"></label>
-                <small th:text="${config.value.description}" th:value="${config.value.name}"
+            <div th:each="threshold : ${antiPattern.thresholds}" class="form-group">
+                <label th:text="${threshold.value.printName} + ':'"
+                       th:for="${threshold.value.name}"></label>
+                <small th:text="${threshold.value.description}" th:value="${threshold.value.name}"
                        class="form-text text-muted"></small>
-                <input th:if="${config.value.isErrorMessageShown}" th:value="${config.value.value}" class="form-control is-invalid" th:id="${config.value.name}"
-                       name="configValues">
-                <input th:unless="${config.value.isErrorMessageShown}" th:value="${config.value.value}" class="form-control" th:id="${config.value.name}"
-                       name="configValues">
-                <input th:value="${config.value.name}" style="display: none" class="form-control"
-                       name="configNames">
-                <small th:if="${config.value.isErrorMessageShown}" th:text="${config.value.errorMessage}" th:value="${config.value.errorMessage}"
+                <input th:if="${threshold.value.isErrorMessageShown}" th:value="${threshold.value.value}" class="form-control is-invalid" th:id="${threshold.value.name}"
+                       name="thresholdValues">
+                <input th:unless="${threshold.value.isErrorMessageShown}" th:value="${threshold.value.value}" class="form-control" th:id="${threshold.value.name}"
+                       name="thresholdValues">
+                <input th:value="${threshold.value.name}" style="display: none" class="form-control"
+                       name="thresholdNames">
+                <small th:if="${threshold.value.isErrorMessageShown}" th:text="${threshold.value.errorMessage}" th:value="${threshold.value.errorMessage}"
                        class="form-text text-danger"></small>
 
             </div>

src/main/webapp/WEB-INF/templates/configuration.html

@@ -37 +37 @@
         <form action="#" th:action="@{/configuration}" th:object="${antiPatterns}" method="post">
             <div th:each="antiPattern : ${antiPatterns}">
                 <h3 th:text="${antiPattern.printName}"></h3>
-                <div th:each="config : ${antiPattern.configurations}">
+                <div th:each="threshold : ${antiPattern.thresholds}">
                     <div class="form-group row">
-                        <label th:text="${config.value.printName} + ':'" th:for="${config.value.name}"
+                        <label th:text="${threshold.value.printName} + ':'" th:for="${threshold.value.name}"
                                class="col-sm-5 col-form-label"></label>
                         <div class="col-sm-5">
-                            <small th:text="${config.value.description}" th:value="${config.value.name}"
+                            <small th:text="${threshold.value.description}" th:value="${threshold.value.name}"
                                    class="form-text text-muted"></small>
                             <div id="wrapper">
                                 <div id="first">
-                                    <input th:if="${config.value.isErrorMessageShown}" th:value="${config.value.value}"
-                                           class="form-control is-invalid" th:id="${config.value.name}"
-                                           name="configValues">
-                                    <input th:unless="${config.value.isErrorMessageShown}"
-                                           th:value="${config.value.value}"
-                                           class="form-control" th:id="${config.value.name}"
-                                           name="configValues">
-                                    <input th:value="${config.value.name}" style="display: none" class="form-control"
-                                           name="configNames">
+                                    <input th:if="${threshold.value.isErrorMessageShown}" th:value="${threshold.value.value}"
+                                           class="form-control is-invalid" th:id="${threshold.value.name}"
+                                           name="thresholdValues">
+                                    <input th:unless="${threshold.value.isErrorMessageShown}"
+                                           th:value="${threshold.value.value}"
+                                           class="form-control" th:id="${threshold.value.name}"
+                                           name="thresholdValues">
+                                    <input th:value="${threshold.value.name}" style="display: none" class="form-control"
+                                           name="thresholdNames">
                                 </div>
-                                <div th:if="${#strings.contains(config.value.name,'Substrings')}" id="second"
+                                <div th:if="${#strings.contains(threshold.value.name,'Substrings')}" id="second"
                                      style="margin: 10px; padding-left: 10px">
                                     <svg style="display: inline-block"
                                          xmlns="http://www.w3.org/2000/svg"
@@ -74 +74 @@
                                 </div>
                             </div>
 
-                            <small th:if="${config.value.isErrorMessageShown}" th:text="${config.value.errorMessage}"
-                                   th:value="${config.value.errorMessage}"
+                            <small th:if="${threshold.value.isErrorMessageShown}" th:text="${threshold.value.errorMessage}"
+                                   th:value="${threshold.value.errorMessage}"
                                    class="form-text text-danger"></small>
 
                         </div>

src/main/webapp/antipatterns/BusinessAsUsual.json

@@ -3 +3 @@
     "printName": "Business As Usual",
     "name": "BusinessAsUsual",
     "description": "Absence of a retrospective after individual iterations or after the completion project.",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "divisionOfIterationsWithRetrospective",
-            "configType": "Percentage",
-            "configuration": {
-                "configName": "divisionOfIterationsWithRetrospective",
-                "configPrintName": "Division of iterations with retrospective",
-                "configDescription": "Minimum percentage of the total number of iterations with a retrospective (0,100)",
-                "configErrorMess": "Percentage must be float number between 0 and 100",
-                "configValue": "66.66f"
+            "thresholdName": "divisionOfIterationsWithRetrospective",
+            "thresholdType": "Percentage",
+            "threshold": {
+                "thresholdName": "divisionOfIterationsWithRetrospective",
+                "thresholdPrintName": "Division of iterations with retrospective",
+                "thresholdDescription": "Minimum percentage of the total number of iterations with a retrospective (0,100)",
+                "thresholdErrorMess": "Percentage must be float number between 0 and 100",
+                "thresholdValue": "66.66f"
             }
         },
         {
-            "configName": "searchSubstringsWithRetrospective",
-            "configType": "String",
-            "configuration": {
-                "configName": "searchSubstringsWithRetrospective",
-                "configPrintName": "Search substrings with retrospective",
-                "configDescription": "Substring that will be search in wikipages and activities",
-                "configErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters || ",
-                "configValue": "%retr%||%revi%||%week%scrum%"
+            "thresholdName": "searchSubstringsWithRetrospective",
+            "thresholdType": "String",
+            "threshold": {
+                "thresholdName": "searchSubstringsWithRetrospective",
+                "thresholdPrintName": "Search substrings with retrospective",
+                "thresholdDescription": "Substring that will be search in wikipages and activities",
+                "thresholdErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters || ",
+                "thresholdValue": "%retr%||%revi%||%week%scrum%"
             }
         }
     ],

src/main/webapp/antipatterns/LongOrNonExistentFeedbackLoops.json

@@ -3 +3 @@
     "printName": "Long Or Non Existent Feedback Loops",
     "name": "LongOrNonExistentFeedbackLoops",
     "description": "Long spacings between customer feedback or no feedback. The customer enters the project and sees the final result. In the end, the customer may not get what he really wanted. With long intervals of feedback, some misunderstood functionality can be created and we have to spend a lot of effort and time to redo it. ",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "divisionOfIterationsWithFeedbackLoop",
-            "configType": "Percentage",
-            "configuration": {
-                "configName": "divisionOfIterationsWithFeedbackLoop",
-                "configPrintName": "Division of iterations with feedback loop",
-                "configDescription": "Minimum percentage of the total number of iterations with feedback loop (0,100)",
-                "configErrorMess": "Percentage must be float number between 0 and 100",
-                "configValue": "50.00f"
+            "thresholdName": "divisionOfIterationsWithFeedbackLoop",
+            "thresholdType": "Percentage",
+            "threshold": {
+                "thresholdName": "divisionOfIterationsWithFeedbackLoop",
+                "thresholdPrintName": "Division of iterations with feedback loop",
+                "thresholdDescription": "Minimum percentage of the total number of iterations with feedback loop (0,100)",
+                "thresholdErrorMess": "Percentage must be float number between 0 and 100",
+                "thresholdValue": "50.00f"
             }
         },
         {
-            "configName": "maxGapBetweenFeedbackLoopRate",
-            "configType": "PositiveFloat",
-            "configuration": {
-                "configName": "maxGapBetweenFeedbackLoopRate",
-                "configPrintName": "Maximum gap between feedback loop rate",
-                "configDescription": "Value that multiplies average iteration length for given project. Result is maximum threshold value for gap between feedback loops in days.",
-                "configErrorMess": "Maximum gap between feedback loop rate must be positive float number",
-                "configValue": "2f"
+            "thresholdName": "maxGapBetweenFeedbackLoopRate",
+            "thresholdType": "PositiveFloat",
+            "threshold": {
+                "thresholdName": "maxGapBetweenFeedbackLoopRate",
+                "thresholdPrintName": "Maximum gap between feedback loop rate",
+                "thresholdDescription": "Value that multiplies average iteration length for given project. Result is maximum threshold value for gap between feedback loops in days.",
+                "thresholdErrorMess": "Maximum gap between feedback loop rate must be positive float number",
+                "thresholdValue": "2f"
             }
         },
         {
-            "configName": "searchSubstringsWithFeedbackLoop",
-            "configType": "String",
-            "configuration": {
-                "configName": "searchSubstringsWithFeedbackLoop",
-                "configPrintName": "Search substrings with feedback loop",
-                "configDescription": "Substring that will be search in wikipages and activities",
-                "configErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
-                "configValue": "%schůz%zákazník%||%předvedení%zákazník%||%zákazn%demo%||%schůz%zadavat%||%inform%schůz%||%zákazn%||%zadavatel%"
+            "thresholdName": "searchSubstringsWithFeedbackLoop",
+            "thresholdType": "String",
+            "threshold": {
+                "thresholdName": "searchSubstringsWithFeedbackLoop",
+                "thresholdPrintName": "Search substrings with feedback loop",
+                "thresholdDescription": "Substring that will be search in wikipages and activities",
+                "thresholdErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
+                "thresholdValue": "%schůz%zákazník%||%předvedení%zákazník%||%zákazn%demo%||%schůz%zadavat%||%inform%schůz%||%zákazn%||%zadavatel%"
             }
         }
     ]

src/main/webapp/antipatterns/NinetyNinetyRule.json

@@ -3 +3 @@
     "printName": "Ninety Ninety Rule",
     "name": "NinetyNinetyRule",
     "description": "The first 90 percent of the code represents the first 90 percent of development time. The remaining 10 percent of the code represents another 90 percent of development time. Then decide on a long delay of the project compared to the original estimate. The functionality is almost done, some number is already closed and is only waiting for one activity to close, but it has been open for a long time.",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "maxDivisionRange",
-            "configType": "PositiveFloat",
-            "configuration": {
-                "configName": "maxDivisionRange",
-                "configPrintName": "Maximum ration value",
-                "configDescription": "Maximum ratio value of spent and estimated time",
-                "configErrorMess": "Ration values must be positive float number",
-                "configValue": "1.25f"
+            "thresholdName": "maxDivisionRange",
+            "thresholdType": "PositiveFloat",
+            "threshold": {
+                "thresholdName": "maxDivisionRange",
+                "thresholdPrintName": "Maximum ration value",
+                "thresholdDescription": "Maximum ratio value of spent and estimated time",
+                "thresholdErrorMess": "Ration values must be positive float number",
+                "thresholdValue": "1.25f"
             }
         },
         {
-            "configName": "maxBadDivisionLimit",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "maxBadDivisionLimit",
-                "configPrintName": "Maximum iterations thresholds",
-                "configDescription": "Maximum number of consecutive iterations where the thresholds were exceeded",
-                "configErrorMess": "Maximum number of consecutive iterations must be positive integer number",
-                "configValue": "2"
+            "thresholdName": "maxBadDivisionLimit",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "maxBadDivisionLimit",
+                "thresholdPrintName": "Maximum iterations thresholds",
+                "thresholdDescription": "Maximum number of consecutive iterations where the thresholds were exceeded",
+                "thresholdErrorMess": "Maximum number of consecutive iterations must be positive integer number",
+                "thresholdValue": "2"
             }
         }
     ]

src/main/webapp/antipatterns/RoadToNowhere.json

@@ -3 +3 @@
     "printName": "Road To Nowhere",
     "name": "RoadToNowhere",
     "description": "The project is not sufficiently planned and therefore takes place on an ad hoc basis with an uncertain outcome and deadline. There is no project plan in the project.",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "minNumberOfWikiPagesWithProjectPlan",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "minNumberOfWikiPagesWithProjectPlan",
-                "configPrintName": "Minimum number of wiki pages with project plan",
-                "configDescription": "Number of wiki pages",
-                "configErrorMess": "Minimum number of wikipages must be positive integer number",
-                "configValue": "1"
+            "thresholdName": "minNumberOfWikiPagesWithProjectPlan",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "minNumberOfWikiPagesWithProjectPlan",
+                "thresholdPrintName": "Minimum number of wiki pages with project plan",
+                "thresholdDescription": "Number of wiki pages",
+                "thresholdErrorMess": "Minimum number of wikipages must be positive integer number",
+                "thresholdValue": "1"
             }
         },
         {
-            "configName": "minNumberOfActivitiesWithProjectPlan",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "minNumberOfActivitiesWithProjectPlan",
-                "configPrintName": "Minimum number of activities with project plan",
-                "configDescription": "Number of activities",
-                "configErrorMess": "Minimum number of activities must be positive integer number",
-                "configValue": "1"
+            "thresholdName": "minNumberOfActivitiesWithProjectPlan",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "minNumberOfActivitiesWithProjectPlan",
+                "thresholdPrintName": "Minimum number of activities with project plan",
+                "thresholdDescription": "Number of activities",
+                "thresholdErrorMess": "Minimum number of activities must be positive integer number",
+                "thresholdValue": "1"
             }
         },
         {
-            "configName": "searchSubstringsWithProjectPlan",
-            "configType": "String",
-            "configuration": {
-                "configName": "searchSubstringsWithProjectPlan",
-                "configPrintName": "Search substrings with project plan",
-                "configDescription": "Substring that will be search in wikipages and activities",
-                "configErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
-                "configValue": "%plán projektu%||%project plan%||%plan project%||%projektový plán%"
+            "thresholdName": "searchSubstringsWithProjectPlan",
+            "thresholdType": "String",
+            "threshold": {
+                "thresholdName": "searchSubstringsWithProjectPlan",
+                "thresholdPrintName": "Search substrings with project plan",
+                "thresholdDescription": "Substring that will be search in wikipages and activities",
+                "thresholdErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
+                "thresholdValue": "%plán projektu%||%project plan%||%plan project%||%projektový plán%"
             }
         }
     ],

src/main/webapp/antipatterns/SpecifyNothing.json

@@ -3 +3 @@
     "printName": "Specify Nothing",
     "name": "SpecifyNothing",
     "description": "The specification is not done intentionally. Programmers are expected to work better without written specifications.",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "minNumberOfWikiPagesWithSpecification",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "minNumberOfWikiPagesWithSpecification",
-                "configPrintName": "Minimum number of wiki pages with project specification",
-                "configDescription": "Number of wiki pages",
-                "configErrorMess": "Minimum number of wikipages must be positive integer number",
-                "configValue": "1"
+            "thresholdName": "minNumberOfWikiPagesWithSpecification",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "minNumberOfWikiPagesWithSpecification",
+                "thresholdPrintName": "Minimum number of wiki pages with project specification",
+                "thresholdDescription": "Number of wiki pages",
+                "thresholdErrorMess": "Minimum number of wikipages must be positive integer number",
+                "thresholdValue": "1"
             }
         },
         {
-            "configName": "minNumberOfActivitiesWithSpecification",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "minNumberOfActivitiesWithSpecification",
-                "configPrintName": "Minimum number of activities with project specification",
-                "configDescription": "Number of activities",
-                "configErrorMess": "Minimum number of activities with project specification must be positive integer number",
-                "configValue": "1"
+            "thresholdName": "minNumberOfActivitiesWithSpecification",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "minNumberOfActivitiesWithSpecification",
+                "thresholdPrintName": "Minimum number of activities with project specification",
+                "thresholdDescription": "Number of activities",
+                "thresholdErrorMess": "Minimum number of activities with project specification must be positive integer number",
+                "thresholdValue": "1"
             }
         },
         {
-            "configName": "minAvgLengthOfActivityDescription",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "minAvgLengthOfActivityDescription",
-                "configPrintName": "Minimum average length of activity description",
-                "configDescription": "Minimum average number of character of activity description",
-                "configErrorMess": "Minimum average length of activity description must be positive integer number",
-                "configValue": "150"
+            "thresholdName": "minAvgLengthOfActivityDescription",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "minAvgLengthOfActivityDescription",
+                "thresholdPrintName": "Minimum average length of activity description",
+                "thresholdDescription": "Minimum average number of character of activity description",
+                "thresholdErrorMess": "Minimum average length of activity description must be positive integer number",
+                "thresholdValue": "150"
             }
         },
         {
-            "configName": "searchSubstringsWithProjectSpecification",
-            "configType": "String",
-            "configuration": {
-                "configName": "searchSubstringsWithProjectSpecification",
-                "configPrintName": "Search substrings with project specification",
-                "configDescription": "Substring that will be search in wikipages and activities",
-                "configErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
-                "configValue": "%dsp%||%specifikace%||%specification%||%vize%proj%||%vize%produ%"
+            "thresholdName": "searchSubstringsWithProjectSpecification",
+            "thresholdType": "String",
+            "threshold": {
+                "thresholdName": "searchSubstringsWithProjectSpecification",
+                "thresholdPrintName": "Search substrings with project specification",
+                "thresholdDescription": "Substring that will be search in wikipages and activities",
+                "thresholdErrorMess": "Maximum number of substrings is ten and must not starts and ends with characters ||",
+                "thresholdValue": "%dsp%||%specifikace%||%specification%||%vize%proj%||%vize%produ%"
             }
         }
     ],

src/main/webapp/antipatterns/TooLongSprint.json

@@ -3 +3 @@
     "printName": "Too Long Sprint",
     "name": "TooLongSprint",
     "description": "Iterations too long. (ideal iteration length is about 1-2 weeks, maximum 3 weeks). It could also be detected here if the length of the iteration does not change often (It can change at the beginning and at the end of the project, but it should not change in the already started project).",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "maxIterationLength",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "maxIterationLength",
-                "configPrintName": "Max Iteration Length",
-                "configDescription": "Maximum iteration length in days",
-                "configErrorMess": "Max iteration length must be positive integer",
-                "configValue": "21"
+            "thresholdName": "maxIterationLength",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "maxIterationLength",
+                "thresholdPrintName": "Max Iteration Length",
+                "thresholdDescription": "Maximum iteration length in days",
+                "thresholdErrorMess": "Max iteration length must be positive integer",
+                "thresholdValue": "21"
             }
         },
         {
-            "configName": "maxNumberOfTooLongIterations",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "maxNumberOfTooLongIterations",
-                "configPrintName": "Max number of foo long iterations",
-                "configDescription": "Maximum number of too long iterations in project",
-                "configErrorMess": "Max number of too long iterations must be positive integer",
-                "configValue": "0"
+            "thresholdName": "maxNumberOfTooLongIterations",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "maxNumberOfTooLongIterations",
+                "thresholdPrintName": "Max number of foo long iterations",
+                "thresholdDescription": "Maximum number of too long iterations in project",
+                "thresholdErrorMess": "Max number of too long iterations must be positive integer",
+                "thresholdValue": "0"
             }
         }
     ]

src/main/webapp/antipatterns/VaryingSprintLength.json

@@ -3 +3 @@
     "printName": "Varying Sprint Length",
     "name": "VaryingSprintLength",
     "description": "The length of the sprint changes very often. It is clear that iterations will be different lengths at the beginning and end of the project, but the length of the sprint should not change during the project.",
-    "configurations": [
+    "thresholds": [
         {
-            "configName": "maxDaysDifference",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "maxDaysDifference",
-                "configPrintName": "Max days difference",
-                "configDescription": "Maximum distance of two consecutive iterations in days",
-                "configErrorMess": "Maximum distance must be positive integer number",
-                "configValue": "7"
+            "thresholdName": "maxDaysDifference",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "maxDaysDifference",
+                "thresholdPrintName": "Max days difference",
+                "thresholdDescription": "Maximum distance of two consecutive iterations in days",
+                "thresholdErrorMess": "Maximum distance must be positive integer number",
+                "thresholdValue": "7"
             }
         },
         {
-            "configName": "maxIterationChanged",
-            "configType": "PositiveInteger",
-            "configuration": {
-                "configName": "maxIterationChanged",
-                "configPrintName": "Max number of iteration changed",
-                "configDescription": "Maximum allowed number of significant changes in iteration lengths",
-                "configErrorMess": "Maximum number of iterations changed must be positive integer number",
-                "configValue": "1"
+            "thresholdName": "maxIterationChanged",
+            "thresholdType": "PositiveInteger",
+            "threshold": {
+                "thresholdName": "maxIterationChanged",
+                "thresholdPrintName": "Max number of iteration changed",
+                "thresholdDescription": "Maximum allowed number of significant changes in iteration lengths",
+                "thresholdErrorMess": "Maximum number of iterations changed must be positive integer number",
+                "thresholdValue": "1"
             }
         }
     ]