Skip to content

Commit 8cb594e

Browse files
committed
only allow owner of message to edit/delete
1 parent 52d24d0 commit 8cb594e

File tree

3 files changed

+22
-15
lines changed

3 files changed

+22
-15
lines changed

src/components/Messages/MessageItem.js

+19-15
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class MessageItem extends Component {
2828
};
2929

3030
render() {
31-
const { message, onRemoveMessage } = this.props;
31+
const { authUser, message, onRemoveMessage } = this.props;
3232
const { editMode, editText } = this.state;
3333

3434
return (
@@ -46,22 +46,26 @@ class MessageItem extends Component {
4646
</span>
4747
)}
4848

49-
{editMode ? (
49+
{authUser.uid === message.userId && (
5050
<span>
51-
<button onClick={this.onSaveEditText}>Save</button>
52-
<button onClick={this.onToggleEditMode}>Reset</button>
53-
</span>
54-
) : (
55-
<button onClick={this.onToggleEditMode}>Edit</button>
56-
)}
51+
{editMode ? (
52+
<span>
53+
<button onClick={this.onSaveEditText}>Save</button>
54+
<button onClick={this.onToggleEditMode}>Reset</button>
55+
</span>
56+
) : (
57+
<button onClick={this.onToggleEditMode}>Edit</button>
58+
)}
5759

58-
{!editMode && (
59-
<button
60-
type="button"
61-
onClick={() => onRemoveMessage(message.uid)}
62-
>
63-
Delete
64-
</button>
60+
{!editMode && (
61+
<button
62+
type="button"
63+
onClick={() => onRemoveMessage(message.uid)}
64+
>
65+
Delete
66+
</button>
67+
)}
68+
</span>
6569
)}
6670
</li>
6771
);

src/components/Messages/MessageList.js

+2
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,15 @@ import React from 'react';
33
import MessageItem from './MessageItem';
44

55
const MessageList = ({
6+
authUser,
67
messages,
78
onEditMessage,
89
onRemoveMessage,
910
}) => (
1011
<ul>
1112
{messages.map(message => (
1213
<MessageItem
14+
authUser={authUser}
1315
key={message.uid}
1416
message={message}
1517
onEditMessage={onEditMessage}

src/components/Messages/Messages.js

+1
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ class Messages extends Component {
104104

105105
{messages && (
106106
<MessageList
107+
authUser={authUser}
107108
messages={messages}
108109
onEditMessage={this.onEditMessage}
109110
onRemoveMessage={this.onRemoveMessage}

0 commit comments

Comments
 (0)