Commit e683f6af by Torkel Ödegaard

Merge remote-tracking branch 'origin/10277_logout_route_full_page_reload'

parents 4acf6b0e 23cceaec
......@@ -6,10 +6,12 @@ import appEvents from "app/core/app_events";
// Good for communication react > angular and vice verse
export class GlobalEventSrv {
private appSubUrl;
private fullPageReloadRoutes;
/** @ngInject */
constructor(private $location, private $timeout) {
constructor(private $location, private $timeout, private $window) {
this.appSubUrl = config.appSubUrl;
this.fullPageReloadRoutes = ["/logout"];
}
// Angular's $location does not like <base href...> and absolute urls
......@@ -27,6 +29,10 @@ export class GlobalEventSrv {
init() {
appEvents.on("location-change", payload => {
const urlWithoutBase = this.stripBaseFromUrl(payload.href);
if (this.fullPageReloadRoutes.indexOf(urlWithoutBase) > -1) {
this.$window.location.href = payload.href;
return;
}
this.$timeout(() => {
// A hack to use timeout when we're changing things (in this case the url) from outside of Angular.
......
import { GlobalEventSrv } from "app/core/services/global_event_srv";
import { GlobalEventSrv } from "app/core/services/global_event_srv";
import { beforeEach } from "test/lib/common";
jest.mock("app/core/config", () => {
......@@ -11,7 +11,7 @@ describe("GlobalEventSrv", () => {
let searchSrv;
beforeEach(() => {
searchSrv = new GlobalEventSrv(null, null);
searchSrv = new GlobalEventSrv(null, null, null);
});
describe("With /subUrl as appSubUrl", () => {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment