mirror of
https://github.com/TryGhost/Ghost.git
synced 2024-12-24 11:22:19 +03:00
7b443d4b63
no issue
The `config` service has been a source of confusion when writing with modern Ember patterns because it's use of the deprecated `ProxyMixin` forced all property access/setting to go via `.get()` and `.set()` whereas the rest of the system has mostly (there are a few other uses of ProxyObjects remaining) eliminated the use of the non-native get/set methods.
- removed use of `ProxyMixin` in the `config` service by grabbing the API response after fetching and using `Object.defineProperty()` to add native getters/setters that pass through to a tracked object holding the API response data. Ember's autotracking automatically works across the native getters/setters so we can then use the service as if it was any other native object
- updated all code to use `config.{attrName}` directly for getting/setting instead of `.get()` and `.set()`
- removed unnecessary async around `config.availableTimezones` which wasn't making any async calls
94 lines
3.2 KiB
JavaScript
94 lines
3.2 KiB
JavaScript
import EmberObject from '@ember/object';
|
|
import UnauthenticatedRoute from 'ghost-admin/routes/unauthenticated';
|
|
import ValidationEngine from 'ghost-admin/mixins/validation-engine';
|
|
import classic from 'ember-classic-decorator';
|
|
import {inject as service} from '@ember/service';
|
|
import {tracked} from '@glimmer/tracking';
|
|
|
|
// EmberObject is still needed here for ValidationEngine
|
|
@classic
|
|
class SignupDetails extends EmberObject.extend(ValidationEngine) {
|
|
@tracked name = '';
|
|
@tracked email = '';
|
|
@tracked password = '';
|
|
|
|
token = '';
|
|
blogTitle = ''; // used for password validation
|
|
|
|
validationType = 'signup';
|
|
}
|
|
|
|
export default class SignupRoute extends UnauthenticatedRoute {
|
|
@service ghostPaths;
|
|
@service notifications;
|
|
@service session;
|
|
@service ajax;
|
|
@service config;
|
|
|
|
beforeModel() {
|
|
if (this.session.isAuthenticated) {
|
|
this.notifications.showAlert('You need to sign out to register as a new user.', {type: 'warn', delayed: true, key: 'signup.create.already-authenticated'});
|
|
}
|
|
|
|
super.beforeModel(...arguments);
|
|
}
|
|
|
|
model(params) {
|
|
let signupDetails = SignupDetails.create();
|
|
let re = /^(?:[A-Za-z0-9_-]{4})*(?:[A-Za-z0-9_-]{2}|[A-Za-z0-9_-]{3})?$/;
|
|
let email,
|
|
tokenText;
|
|
|
|
return new Promise((resolve) => {
|
|
if (!re.test(params.token)) {
|
|
this.notifications.showAlert('Invalid token.', {type: 'error', delayed: true, key: 'signup.create.invalid-token'});
|
|
|
|
resolve(this.transitionTo('signin'));
|
|
return;
|
|
}
|
|
|
|
tokenText = atob(params.token);
|
|
email = tokenText.split('|')[1];
|
|
|
|
// leave e-mail blank even though we get it from the token because
|
|
// we need the user to type it in for Chrome to remember the
|
|
// email/password combo properly
|
|
signupDetails.email = '';
|
|
signupDetails.token = params.token;
|
|
|
|
let authUrl = this.ghostPaths.url.api('authentication', 'invitation');
|
|
|
|
this.ajax.request(authUrl, {
|
|
dataType: 'json',
|
|
data: {
|
|
email
|
|
}
|
|
}).then((response) => {
|
|
if (response && response.invitation && response.invitation[0].valid === false) {
|
|
this.notifications.showAlert('The invitation does not exist or is no longer valid.', {type: 'warn', delayed: true, key: 'signup.create.invalid-invitation'});
|
|
|
|
resolve(this.transitionTo('signin'));
|
|
return;
|
|
}
|
|
|
|
// set blogTitle, so password validation has access to it
|
|
signupDetails.blogTitle = this.config.blogTitle;
|
|
|
|
resolve(signupDetails);
|
|
}).catch(() => {
|
|
resolve(signupDetails);
|
|
});
|
|
});
|
|
}
|
|
|
|
deactivate() {
|
|
super.deactivate(...arguments);
|
|
|
|
// clear the properties that hold the sensitive data from the controller
|
|
const signupDetails = this.controllerFor('signup').signupDetails;
|
|
signupDetails.email = '';
|
|
signupDetails.password = '';
|
|
signupDetails.token = '';
|
|
}
|
|
}
|