From 8e21fc41125c8bac0dfcf1e9ae6896fce3fdf34e Mon Sep 17 00:00:00 2001
From: SamR1 <samira.rabaaoui@coding-academy.fr>
Date: Mon, 1 Jan 2018 17:50:12 +0100
Subject: [PATCH] API & Client: Profile update

---
 mpwo_api/mpwo_api/users/auth.py               | 19 ++++++++++++-
 mpwo_client/src/actions/index.js              | 20 +++++++++----
 .../src/components/User/ProfileEdit.jsx       | 28 +++++++++++++++++++
 mpwo_client/src/mpwoApi.js                    |  4 ++-
 mpwo_client/src/reducers/index.js             |  1 +
 mpwo_client/src/reducers/initial.js           |  4 ++-
 6 files changed, 68 insertions(+), 8 deletions(-)

diff --git a/mpwo_api/mpwo_api/users/auth.py b/mpwo_api/mpwo_api/users/auth.py
index d7f1df1f..68104210 100644
--- a/mpwo_api/mpwo_api/users/auth.py
+++ b/mpwo_api/mpwo_api/users/auth.py
@@ -1,5 +1,5 @@
 import datetime
-from flask import Blueprint, jsonify, request
+from flask import Blueprint, current_app, jsonify, request
 from sqlalchemy import exc, or_
 
 from mpwo_api import appLog, bcrypt, db
@@ -182,6 +182,21 @@ def edit_user(user_id):
     bio = post_data.get('bio')
     birth_date = post_data.get('birth_date')
     location = post_data.get('location')
+    password = post_data.get('password')
+    password_conf = post_data.get('password_conf')
+
+    if password is not None and password != '':
+        if password_conf != password:
+            response_object = {
+                'status': 'error',
+                'message': 'Password and password confirmation don\'t match.\n'
+            }
+            return jsonify(response_object), 400
+        else:
+            password = bcrypt.generate_password_hash(
+                password, current_app.config.get('BCRYPT_LOG_ROUNDS')
+            ).decode()
+
     try:
         user = User.query.filter_by(id=user_id).first()
         user.first_name = first_name
@@ -193,6 +208,8 @@ def edit_user(user_id):
             if birth_date
             else None
         )
+        if password is not None and password != '':
+            user.password = password
         db.session.commit()
 
         response_object = {
diff --git a/mpwo_client/src/actions/index.js b/mpwo_client/src/actions/index.js
index d7c21acd..d4fb2445 100644
--- a/mpwo_client/src/actions/index.js
+++ b/mpwo_client/src/actions/index.js
@@ -19,6 +19,14 @@ function ProfileError(message) {
   return { type: 'PROFILE_ERROR', message }
 }
 
+function PwdError(message) {
+  return { type: 'PWD_ERROR', message }
+}
+
+function initProfileFormData(user) {
+  return { type: 'INIT_PROFILE_FORM', user }
+}
+
 export const handleFormChange = (target, value) => ({
   type: 'UPDATE_USER_FORMDATA',
   target,
@@ -31,10 +39,6 @@ export const updateProfileFormData = (target, value) => ({
   value,
 })
 
-function initProfileFormData(user) {
-  return { type: 'INIT_PROFILE_FORM', user }
-}
-
 export function getProfile(dispatch) {
   return mpwoApi
     .getProfile()
@@ -147,7 +151,11 @@ export function handleProfileFormSubmit(event) {
   event.preventDefault()
   return (dispatch, getState) => {
     const state = getState()
-    return mpwoApi
+    if (state.formProfile.formProfile.password !==
+        state.formProfile.formProfile.passwordConf) {
+      dispatch(PwdError('Password and password confirmation don\'t match.'))
+    } else {
+      return mpwoApi
       .updateProfile(state.formProfile.formProfile)
       .then(ret => {
         if (ret.status === 'success') {
@@ -160,5 +168,7 @@ export function handleProfileFormSubmit(event) {
       .catch(error => {
         throw error
       })
+    }
+
   }
 }
diff --git a/mpwo_client/src/components/User/ProfileEdit.jsx b/mpwo_client/src/components/User/ProfileEdit.jsx
index 18c48e7f..78dd96f5 100644
--- a/mpwo_client/src/components/User/ProfileEdit.jsx
+++ b/mpwo_client/src/components/User/ProfileEdit.jsx
@@ -17,6 +17,7 @@ class ProfileEdit extends React.Component {
     const { formProfile,
             onHandleFormChange,
             onHandleProfileFormSubmit,
+            message,
             user
     } = this.props
     return (
@@ -24,6 +25,9 @@ class ProfileEdit extends React.Component {
         <Helmet>
           <title>mpwo - {user.username} - Edit Profile</title>
         </Helmet>
+        { message !== '' && (
+            <code>{message}</code>
+          )}
         <div className="container">
           <h1 className="page-title">Profile Edition</h1>
           <div className="row">
@@ -62,6 +66,29 @@ class ProfileEdit extends React.Component {
                           />
                         </label>
                       </div>
+                      <div className="form-group">
+                        <label>
+                          Password:
+                          <input
+                            name="password"
+                            className="form-control input-lg"
+                            type="password"
+                            onChange={onHandleFormChange}
+                          />
+                        </label>
+                      </div>
+                        <div className="form-group">
+                        <label>
+                          Password Confirmation:
+                          <input
+                            name="passwordConf"
+                            className="form-control input-lg"
+                            type="password"
+                            onChange={onHandleFormChange}
+                          />
+                        </label>
+                      </div>
+                      <hr />
                       <div className="form-group">
                         <label>
                           First Name:
@@ -146,6 +173,7 @@ class ProfileEdit extends React.Component {
 export default connect(
   state => ({
     formProfile: state.formProfile.formProfile,
+    message: state.message,
     user: state.user,
   }),
   dispatch => ({
diff --git a/mpwo_client/src/mpwoApi.js b/mpwo_client/src/mpwoApi.js
index 207fbb72..0977e4ba 100644
--- a/mpwo_client/src/mpwoApi.js
+++ b/mpwo_client/src/mpwoApi.js
@@ -58,7 +58,9 @@ export default class MpwoApi {
         last_name: form.lastName,
         bio: form.bio,
         location: form.location,
-        birth_date: form.birthdate,
+        birth_date: form.birthDate,
+        password: form.password,
+        password_conf: form.passwordConf,
       }),
     })
     return fetch(request)
diff --git a/mpwo_client/src/reducers/index.js b/mpwo_client/src/reducers/index.js
index 89b7605f..acedc4a4 100644
--- a/mpwo_client/src/reducers/index.js
+++ b/mpwo_client/src/reducers/index.js
@@ -49,6 +49,7 @@ const message = (state = initial.message, action) => {
   switch (action.type) {
     case 'AUTH_ERROR':
     case 'PROFILE_ERROR':
+    case 'PWD_ERROR':
       return action.message
     case 'LOGOUT':
     case 'PROFILE_SUCCESS':
diff --git a/mpwo_client/src/reducers/initial.js b/mpwo_client/src/reducers/initial.js
index 3cf5f5e2..7ebd6bad 100644
--- a/mpwo_client/src/reducers/initial.js
+++ b/mpwo_client/src/reducers/initial.js
@@ -28,7 +28,9 @@ export default {
       lastName: '',
       bio: '',
       location: '',
-      birthDate: ''
+      birthDate: '',
+      password: '',
+      passwordConf: ''
     }
   },
 }