Skip to content

Commit fffa333

Browse files
shreyanshdwivediabhinavk96
authored andcommitted
fix: event settings visible to admin and owner only (#3432)
1 parent 6a563e2 commit fffa333

File tree

3 files changed

+12
-4
lines changed

3 files changed

+12
-4
lines changed

app/routes/events/view.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ export default Route.extend({
1010

1111
model(params) {
1212
return this.store.findRecord('event', params.event_id, {
13-
include: 'event-topic,event-sub-topic,event-type,event-copyright,tax,stripe-authorization'
13+
include: 'event-topic,event-sub-topic,event-type,event-copyright,tax,owner,stripe-authorization'
1414
});
1515
},
1616

app/routes/events/view/settings.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,12 @@ export default Route.extend({
44
titleToken() {
55
return this.l10n.t('Settings');
66
},
7+
beforeModel() {
8+
let { currentUser } = this.authManager;
9+
if (!(currentUser.isAnAdmin || this.modelFor('events.view').owner.get('email') === currentUser.email)) {
10+
this.transitionTo('events.view');
11+
}
12+
},
713
async model() {
814
let eventDetails = this.modelFor('events.view');
915
return {

app/templates/events/view.hbs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,11 @@
8181
{{#link-to 'events.view.export' class='item'}}
8282
{{t 'Export'}}
8383
{{/link-to}}
84-
{{#link-to 'events.view.settings' class='item'}}
85-
{{t 'Settings'}}
86-
{{/link-to}}
84+
{{#if (or authManager.currentUser.isAnAdmin (eq model.owner.email authManager.currentUser.email))}}
85+
{{#link-to 'events.view.settings' class='item'}}
86+
{{t 'Settings'}}
87+
{{/link-to}}
88+
{{/if}}
8789
{{/tabbed-navigation}}
8890
</div>
8991
</div>

0 commit comments

Comments
 (0)