Merge "PolyGerrit: Fix redirecting to /login/ in gr-router"
This commit is contained in:
@@ -103,13 +103,18 @@
|
||||
});
|
||||
});
|
||||
|
||||
function redirectToLogin(data) {
|
||||
const basePath = base || '';
|
||||
page('/login/' + encodeURIComponent(data.substring(basePath.length)));
|
||||
}
|
||||
|
||||
page('/dashboard/(.*)', loadUser, data => {
|
||||
restAPI.getLoggedIn().then(loggedIn => {
|
||||
if (loggedIn) {
|
||||
data.params.view = Gerrit.Nav.View.DASHBOARD;
|
||||
app.params = data.params;
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
@@ -125,7 +130,7 @@
|
||||
filter: null,
|
||||
};
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
@@ -140,7 +145,7 @@
|
||||
filter: data.params.filter,
|
||||
};
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
@@ -154,7 +159,7 @@
|
||||
filter: data.params.filter || null,
|
||||
};
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
@@ -273,7 +278,7 @@
|
||||
adminView: 'gr-plugin-list',
|
||||
};
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
@@ -285,7 +290,7 @@
|
||||
data.params.placeholder = true;
|
||||
app.params = data.params;
|
||||
} else {
|
||||
page.redirect('/login/' + encodeURIComponent(data.canonicalPath));
|
||||
redirectToLogin(data.canonicalPath);
|
||||
}
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user