Added some comments and removed redundant database columns

src/main/java/Team5/SmartTowns/security/SecurityConfiguration.java

@@ -17,6 +17,7 @@ import javax.sql.DataSource;
 @Configuration
 @EnableWebSecurity
 public class SecurityConfiguration {
+    /* Configures the longin features and tracks logged on users on the page */
 
 
     @Bean
@@ -36,6 +37,7 @@ public class SecurityConfiguration {
     }
     @Bean
     public PasswordEncoder passwordEncoder(){
+        /* Nothing here yet, this just saves passwords in plaintext. TODO password encryption */
         return NoOpPasswordEncoder.getInstance();
     }
 

src/main/java/Team5/SmartTowns/users/UserController.java

@@ -69,7 +69,6 @@ public class UserController {
         System.out.println(user.getUsername());
         mav.addObject("user", userRepository.findUserByName("Admin"));
         mav.addAllObjects(getPackInfo("Admin", 1).getModelMap());
-
         return mav;
     }
 
@@ -82,7 +81,6 @@ public class UserController {
         mav.addObject("user", userRepository.findUserByName("Admin"));
         mav.addObject("packs", allPacks);
         mav.addAllObjects(getPackInfo(username, 1).getModelMap());
-
         return mav;
     }
 

src/main/resources/schema.sql

@@ -37,10 +37,10 @@ create table if not exists locations
 
 CREATE TABLE IF NOT EXISTS users (
     username varchar(30) primary key NOT NULL,
+    id bigint auto_increment unique,
     email varchar(128),
     password varchar(30) NOT NULL,
-    enabled boolean default true,
-    roles varchar(128)
+    enabled boolean default true
 );
 
 CREATE TABLE IF NOT EXISTS authorities (