Revize 29dfa1d9

Show detection details on result page

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/detecting/detectors/LongOrNonExistentFeedbackLoopsDetectorImpl.java

@@ -4 +4 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.AntiPattern;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.Project;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.QueryResultItem;
+import cz.zcu.fav.kiv.antipatterndetectionapp.model.ResultDetail;
+import cz.zcu.fav.kiv.antipatterndetectionapp.utils.Utils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -85 +87 @@
         }
         int minFeedbackLimit =  totalNumberIterations.intValue() - Math.round(totalNumberIterations * DIVISION_LONG_OR_NON_FEEDBACK_ITERATIONS);
 
+        List<ResultDetail> resultDetails = Utils.createResultDetailsList(
+                new ResultDetail("Project id", project.getId().toString()));
+
         if ((totalNumberIterations.intValue() != iterationsResults.size())) {
             if (totalNumberIterations - iterationsResults.size() > minFeedbackLimit) {
-                return new QueryResultItem(this.antiPattern, true , null);
+                return new QueryResultItem(this.antiPattern, true , resultDetails);
             } else {
-                return new QueryResultItem(this.antiPattern, false , null);
+                return new QueryResultItem(this.antiPattern, false , resultDetails);
 
             }
         } else {
-            return new QueryResultItem(this.antiPattern, false , null);
+            return new QueryResultItem(this.antiPattern, false , resultDetails);
         }
     }
 }

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/detecting/detectors/RoadToNowhereDetectorImpl.java

@@ -4 +4 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.AntiPattern;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.Project;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.QueryResultItem;
+import cz.zcu.fav.kiv.antipatterndetectionapp.model.ResultDetail;
+import cz.zcu.fav.kiv.antipatterndetectionapp.utils.Utils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -66 +68 @@
         } catch (SQLException e) {
             e.printStackTrace();
         }
+        List<ResultDetail> resultDetails = Utils.createResultDetailsList(
+                new ResultDetail("Project id", project.getId().toString()));
 
         if( numberOfIssuesForProjectPlan >= this.MINIMUM_NUMBER_OF_ACTIVITIES || numberOfWikiPagesForProjectPlan >= this.MINIMUM_NUMBER_OF_WIKI_PAGES) {
 
-            return new QueryResultItem(this.antiPattern, false, null);
+            return new QueryResultItem(this.antiPattern, false, resultDetails);
         } else {
-            return new QueryResultItem(this.antiPattern, true, null);
+            return new QueryResultItem(this.antiPattern, true, resultDetails);
         }
     }
 }

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/detecting/detectors/SpecifyNothingDetectorImpl.java

@@ -4 +4 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.AntiPattern;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.Project;
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.QueryResultItem;
+import cz.zcu.fav.kiv.antipatterndetectionapp.model.ResultDetail;
+import cz.zcu.fav.kiv.antipatterndetectionapp.utils.Utils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -78 +80 @@
         LOGGER.info("wiki:" + numberOfWikiPages);
         LOGGER.info("average:" + averageLengthOfIssueDescription);
 
+        List<ResultDetail> resultDetails = Utils.createResultDetailsList(
+                new ResultDetail("Project id", project.getId().toString()));
+
         if (numberOfActivitiesForSpecification >= MINIMUM_NUMBER_OF_ACTIVITIES ||
                 numberOfWikiPages >= MINIMUM_NUMBER_OF_WIKI_PAGES) {
-            return new QueryResultItem(this.antiPattern, false, null);
+            return new QueryResultItem(this.antiPattern, false, resultDetails);
         } else {
             if (averageLengthOfIssueDescription > MINIMUM_AVERAGE_LENGTH_OF_ACTIVITY_DESCRIPTION) {
-                return new QueryResultItem(this.antiPattern, false, null);
+                return new QueryResultItem(this.antiPattern, false, resultDetails);
             } else {
-                return new QueryResultItem(this.antiPattern, true, null);
+                return new QueryResultItem(this.antiPattern, true, resultDetails);
             }
         }
     }

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

@@ -46 +46 @@
     public void setResultDetails(List<ResultDetail> resultDetails) {
         this.resultDetails = resultDetails;
     }
