Remove double screen blink after logout

documize · May 14, 2019 · e2a3962 · e2a3962
1 parent 8ecbb9c
commit e2a3962
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/gui/app/pods/auth/logout/route.js b/gui/app/pods/auth/logout/route.js
@@ -15,6 +15,8 @@ import Route from '@ember/routing/route';
 export default Route.extend({
 	session: service(),
 	appMeta: service(),
+	router: service(),
+	localStorage: service(),
 
 	activate: function () {
 		this.get('session').invalidate().then(() => {

diff --git a/gui/app/routes/application.js b/gui/app/routes/application.js
@@ -25,7 +25,7 @@ export default Route.extend(ApplicationRouteMixin, {
 
 		let sa = this.get('session.session.authenticator');
 
-		return this.get('appMeta').boot(transition.targetName, window.location.href).then(data => {
+		return this.get('appMeta').boot(transition.targetName, '').then(data => {
 			if (sa !== "authenticator:documize" && sa !== "authenticator:keycloak" && sa !== "authenticator:ldap" && data.allowAnonymousAccess) {
 				if (!this.get('appMeta.setupMode') && !this.get('appMeta.secureMode')) {
 					return this.get('session').authenticate('authenticator:anonymous', data);
@@ -46,7 +46,7 @@ export default Route.extend(ApplicationRouteMixin, {
 			this.get('localStorage').clearSessionItem('entryUrl')
 
 			if (!_.includes(next, '/auth/')) {
-				window.location.href= next;
+				// window.location.href= next;
 			}
 		}
 	},

diff --git a/gui/app/services/app-meta.js b/gui/app/services/app-meta.js
@@ -89,7 +89,7 @@ export default Service.extend({
 
 				this.get('localStorage').clearAll();
 				return resolve(this);
-			} else if (!_.includes(requestedUrl, '/auth/')) {
+			} else if (!_.includes(requestedUrl, '/auth/') && !_.isEmpty(requestedUrl)) {
 				this.get('localStorage').storeSessionItem('entryUrl', requestedUrl);
 			}
 

diff --git a/gui/app/templates/components/layout/master-sidebar.hbs b/gui/app/templates/components/layout/master-sidebar.hbs
@@ -99,7 +99,7 @@
 								<a href="https://docs.documize.com" target="_blank" class="item">Help</a>
 								{{#if enableLogout}}
 									<div class="divider"></div>
-									{{#link-to "auth.logout" class="item" }}Logout{{/link-to}}
+									{{#link-to "auth.logout" class="item"}}Logout{{/link-to}}
 								{{/if}}
 							</div>
 						{{/attach-popover}}