changes to grant creation and model clean

app/controllers/donor/grant.controller.js

@@ -1,35 +1,44 @@
 const db = require('../../config/db.config.js'),
 	errorMaker = require('../../helpers/error.maker');
 
-const { Grant, Cause, Region, Organization } = db;
+const { Grant, Cause, Region, Organization, sequelize } = db;
 
 // Create a grant for certain donor
 exports.create = (req, res, next) => {
-	const donor_id = req.params.donor_id;
+	const donor_id = req.user.id;
 	const { name, amount, monthly, causes, regions, organizations } = req.body;
 
-	Grant.create({
+	return sequelize
-		donor_id,
+		.transaction(function(t) {
-		name,
+			return Grant.create(
-		amount,
+				{
-		monthly,
+					donor_id,
-		num_causes: causes.length,
+					name,
-		num_regions: regions.length
+					amount,
-	})
+					monthly,
-		.then(grant => {
+					num_causes: causes.length,
-			return Promise.all([
+					num_regions: regions.length
-				grant.addCauses(causes),
+				},
-				grant.addRegions(regions),
+				{ transaction: t }
-				grant.addOrganizations(organizations)
+			).then(grant => {
-			]).then(result => result);
+				return Promise.all([
+					grant.addCauses(causes, { transaction: t }),
+					grant.addRegions(regions, { transaction: t }),
+					grant.addOrganizations(organizations, { transaction: t })
+				]).then(result => grant);
+			});
 		})
-		.then(result => {
+		.then(function(grant) {
+			// transaction committed
 			res.status(201).json({
-				result,
+				grant,
 				message: 'Grant Created'
 			});
 		})
-		.catch(error => next(error));
+		.catch(function(error) {
+			// transaction rollback
+			next(error);
+		});
 };
 
 // Find grants with causes, regions, and organizations by donor_id

app/controllers/donor/organization.controller.js

@@ -20,7 +20,7 @@ exports.findSuggested = (req, res, next) => {
 
 	const QUERY =
 		'SELECT * from organizations ' +
-		'WHERE primary_cause IN (:causes) and primary_region IN (:regions) ' +
+		'WHERE primary_cause IN (:causes) or primary_region IN (:regions) ' +
 		'LIMIT :max_orgs';
 	db.sequelize
 		.query(QUERY, {

app/models/grant.model.js

@@ -14,10 +14,6 @@ module.exports = (sequelize, DataTypes) => {
 			type: DataTypes.INTEGER,
 			allowNull: false
 		},
-		last_payment: {
-			type: DataTypes.DATE,
-			default: new Date()
-		},
 		monthly: {
 			type: DataTypes.BOOLEAN,
 			allowNull: false,

app/routes/donor.route.js

@@ -20,11 +20,7 @@ router.get('/grants/', checkAuth(roles.DONOR), controllers.grant.findByDonorId);
  * - monthly true or false
  * - donor_id
  * */
-router.post(
+router.post('/grants/', checkAuth(roles.DONOR), controllers.grant.create);
-	'/grants/:donor_id',
-	checkAuth(roles.DONOR),
-	controllers.grant.create
-);
 
 // DELECT a grant of a donor
 router.delete(