Skip to content

Commit 5888109

Browse files
committed
fix comment images
1 parent 320b511 commit 5888109

File tree

8 files changed

+45
-38
lines changed

8 files changed

+45
-38
lines changed

Diff for: app.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ if (process.env.NODE_ENV === "production") {
2121
}
2222

2323
mongoose
24-
.connect(db, { useNewUrlParser: true })
24+
.connect(db, { useNewUrlParser: true, useUnifiedTopology: true })
2525
.then(() => console.log("Connected to mongoDB"))
2626
.catch(err => console.log(error));
2727

Diff for: frontend/src/components/businesses/business_show_item.jsx

-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ const BusinessShowItem = ({ review, user, removeReview, currentUser}) => {
1111
});
1212
}
1313
}
14-
1514
let deleteButton;
1615
if (review.author === currentUser.id) {
1716
deleteButton = (
@@ -27,7 +26,6 @@ const BusinessShowItem = ({ review, user, removeReview, currentUser}) => {
2726
user.reviews.forEach(review => {
2827
review.photos.forEach(() => photoCount += 1)
2928
})
30-
3129
return (
3230
<li>
3331
<div className="review-item">

Diff for: frontend/src/components/reviews/review_form.jsx

-3
Original file line numberDiff line numberDiff line change
@@ -89,13 +89,10 @@ class ReviewForm extends React.Component{
8989
render(){
9090
let placeHolder = 'Your review helps others learn about great local businesses.\n\n Please don’t review this business if you received a freebie for writing this review, or if you’re connected in any way to the owner or employees.'
9191
return(
92-
9392
<div>
9493
<div className="search-nav">
9594
<SearchNav />
9695
</div>
97-
98-
9996
<form encType="multipart/form-data" className="review-form" onSubmit={this.handleSubmit}>
10097
<ul className="form-wrapper">
10198
<h3 className="form-header">Business Name</h3>

Diff for: frontend/src/reducers/businesses_reducer.js

-2
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,6 @@ const businessesReducer = (oldState = {}, action) => {
99
imageURLs = action.payload.imgUrls;
1010
let business = action.payload.business;
1111
let img = imageURLs.filter(item => item.includes(business._id));
12-
debugger
13-
1412
business.imgURL = img;
1513
return Object.assign({}, oldState, { [business._id]: business });
1614
case RECEIVE_BUSINESSES:

Diff for: frontend/src/reducers/reviews_reducer.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const ReviewsReducer = (state = {}, action) => {
77
switch (action.type) {
88
case RECEIVE_BUSINESS:
99
return action.payload.reviews;
10-
case RECEIVE_REVIEW:
10+
case RECEIVE_REVIEW:
1111
return Object.assign({}, state, {[action.review.data.review._id]: action.review.data.review});
1212
case REMOVE_REVIEW:
1313
newState = Object.assign({}, state);

Diff for: models/Images.js

-1
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,6 @@ function getImages(prefix) {
131131
}
132132
});
133133
})
134-
135134
}
136135

137136
module.exports = {

Diff for: routes/.DS_Store

6 KB
Binary file not shown.

Diff for: routes/api/businesses.js

+43-28
Original file line numberDiff line numberDiff line change
@@ -48,45 +48,60 @@ router.get('/user/:user_id', (req, res) => {
4848
);
4949
});
5050

51-
// get buisiness details
52-
router.get('/:id', (req, res) => {
53-
Business.findById(req.params.id)
54-
.then(business => Review.find({ business: business.id })
55-
.then(reviews => {
56-
getImages('businesses')
57-
.then(imgUrls => {
58-
const reviewsObj = {};
59-
reviews.forEach(review => reviewsObj[review.id] = review);
60-
res.send({ business: business, reviews: reviewsObj, imgUrls:imgUrls })
61-
}
62-
)})
63-
)
64-
.catch(err =>
65-
res.status(404).json({ nobusinessfound: 'No business found with that ID' })
66-
);
67-
});
51+
// get business details
52+
// router.get('/:id', (req, res) => {
53+
// Business.findById(req.params.id)
54+
// .then(business => Review.find({ business: business.id })
55+
// .then(reviews => {
56+
// getImages('businesses').
57+
// then(imgUrls => {
58+
// const reviewsObj = {};
59+
// reviews.forEach(review => reviewsObj[review.id] = review);
60+
// res.send({ business: business, reviews: reviewsObj, imgUrls:imgUrls })
61+
// }
62+
// )})
63+
// )
64+
// .catch(err =>
65+
// res.status(404).json({ nobusinessfound: 'No business found with that ID' })
66+
// );
67+
// });
68+
69+
router.get('/:id', async (req, res) => {
70+
let business = await Business.findById(req.params.id);
71+
let reviews = await Review.find({ business: business.id });
72+
let imgUrls = await getImages('businesses');
73+
const reviewsObj = {};
74+
const reviewsImages = {};
75+
for (let index = 0; index < reviews.length; index++) {
76+
const review = reviews[index]
77+
const images = await getImages(review.id);
78+
review.photos = images;
79+
reviewsImages[review.id] = images;
80+
reviewsObj[review.id] = review;
81+
}
82+
res.send({ business: business, reviews: reviewsObj, imgUrls: imgUrls, reviewImgUrls: reviewsImages })
83+
})
6884

6985
router.post('/:id/review', (req, res, next) => {
70-
const token = req.headers.authorization;
86+
const token = req.headers.authorization;
7187
const user = jwt_decode(token);
7288

7389
if (user) {
74-
const newReview = new Review ({
90+
const newReview = new Review({
7591
rate: req.body.rate,
7692
comment: req.body.comment,
7793
business: req.body.businessId,
7894
author: user.id
7995
})
80-
newReview.save().then(review => {
81-
82-
uploadMultiple(review._id, req, res)
83-
.then(data => {
96+
newReview.save().then(review => {
97+
uploadMultiple(review._id, req, res)
98+
.then(data => {
8499
getImages(review.id)
85-
.then((imgUrls)=>{
86-
review.photos = imgUrls;
87-
review.save();
88-
res.send({review, images: imgUrls})});
89-
}, (err)=> console.log(err))
100+
.then((imgUrls) => {
101+
review.save();
102+
res.send({ review, images: imgUrls })
103+
});
104+
}, (err) => console.log(err))
90105
});
91106
}
92107
})

0 commit comments

Comments
 (0)