Resolve github maxi comments.

This commit is contained in:
LautaroCesso 2020-08-18 23:48:06 -03:00
parent e397d45c53
commit e36b984b23
4 changed files with 10 additions and 8 deletions

View File

@ -54,7 +54,7 @@ export default {
data: { data: {
userId: rememberData.userId, userId: rememberData.userId,
rememberToken: rememberData.token, rememberToken: rememberData.token,
staff: (rememberData.isStaff === "true") ? 1 : 0, staff: rememberData.isStaff,
remember: 1, remember: 1,
} }
}).then((result) => { }).then((result) => {

View File

@ -95,7 +95,7 @@ class SessionReducer extends Reducer {
sessionStore.storeRememberData({ sessionStore.storeRememberData({
token: resultData.rememberToken, token: resultData.rememberToken,
userId: resultData.userId, userId: resultData.userId,
isStaff: resultData.staff, isStaff: resultData.staff ? 1 : 0,
expiration: resultData.rememberExpiration expiration: resultData.rememberExpiration
}); });
} }

View File

@ -120,14 +120,17 @@ class LoginController extends Controller {
if($rememberToken) { if($rememberToken) {
$sessionCookie = SessionCookie::getDataStore($rememberToken, 'token'); $sessionCookie = SessionCookie::getDataStore($rememberToken, 'token');
$userId = Controller::request('userId'); $userId = Controller::request('userId');
$isStaff = Controller::request('staff'); $isStaff = !!Controller::request('staff');
if(!$sessionCookie->isNull()) {
$loggedInstance = $isStaff ? $sessionCookie->staff : $sessionCookie->user; $loggedInstance = $isStaff ? $sessionCookie->staff : $sessionCookie->user;
if ((!$sessionCookie->isNull()) && ($userId === $loggedInstance->id) && ($isStaff === $sessionCookie->isStaff)) { if(($userId == $loggedInstance->id) && ($isStaff == $sessionCookie->isStaff)) {
$userInstance = $loggedInstance; $userInstance = $loggedInstance;
$sessionCookie->delete(); $sessionCookie->delete();
} }
} }
}
return $userInstance; return $userInstance;
} }

View File

@ -89,7 +89,6 @@ describe '/user/login' do
(result['status']).should.equal('success') (result['status']).should.equal('success')
@rememberToken = result['data']['rememberToken'] @rememberToken = result['data']['rememberToken']
puts result['data']
@staffId = result['data']['userId'] @staffId = result['data']['userId']
request('/user/logout', {}) request('/user/logout', {})