depau/HackMD
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix checkLoginStateChanged might fall into infinite loop while calling loginStateChangeEvent

Browse source
This commit is contained in:
Wu Cheng-Han 2017-01-15 17:23:19 +08:00
parent 3cf40a8dec
commit f2a441061b
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
public/js/common.js
View file

@ -18,7 +18,8 @@ var checkAuth = false;
var profile = null;
var lastLoginState = getLoginState();
var lastUserId = getUserId();
var loginStateChangeEvent = null;
window.loginStateChangeEvent = null;
function resetCheckAuth() {
checkAuth = false;
@ -42,8 +43,7 @@ function setLoginState(bool, id) {
function checkLoginStateChanged() {
if (getLoginState() != lastLoginState || getUserId() != lastUserId) {
if(loginStateChangeEvent)
loginStateChangeEvent();
if(loginStateChangeEvent) setTimeout(loginStateChangeEvent, 100);
return true;
} else {
return false;
@ -65,8 +65,7 @@ function clearLoginState() {
function checkIfAuth(yesCallback, noCallback) {
var cookieLoginState = getLoginState();
if (checkLoginStateChanged())
checkAuth = false;
if (checkLoginStateChanged()) checkAuth = false;
if (!checkAuth || typeof cookieLoginState == 'undefined') {
$.get(serverurl + '/me')
.done(function (data) {
@ -107,7 +106,6 @@ module.exports = {
profile: profile,
lastLoginState: lastLoginState,
lastUserId: lastUserId,
loginStateChangeEvent: loginStateChangeEvent,
/* export functions */
resetCheckAuth: resetCheckAuth,