Merge pull request #19 from CSC207-2022F-UofT/4-feature-6-user-attribute-modification

parmism · web-flow · commit b729dd4002da · 2022-11-20T14:06:16.000-05:00
User attribute modificatin pull request
diff --git a/src/main/java/interface_adapters/profile_modification_IA/ChangeController.java b/src/main/java/interface_adapters/profile_modification_IA/ChangeController.java
@@ -0,0 +1,5 @@
+package interface_adapters.profile_modification_IA;
+
+public interface ChangeController {
+    public boolean reportChange(String username, String password, String feature, String newFeature);
+    }
diff --git a/src/main/java/screens/profile_update_screen/UserModificationUI.java b/src/main/java/screens/profile_update_screen/UserModificationUI.java
@@ -0,0 +1,97 @@
+package screens.profile_update_screen; /**
+ * Provides the UI elements
+ */
+import interface_adapters.profile_modification_IA.ChangeController;
+
+import javax.swing.*;
+import java.awt.*;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
+
+public class UserModificationUI implements ChangeController {
+    private JLabel label;
+    public UserModificationUI() {
+        final JFrame frame = new JFrame();
+        frame.setSize(500, 300);
+
+        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
+        frame.setLayout(new FlowLayout());
+
+//      Field for username
+        final JTextField usernameField = new JTextField("Enter your current username");
+        usernameField.setBounds(10, 25, 100, 25);
+
+//      Field for password
+        final JTextField passwordField = new JTextField("Enter your current password");
+        passwordField.setBounds(10, 25, 100, 25);
+
+//      to select feature that user wants to change
+        JLabel lbl = new JLabel("<html>Select what feature<br>you wish to change</html>");
+        lbl.setVisible(true);
+        lbl.setBounds(10, 25, 100, 25);
+        String[] choices = { "Username","Password", "Email"};
+        final JComboBox<String> cb = new JComboBox<>(choices);
+
+//      new value of the feature
+        final JTextField newFeatureField = new JTextField("Enter the new value for this feature and click OK");
+        newFeatureField.setBounds(10, 25, 100, 25);
+
+
+//      OK button to process
+        JButton btn = new JButton("OK");
+
+        btn.addActionListener(new ActionListener() {
+            @Override
+            public void actionPerformed(ActionEvent e) {
+                String username = (usernameField.getText());
+                String password = (passwordField.getText());
+                String item = cb.getSelectedItem().toString();
+                String newFeature = (newFeatureField.getText());
+                boolean success = reportChange(username, password, item, newFeature);
+                if (success){
+                    label.setText("Your " + item + " was successfully changed.");
+                }
+                else{
+                    label.setText("<html>Change was not successful.<br>Please make sure your username and " +
+                            "<br>password match an existing account.</html>");
+                }
+            }
+        });
+
+        label = new JLabel();
+
+        frame.add(usernameField);
+        frame.add(passwordField);
+        frame.add(cb);
+        frame.add(lbl);
+        frame.add(newFeatureField);
+        frame.add(btn);
+        frame.add(label);
+
+
+        frame.setVisible(true);
+    }
+
+//      profile_modification_IA.ChangeController makes UI implement reportChange to invert the use-case --> UI dependency
+    @Override
+    public boolean reportChange(String username, String password, String feature, String newFeature) {
+        UserDatabase db = new UserDatabase();
+        if (db.UserExists(username)){
+            User user = db.getUser(username);
+            if (user.PasswordMatch(password) & user.getUsername().equals(username)){
+                user.changeFeature(feature, newFeature);
+                // this serializes the change
+                db.modifyUser(username, user);
+                return true;
+            }
+        }
+        return false;
+    }
+
+// for trying out the code:
+//    public static void main(String[] args) {
+//        new profile_update_screen.UserModificationUI();
+//
+//    }
+
+}
diff --git a/src/main/java/tutorial/HelloWorld.java b/src/main/java/tutorial/HelloWorld.java
diff --git a/src/main/java/use_cases/user_attribute_modification_use_case/Changeable.java b/src/main/java/use_cases/user_attribute_modification_use_case/Changeable.java
@@ -0,0 +1,5 @@
+package use_cases.user_attribute_modification_use_case;
+
+public interface Changeable {
+    public void changeFeature(String feature, String newFeature);
+}
diff --git a/src/test/java/TestUserChange.java b/src/test/java/TestUserChange.java
@@ -0,0 +1,14 @@
+import org.junit.jupiter.api.Assertions;
+import screens.profile_update_screen.UserModificationUI;
+
+public class TestUserChange {
+    public void correctChange() {
+        UserDatabase db = new UserDatabase();
+        db.createUser("parmism", "123", "parmis@gmail.com", "Basic");
+        UserModificationUI ui = new UserModificationUI();
+        ui.reportChange("parmism", "123", "Email", "parmis@yahoo.com");
+        UserDatabase db2 = new UserDatabase();
+        String actual = db2.getUser("parmism").getEmail();
+        Assertions.assertEquals("parmis@yahoo.com", actual);
+    }
+}
diff --git a/src/test/java/tutorial/HelloWorldTest.java b/src/test/java/tutorial/HelloWorldTest.java
