Skip to content

Commit fe7f098

Browse files
authored
DEV: attemps to limit Discourse.User.current() usage (discourse#7943)
1 parent 0603636 commit fe7f098

File tree

14 files changed

+19
-42
lines changed

14 files changed

+19
-42
lines changed

app/assets/javascripts/admin/controllers/admin-backups-index.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export default Ember.Controller.extend({
2929
I18n.t("yes_value"),
3030
confirmed => {
3131
if (confirmed) {
32-
Discourse.User.currentProp("hideReadOnlyAlert", true);
32+
this.set("currentUser.hideReadOnlyAlert", true);
3333
this._toggleReadOnlyMode(true);
3434
}
3535
}

app/assets/javascripts/admin/controllers/admin-user-index.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ export default Ember.Controller.extend(CanCheckEmails, {
134134
return this.model.resetBounceScore();
135135
},
136136
approve() {
137-
return this.model.approve();
137+
return this.model.approve(this.currentUser);
138138
},
139139
deactivate() {
140140
return this.model.deactivate();

app/assets/javascripts/admin/models/admin-user.js.es6

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -227,14 +227,14 @@ const AdminUser = Discourse.User.extend({
227227
.catch(popupAjaxError);
228228
},
229229

230-
approve() {
230+
approve(approvedBy) {
231231
return ajax(`/admin/users/${this.id}/approve`, {
232232
type: "PUT"
233233
}).then(() => {
234234
this.setProperties({
235235
can_approve: false,
236236
approved: true,
237-
approved_by: Discourse.User.current()
237+
approved_by: approvedBy
238238
});
239239
});
240240
},

app/assets/javascripts/admin/routes/admin-users-list.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export default Discourse.Route.extend({
1111
},
1212

1313
sendInvites() {
14-
this.transitionTo("userInvited", Discourse.User.current());
14+
this.transitionTo("userInvited", this.currentUser);
1515
},
1616

1717
deleteUser(user) {

app/assets/javascripts/discourse/components/composer-editor.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,7 @@ export default Ember.Component.extend({
237237
reason = I18n.t("composer.error.post_missing");
238238
} else if (missingReplyCharacters > 0) {
239239
reason = I18n.t("composer.error.post_length", { min: minimumPostLength });
240-
const tl = Discourse.User.currentProp("trust_level");
240+
const tl = this.get("currentUser.trust_level");
241241
if (tl === 0 || tl === 1) {
242242
reason +=
243243
"<br/>" +

app/assets/javascripts/discourse/components/topic-status.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export default Ember.Component.extend(
2929

3030
@computed("disableActions")
3131
canAct(disableActions) {
32-
return Discourse.User.current() && !disableActions;
32+
return this.currentUser && !disableActions;
3333
},
3434

3535
buildBuffer(buffer) {

app/assets/javascripts/discourse/controllers/application.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ export default Ember.Controller.extend({
1717

1818
@computed
1919
loginRequired() {
20-
return Discourse.SiteSettings.login_required && !Discourse.User.current();
20+
return Discourse.SiteSettings.login_required && !this.currentUser;
2121
},
2222

2323
@computed

app/assets/javascripts/discourse/controllers/composer.js.es6

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ export default Ember.Controller.extend({
292292

293293
@computed("model.creatingPrivateMessage", "model.targetUsernames")
294294
showWarning(creatingPrivateMessage, usernames) {
295-
if (!Discourse.User.currentProp("staff")) {
295+
if (!this.get("currentUser.staff")) {
296296
return false;
297297
}
298298

app/assets/javascripts/discourse/controllers/discovery/categories.js.es6

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,7 @@ export default DiscoveryController.extend({
1414
// this makes sure the composer isn't scoping to a specific category
1515
category: null,
1616

17-
@computed
18-
canEdit() {
19-
return Discourse.User.currentProp("staff");
20-
},
17+
canEdit: Ember.computed.reads("currentUser.staff"),
2118

2219
@computed("model.categories.[].featuredTopics.length")
2320
latestTopicOnly() {

app/assets/javascripts/discourse/controllers/user-invited-show.js.es6

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,9 @@ export default Ember.Controller.extend({
4242
);
4343
},
4444

45-
@computed
46-
canInviteToForum() {
47-
return Discourse.User.currentProp("can_invite_to_forum");
48-
},
45+
canInviteToForum: Ember.computed.reads("currentUser.can_invite_to_forum"),
4946

50-
@computed
51-
canBulkInvite() {
52-
return Discourse.User.currentProp("admin");
53-
},
47+
canBulkInvite: Ember.computed.reads("currentUser.admin"),
5448

5549
showSearch: Ember.computed.gte("totalInvites", 10),
5650

0 commit comments

Comments
 (0)