mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-01-09 04:31:17 +03:00
b4cdc85a59
refs https://github.com/TryGhost/Ghost/issues/6949 Handle version mismatch errors by: - displaying an alert asking the user to copy any data and refresh - disabling navigation so that unsaved data is not accidentally lost Detailed changes: - add `error` action to application route for global route-based error handling - remove 404-handler mixin, move logic into app route error handler - update `.catch` in validation-engine so that promises are rejected with the original error objects - add `VersionMismatchError` and `isVersionMismatchError` to ajax service - add `upgrade-status` service - has a method to trigger the alert and toggle the "upgrade required" mode - is injected into all routes by default so that it can be checked before transitioning - add `Route` override - updates the `willTransition` hook to check the `upgrade-status` service and abort the transition if we're in "upgrade required" mode - update notifications `showAPIError` method to handle version mismatch errors - update any areas where we were catching ajax errors manually so that the version mismatch error handling is obeyed - fix redirect tests in editor acceptance test - fix mirage's handling of 404s for unknown posts in get post requests - adjust alert z-index to to appear above modal backgrounds
98 lines
2.6 KiB
JavaScript
98 lines
2.6 KiB
JavaScript
import RSVP from 'rsvp';
|
|
import Controller from 'ember-controller';
|
|
import computed from 'ember-computed';
|
|
import injectService from 'ember-service/inject';
|
|
import SettingsSaveMixin from 'ghost-admin/mixins/settings-save';
|
|
import NavigationItem from 'ghost-admin/models/navigation-item';
|
|
|
|
export default Controller.extend(SettingsSaveMixin, {
|
|
config: injectService(),
|
|
notifications: injectService(),
|
|
|
|
newNavItem: null,
|
|
|
|
blogUrl: computed('config.blogUrl', function () {
|
|
let url = this.get('config.blogUrl');
|
|
|
|
return url.slice(-1) !== '/' ? `${url}/` : url;
|
|
}),
|
|
|
|
init() {
|
|
this._super(...arguments);
|
|
this.set('newNavItem', NavigationItem.create({isNew: true}));
|
|
},
|
|
|
|
save() {
|
|
let navItems = this.get('model.navigation');
|
|
let newNavItem = this.get('newNavItem');
|
|
let notifications = this.get('notifications');
|
|
let validationPromises = [];
|
|
|
|
if (!newNavItem.get('isBlank')) {
|
|
validationPromises.pushObject(this.send('addItem'));
|
|
}
|
|
|
|
navItems.map((item) => {
|
|
validationPromises.pushObject(item.validate());
|
|
});
|
|
|
|
return RSVP.all(validationPromises).then(() => {
|
|
return this.get('model').save().catch((err) => {
|
|
notifications.showAPIError(err);
|
|
});
|
|
}).catch(() => {
|
|
// TODO: noop - needed to satisfy spinner button
|
|
});
|
|
},
|
|
|
|
addNewNavItem() {
|
|
let navItems = this.get('model.navigation');
|
|
let newNavItem = this.get('newNavItem');
|
|
|
|
newNavItem.set('isNew', false);
|
|
navItems.pushObject(newNavItem);
|
|
this.set('newNavItem', NavigationItem.create({isNew: true}));
|
|
},
|
|
|
|
actions: {
|
|
addItem() {
|
|
let newNavItem = this.get('newNavItem');
|
|
|
|
// If the url sent through is blank (user never edited the url)
|
|
if (newNavItem.get('url') === '') {
|
|
newNavItem.set('url', '/');
|
|
}
|
|
|
|
return newNavItem.validate().then(() => {
|
|
this.addNewNavItem();
|
|
});
|
|
},
|
|
|
|
deleteItem(item) {
|
|
if (!item) {
|
|
return;
|
|
}
|
|
|
|
let navItems = this.get('model.navigation');
|
|
|
|
navItems.removeObject(item);
|
|
},
|
|
|
|
reorderItems(navItems) {
|
|
this.set('model.navigation', navItems);
|
|
},
|
|
|
|
updateUrl(url, navItem) {
|
|
if (!navItem) {
|
|
return;
|
|
}
|
|
|
|
navItem.set('url', url);
|
|
},
|
|
|
|
reset() {
|
|
this.set('newNavItem', NavigationItem.create({isNew: true}));
|
|
}
|
|
}
|
|
});
|