diff --git a/src/main/java/Team5/SmartTowns/Landmarks/Landmarks.java b/src/main/java/Team5/SmartTowns/Landmarks/Landmarks.java
index 790090acf3feca3c8891be9eedb910159417beb4..ee6628f305b885fab6253be7ab50482d1cc818f2 100644
--- a/src/main/java/Team5/SmartTowns/Landmarks/Landmarks.java
+++ b/src/main/java/Team5/SmartTowns/Landmarks/Landmarks.java
@@ -25,11 +25,5 @@ public class Landmarks {
this.trailID =0;
}
-// public Landmarks(String landmarkName, String landmarkEmail, String landmarkDescription, String landmarkLocation, Integer trailID) {
-// this.landmarkName = landmarkName;
-// this.landmarkEmail = landmarkEmail;
-// this.landmarkDescription = landmarkDescription;
-// this.landmarkLocation = landmarkLocation;
-// this.trailID = trailID;
-// }
+
}
diff --git a/src/main/java/Team5/SmartTowns/Landmarks/LandmarksArray.java b/src/main/java/Team5/SmartTowns/Landmarks/LandmarksArray.java
deleted file mode 100644
index d469cc312c3d3ff50e73101ba966922f4da66ce8..0000000000000000000000000000000000000000
--- a/src/main/java/Team5/SmartTowns/Landmarks/LandmarksArray.java
+++ /dev/null
@@ -1,30 +0,0 @@
-package Team5.SmartTowns.Landmarks;
-
-import java.util.ArrayList;
-import java.util.List;
-
-public class LandmarksArray {
- private List<Landmarks> landmarksList;
- private static LandmarksArray singelton;
-
- LandmarksArray() {
- landmarksList = new ArrayList<>(); //ArrayList is mutable
- landmarksList.addAll(
- List.of(
- new Landmarks("park","asd","desc","loc",9)
- )
- );
-
-
- }
-
-
-// public static UserArray getInstance() {
-// if (singleton == null) {
-// singleton = new UserArray();
-//
-// }
-// return singleton;
-// }
-
-}
diff --git a/src/main/java/Team5/SmartTowns/Landmarks/LandmarksController.java b/src/main/java/Team5/SmartTowns/Landmarks/LandmarksController.java
index 901a32ab3229066360850c8c334d6783571cb0aa..98c35594da947d261f6980b10a516a75b4583431 100644
--- a/src/main/java/Team5/SmartTowns/Landmarks/LandmarksController.java
+++ b/src/main/java/Team5/SmartTowns/Landmarks/LandmarksController.java
@@ -13,7 +13,7 @@ import org.springframework.web.servlet.ModelAndView;
@Controller
public class LandmarksController {
-
+// Controllers for LandmarkFormTh.html landmark submission form
@GetMapping("/landmarkSubmission")
public ModelAndView landmarkSubmission(){
ModelAndView modelAndView1 = new ModelAndView("Landmarks/LandmarkFormTh.html");
@@ -33,19 +33,15 @@ public class LandmarksController {
} else{
System.out.println(landmarks);
+ // current functionality only prints successful Landmarks, (todo )database integration is necessary here
+
-// LandmarksArray userArray = LandmarksArray.getInstance();
-// Landmarks newUserSubmission = new Landmarks(landmarkData.getFormUsername(),userSubmission.getFormEmail(),
-// userSubmission.getFormPark(), userSubmission.getFormDescription(),userSubmission.getFormCafe(), userSubmission.getFormToilets());
-// userArray.addUserSubmission(newUserSubmission);
-// System.out.println(userArray.getFormItems());
-//
ModelAndView modelAndView = new ModelAndView("redirect:/test.html");
return modelAndView;
}
-// return modelAndView;
+
@@ -53,13 +49,5 @@ public class LandmarksController {
}
-// @PostMapping("/landmarkSubmission")
-// public ModelAndView landmarkSubmission( Landmarks landmarks){
-// System.out.println(landmarks);
-// ModelAndView modelAndView = new ModelAndView("redirect:/thankyou.html");
-//
-// return modelAndView;
-//
-// }
}
diff --git a/src/main/resources/static/css/landmarkFormStyle.css b/src/main/resources/static/css/landmarkFormStyle.css
index 5d55ace648577e3117dd272a654f6e624f4a70d0..2d5b27d5658cd00cf4ce9af0d3bd010adffe3e69 100644
--- a/src/main/resources/static/css/landmarkFormStyle.css
+++ b/src/main/resources/static/css/landmarkFormStyle.css
@@ -1,3 +1,4 @@
+/*LandmarkFormTh stylesheet*/
body{
background: rgb(41, 41, 41);
color: wheat;
diff --git a/src/main/resources/templates/Landmarks/LandmarkFormTh.html b/src/main/resources/templates/Landmarks/LandmarkFormTh.html
index 53430d518cfd11b37984c5939056bbf9d2d1f768..2c448b9cac8905f16d913ccccebb1f9f384b236f 100644
--- a/src/main/resources/templates/Landmarks/LandmarkFormTh.html
+++ b/src/main/resources/templates/Landmarks/LandmarkFormTh.html
@@ -7,36 +7,13 @@
<link rel="stylesheet" th:href="@{css/templatingstyle.css}">
- <style>
- /*main{*/
- /* background: rgb(41, 41, 41);*/
- /* color: wheat;*/
- /*}*/
- /*#landmarkSubmission {*/
- /* background-color: rgb(206, 153, 253);;*/
- /* color: black;*/
- /* border-color: white;*/
- /* align-content: center;*/
- /* text-align: center;*/
- /* border-radius: 25px;*/
- /* max-width: 600px;*/
-
- /* margin: 0 auto*/
-
- /*}*/
-
-
-
-
- </style>
-
</head>
<body>
<header th:insert="~{/towns/Templating.html :: header}"></header>
<hr style="height:20px; visibility:hidden;" />
<H2 id="landmarkFormTitle"> Interested in joining our trails? Sign up Here! </H2>
<main>
-
+<!-- Form used to submit potential landmarks for trails-->
<form action="/landmarkSub" id="landmarkSubmission" name="landmarkSubmission" method="post" th:object="${landmarkData}" onsubmit="return landmarkFormValidation()">
<br>
@@ -52,7 +29,6 @@
<textarea th:field="*{landmarkDescription}" rows="8" cols="60" placeholder="Max 200 words please..."></textarea>
</label><br><br>
<label>Your Location:
-<!-- <input type="text" th:field="*{landmarkLocation}">-->
<select th:field="*{landmarkLocation}">
<option value="" disabled selected>Select Location</option>
<option value="Caerphilly">Caerphilly</option>
@@ -85,7 +61,7 @@
-
+// verification function of the above form
function landmarkFormValidation(){
var pass=true;
var trail = document.forms["landmarkSubmission"]["trailID"].value
@@ -97,7 +73,7 @@
pass = false;
}
- if (trail==0){
+ if (trail==0){ // values of 0 and "" used to ensure default drop down option isn't submittable
alert('Invalid trail selected. \nPlease select the trail you wish to join.');
pass = false;
}
@@ -105,7 +81,7 @@
if (location==""){
alert('Invalid location selected. \nPlease select the location you wish to join.');
- pass = false;
+ pass = false; // ensure correct trail is selected in accordance to the town it takes place in
} if ( (location=="Caerphilly" & (parseInt(trail/100)!==1)) ||
(location=="Risca" & (parseInt(trail/100)!==2)) ||