+
+    public String resultDetailsToString() {
+        StringBuilder result = new StringBuilder();
+        for (ResultDetail resultDetails :
+                resultDetails) {
+            result.append(resultDetails.getResultDetailName()).append(" = ").append(resultDetails.getResultDetailValue()).append("<br>");
+        }
+        return result.toString();
+    }
 }

src/main/resources/templates/result.html

@@ -4 +4 @@
     <title>Anti Pattern Detector - Results</title>
     <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
     <meta name="viewport" content="width=device-width, initial-scale=1">
-    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css">
-    <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js"></script>
-    <script src="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/js/bootstrap.min.js"></script>
+    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.5.2/css/bootstrap.min.css">
     <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js"></script>
+    <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.16.0/umd/popper.min.js"></script>
+    <script src="https://maxcdn.bootstrapcdn.com/bootstrap/4.5.2/js/bootstrap.min.js"></script>
 
     <style>
         .not-found-bg {
@@ -46 +46 @@
             <td class="align-middle" th:each="resultItems: ${queryResult.getQueryResultItems()}">
 
                 <div th:if="${resultItems.isDetected()}">
-                    <svg xmlns="http://www.w3.org/2000/svg" width="25" height="25"
-                         fill="currentColor" class="bi bi-check my-center found-bg icon-style" viewBox="0 0 16 16">
-                        <path d="M10.97 4.97a.75.75 0 0 1 1.07 1.05l-3.99 4.99a.75.75 0 0 1-1.08.02L4.324 8.384a.75.75 0 1 1 1.06-1.06l2.094 2.093 3.473-4.425a.267.267 0 0 1 .02-.022z"/>
-                    </svg>
+                    <div class="container">
+                        <a data-toggle="popover" title="Detection details" data-html="true" data-content="" th:attr="data-content=${resultItems.resultDetailsToString()}">
+                            <svg xmlns="http://www.w3.org/2000/svg" width="25" height="25"
+                                fill="currentColor" class="bi bi-check my-center found-bg icon-style" viewBox="0 0 16 16">
+                                <path d="M10.97 4.97a.75.75 0 0 1 1.07 1.05l-3.99 4.99a.75.75 0 0 1-1.08.02L4.324 8.384a.75.75 0 1 1 1.06-1.06l2.094 2.093 3.473-4.425a.267.267 0 0 1 .02-.022z"/>
+                            </svg>
+                        </a>
+                    </div>
                 </div>
                 <div th:unless="${resultItems.isDetected()}">
-                    <svg xmlns="http://www.w3.org/2000/svg" width="25" height="25" fill="currentColor" class="bi bi-x not-found-bg icon-style" viewBox="0 0 16 16">
-                        <path d="M4.646 4.646a.5.5 0 0 1 .708 0L8 7.293l2.646-2.647a.5.5 0 0 1 .708.708L8.707 8l2.647 2.646a.5.5 0 0 1-.708.708L8 8.707l-2.646 2.647a.5.5 0 0 1-.708-.708L7.293 8 4.646 5.354a.5.5 0 0 1 0-.708z"/>
-                    </svg>
+                    <div class="container">
+                        <a data-toggle="popover" title="Detection details" data-html="true" data-content="" th:attr="data-content=${resultItems.resultDetailsToString()}">
+                            <svg xmlns="http://www.w3.org/2000/svg" width="25" height="25" fill="currentColor" class="bi bi-x not-found-bg icon-style" viewBox="0 0 16 16">
+                            <path d="M4.646 4.646a.5.5 0 0 1 .708 0L8 7.293l2.646-2.647a.5.5 0 0 1 .708.708L8.707 8l2.647 2.646a.5.5 0 0 1-.708.708L8 8.707l-2.646 2.647a.5.5 0 0 1-.708-.708L7.293 8 4.646 5.354a.5.5 0 0 1 0-.708z"/>
+                        </svg>
+                        </a>
+                    </div>
                 </div>
             </td>
         </tr>
@@ -78 +86 @@
     </div>
 </div>
 
+
+<script>
+    $(document).ready(function(){
+        $('[data-toggle="popover"]').popover();
+    });
+</script>
 </body>
 </html>