fix(Add player): #79 email duplicate, add mongoose unique to email an…

…d display error message when email is duplicated during submission (#88)

client/src/actions/players/createPlayer.js

@@ -1,6 +1,7 @@
 import axios from 'axios';
-import { ADD_PLAYER, ADD_PLAYER_TO_TEAM } from '../types';
+import { ADD_PLAYER, ADD_PLAYER_TO_TEAM, OPEN_SNACKBAR } from '../types';
 import { ROOT_URL } from '../../../globals';
+import { openSnackbar } from '../index';
 
 
 // Add a new player to DB,
@@ -18,6 +19,8 @@ export function createPlayer(form, dispatch, props) {
 
 	axios.post(`${ROOT_URL}/player/add`, reqBody)
 		.then(({data}) => {
+			// successful message
+			openSnackbar(form, dispatch, props);
 
 			// Send new player to the playersReducer to be appended to the list
 			dispatch({ type: ADD_PLAYER, payload: data });
@@ -29,6 +32,9 @@ export function createPlayer(form, dispatch, props) {
 					payload: { teamId: team.teamId, player: data }
 				});
 			}
-
+		})
+		.catch(error => {
+			// display error message
+			dispatch({ type: OPEN_SNACKBAR, message: error.response.data.message });
 		});
 }

client/src/components/dashboard/players/forms/playerFormTemplate.jsx

@@ -156,8 +156,7 @@ function mapStateToProps(state, ownProps) {
 }
 
 PlayerFormTemplate = reduxForm({
-	validate,
-	onSubmitSuccess: openSnackbar
+	validate
 })(PlayerFormTemplate);
 
 export default connect(mapStateToProps)(PlayerFormTemplate);

package.json

@@ -27,6 +27,7 @@
     "lodash": "^4.17.4",
     "material-ui": "^0.17.4",
     "mongoose": "^4.7.8",
+    "mongoose-unique-validator": "^2.0.0",
     "nodemailer": "^4.0.1",
     "nodemon": "^1.12.1",
     "passport": "^0.3.2",

server/api/player.js

@@ -23,7 +23,9 @@ const addPlayerToLeague = (req, res) => {
 				res.send(newPlayer);
 			}
 		})
-		.catch(error => { throw error; });
+		.catch(error => { 
+			res.status(400).send({ message: error.message });
+		});
 };
 
 const getPlayer = (req, res) => {

server/models/player.js

@@ -6,6 +6,7 @@ const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;
 const capitalize = require('./plugins/capitalize');
+const mongooseUniqueValidator = require('mongoose-unique-validator');
 
 const PlayerSchema = new Schema({
 	firstName: {
@@ -18,7 +19,8 @@ const PlayerSchema = new Schema({
 	},
 	email: {
 		type: String,
-		required: true
+		required: true,
+		unique: true
 	},
 	phoneNum: {
 		type: String
@@ -68,5 +70,6 @@ PlayerSchema.virtual('fullName').get(function() {
 });
 
 PlayerSchema.plugin(capitalize, { fields });
+PlayerSchema.plugin(mongooseUniqueValidator);
 
 module.exports = mongoose.model('player', PlayerSchema);