Skip to content

Commit 8fa0556

Browse files
committed
1 parent a5ec916 commit 8fa0556

File tree

8 files changed

+48
-41
lines changed

8 files changed

+48
-41
lines changed

book/6-end/server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,11 @@ class BookClass {
8484
modifier.slug = await generateSlug(this, name);
8585
}
8686

87-
await this.updateOne({ _id: id }, { $set: modifier });
88-
89-
const editedBook = await this.findById(id, 'slug');
87+
const editedBook = await this.findOneAndUpdate(
88+
{ _id: id },
89+
{ $set: modifier },
90+
{ fields: 'slug', new: true },
91+
);
9092

9193
return editedBook;
9294
}

book/7-end/server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,11 @@ class BookClass {
8484
modifier.slug = await generateSlug(this, name);
8585
}
8686

87-
await this.updateOne({ _id: id }, { $set: modifier });
88-
89-
const editedBook = await this.findById(id, 'slug');
87+
const editedBook = await this.findOneAndUpdate(
88+
{ _id: id },
89+
{ $set: modifier },
90+
{ fields: 'slug', new: true },
91+
);
9092

9193
return editedBook;
9294
}

book/7-start/server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,11 @@ class BookClass {
8484
modifier.slug = await generateSlug(this, name);
8585
}
8686

87-
await this.updateOne({ _id: id }, { $set: modifier });
88-
89-
const editedBook = await this.findById(id, 'slug');
87+
const editedBook = await this.findOneAndUpdate(
88+
{ _id: id },
89+
{ $set: modifier },
90+
{ fields: 'slug', new: true },
91+
);
9092

9193
return editedBook;
9294
}

book/8-end/pages/admin/index.js

Lines changed: 11 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@ import notify from '../../lib/notifier';
88

99
import withLayout from '../../lib/withLayout';
1010
import withAuth from '../../lib/withAuth';
11-
import {
12-
getBookList,
13-
} from '../../lib/api/admin';
14-
11+
import { getBookList } from '../../lib/api/admin';
1512

1613
const Index = ({ books }) => (
1714
<div style={{ padding: '10px 45px' }}>
@@ -22,27 +19,26 @@ const Index = ({ books }) => (
2219
</Link>
2320
<p />
2421
<ul>
25-
{books.map(b => (
22+
{books.map((b) => (
2623
<li key={b._id}>
27-
<Link
28-
as={`/admin/book-detail/${b.slug}`}
29-
href={`/admin/book-detail?slug=${b.slug}`}
30-
>
24+
<Link as={`/admin/book-detail/${b.slug}`} href={`/admin/book-detail?slug=${b.slug}`}>
3125
<a>{b.name}</a>
3226
</Link>
3327
</li>
34-
))}
28+
))}
3529
</ul>
3630
<br />
3731
</div>
3832
</div>
3933
);
4034

4135
Index.propTypes = {
42-
books: PropTypes.arrayOf(PropTypes.shape({
43-
name: PropTypes.string.isRequired,
44-
slug: PropTypes.string.isRequired,
45-
})).isRequired,
36+
books: PropTypes.arrayOf(
37+
PropTypes.shape({
38+
name: PropTypes.string.isRequired,
39+
slug: PropTypes.string.isRequired,
40+
}),
41+
).isRequired,
4642
};
4743

4844
class IndexWithData extends React.Component {
@@ -60,11 +56,7 @@ class IndexWithData extends React.Component {
6056
}
6157

6258
render() {
63-
return (
64-
<Index
65-
{...this.state}
66-
/>
67-
);
59+
return <Index {...this.state} />;
6860
}
6961
}
7062

book/8-end/pages/customer/my-books.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,13 @@ import withAuth from '../../lib/withAuth';
99

1010
class MyBooks extends React.Component {
1111
static propTypes = {
12-
purchasedBooks: PropTypes.arrayOf(PropTypes.shape({
13-
name: PropTypes.string.isRequired,
14-
})),
12+
purchasedBooks: PropTypes.arrayOf(
13+
PropTypes.shape({
14+
name: PropTypes.string.isRequired,
15+
}),
16+
),
1517
};
18+
1619
static defaultProps = {
1720
purchasedBooks: [],
1821
};
@@ -45,7 +48,7 @@ class MyBooks extends React.Component {
4548
<div>
4649
<h3>Your books</h3>
4750
<ul>
48-
{purchasedBooks.map(book => (
51+
{purchasedBooks.map((book) => (
4952
<li key={book._id}>
5053
<Link
5154
as={`/books/${book.slug}/introduction`}

book/8-end/server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,11 @@ class BookClass {
9797
modifier.slug = await generateSlug(this, name);
9898
}
9999

100-
await this.updateOne({ _id: id }, { $set: modifier });
101-
102-
const editedBook = await this.findById(id, 'slug');
100+
const editedBook = await this.findOneAndUpdate(
101+
{ _id: id },
102+
{ $set: modifier },
103+
{ fields: 'slug', new: true },
104+
);
103105

104106
return editedBook;
105107
}

book/8-start/server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,11 @@ class BookClass {
8484
modifier.slug = await generateSlug(this, name);
8585
}
8686

87-
await this.updateOne({ _id: id }, { $set: modifier });
88-
89-
const editedBook = await this.findById(id, 'slug');
87+
const editedBook = await this.findOneAndUpdate(
88+
{ _id: id },
89+
{ $set: modifier },
90+
{ fields: 'slug', new: true },
91+
);
9092

9193
return editedBook;
9294
}

server/models/Book.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -117,9 +117,11 @@ class BookClass {
117117
modifier.slug = await generateSlug(this, name);
118118
}
119119

120-
await this.updateOne({ _id: id }, { $set: modifier });
121-
122-
const editedBook = await this.findById(id, 'slug');
120+
const editedBook = await this.findOneAndUpdate(
121+
{ _id: id },
122+
{ $set: modifier },
123+
{ fields: 'slug', new: true },
124+
);
123125

124126
return editedBook;
125127
}

0 commit comments

Comments
 (0)