Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Bug | Fix Update Item #22

Open
saurookadook wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Bug | Fix Update Item #22

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fbf9f15
Change props.value to original._id
saurookadook Feb 9, 2021
3aa25b2
Fix update button and updateItem endpoint, rename ItemsList component
saurookadook Feb 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions client/src/pages/ItemUpdate.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,16 @@ const InputText = styled.input.attrs({
const Fieldset = styled.fieldset.attrs({
className: 'form-control',
})`
background-color: transparent;
border-color: transparent;
margin: 1em auto 0.5em;
max-width: 50%;
min-height: 6em;

@media screen and (max-width: 420px) {
height: auto;
max-width: 75%;
}
`;

const DayInput = styled.input.attrs({
Expand Down
6 changes: 3 additions & 3 deletions client/src/pages/ItemsList.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ const Table = ({ columns, data }) => {
)
};

class ItemsTable extends Component {
class ItemsList extends Component {

componentDidMount() {
console.log("ItemsList: props");
Expand Down Expand Up @@ -186,7 +186,7 @@ class ItemsTable extends Component {
return (
<Link
data-update-id={original._id}
to={`/item/update/${props.value}`}
to={`/item/update/${original._id}`}
>
Update Item
</Link>
Expand Down Expand Up @@ -237,4 +237,4 @@ const mapStateToProps = state => {

const mapDispatchToProps = dispatch => bindActionCreators(actions, dispatch);

export default connect(mapStateToProps, mapDispatchToProps)(ItemsTable);
export default connect(mapStateToProps, mapDispatchToProps)(ItemsList);
19 changes: 10 additions & 9 deletions server/controllers/item-controller.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,8 @@ getItemById = async (req, res) => {
});
}
console.log(`[Hack.Diversity React Template] - 200 in 'getItemById': Item fetched!`);
console.log('----------------------- getItemById: res -----------------------');
console.log(res);
return res
.status(200)
.json({
Expand Down Expand Up @@ -133,10 +135,12 @@ createItem = (req, res) => {
});
};

updateItem = (req, res) => {
const body = req.body;
updateItem = async (req, res) => {
// console.log('----------------------- updateItem: req -----------------------');
// console.log(req);
// console.log('----------------------- updateItem: res -----------------------');
// console.log(res);
const body = req.body;
// console.log('----------------------- updateItem: body -----------------------');
// console.log(body);
if (!body) {
Expand All @@ -158,10 +162,7 @@ updateItem = (req, res) => {
content: body.content,
};

// console.log('----------------------- updateItem: res -----------------------');
// console.log(res);

return Item.updateOne({ _id: req.params.id }, itemForUpdate, (err, writeOpRes) => {
await Item.updateOne({ _id: req.params.id }, itemForUpdate, (err, writeOpRes) => {
if (err) {
console.error(`[Hack.Diversity React Template] - 404 in 'updateItem': Item not found!`);
console.error(err);
Expand All @@ -178,8 +179,8 @@ updateItem = (req, res) => {
// console.log(item);
return writeOpRes;
})
.then(res => {
// console.log('----------------------- updateItem - findOne: res -----------------------');
.then(result => {
// console.log('----------------------- updateItem - updateOne: res -----------------------');
// console.log(res);
console.log(`[Hack.Diversity React Template] - 200 in 'updateItem': Item updated!`);
return res
Expand All @@ -188,7 +189,7 @@ updateItem = (req, res) => {
success: true,
id: req.params.id,
message: 'Item updated!',
writeOpResult: res
writeOpResult: result
});
}).catch(err => {
console.error(`[Hack.Diversity React Template] - caught error in 'updateItem': ${err}`);
Expand Down