javascript - Login Timing in AngularJs using `$q(resolve, reject)` -


i having issue login timing in angular app. intention when user logs in, api key set in localstorage , available call on accessing api. after login, want app redirect user home state. happening user redirected, gets 401 - , based off have setup - redirected login. if hard refresh on browser after logging in works. here have going.

.factory('localstor', function($q, $log, $window, $rootscope) { var apikey = false;  return {     getapi: function() {         return $q(function(resolve, reject) {             var data = $window.localstorage.getitem('apikey');              if (data == null) {                  return $rootscope.$broadcast('notauthorized');             } else {                 var key = data;                 return resolve(key);              }          });     },     setapi: function(key) {         return $q(function(resolve, reject) {             apikey = false;             if (key === undefined) {                 return reject();             }              return $window.localstorage.setitem('apikey', key);         });     } }   })  function getlogin(email, password) {     return $q(function(resolve, reject) {         $http({             method: 'post',             data: {                 'email': email,                 'password': password             },             url: config.api.baseurl + '/user/login'         }).then(function(result) {             if (result) {                 var token = result.data.data.token;                 resolve(token);             } else {                 $log.debug('incorrect email or password');                 reject(1);             }         }, function(reason) {             reject(2);         });      }) }  $scope.submitlogin = function(email, password, form) {      $scope.errornote = false;      if (form.$valid) {         $scope.loading = true;         login.getapi(email, password).then(function(data) {             var key = data;             localstor.setapi(key).then(function() {                 $timeout(function() {                     $state.go('app.loading', {}, { reload: true });                 }, 30);             });             $scope.loading = false;           }, function(reason) {             $scope.errornote = reason;             $scope.loading = false;         });      } };

my calls api set 

var apikey = false; localstor.getapi().then(function(data) {     apikey = data;     $log.debug(apikey); }, function() {     $rootscope.$broadcast('notauthorized'); });  function withapikey(callback, errorcallback) {     localstor.getapi().then(function(data) {         if (typeof callback === 'function') {             callback(data);         }     }, function(err) {         if (typeof errorcallback === 'function') {             errorcallback(err);         }     }); }  function notificationcount() {     return $q(function(resolve, reject) {         withapikey(function(key) {             $http({                 method: 'get',                 headers: { 'authorization': 'bearer ' + key },                 url: config.api.baseurl + '/user/notifications/count'             }).then(function(result) {                 if (result) {                     resolve(result);                 } else {                     reject('uh oh');                 }             }, function() {                 reject('uh oh');             });         });     }) }

is $q(resolve, reject) anti-pattern?

using $q(resolve, reject) $http service example of $q defer anti-pattern. unnecessary , prone erroneous implementation.

simply return $http promise. use .then method transform response:

function getlogin(email, password) {     ̶r̶e̶t̶u̶r̶n̶ ̶$̶q̶(̶f̶u̶n̶c̶t̶i̶o̶n̶(̶r̶e̶s̶o̶l̶v̶e̶,̶ ̶r̶e̶j̶e̶c̶t̶)̶ ̶{̶     ͟r͟e͟t͟u͟r͟n͟ $http({             method: 'post',             data: {                 'email': email,                 'password': password             },             url: config.api.baseurl + '/user/login'         }).then(function(result) {             if (result) {                 var token = result.data.data.token;                 ̶r̶e̶s̶o̶l̶v̶e̶(̶t̶o̶k̶e̶n̶)̶;̶                 ͟r͟e͟t͟u͟r͟n͟ token;             } else {                 $log.debug('incorrect email or password');                 ̶r̶e̶j̶e̶c̶t̶(̶2̶)̶;̶                 ͟t͟h͟r͟o͟w͟  1;             }         }, function(reason) {             ̶r̶e̶j̶e̶c̶t̶(̶2̶)̶;̶             ͟t͟h͟r͟o͟w͟  2;         });     }) }

the .then method returns new promise settles respective handlers either return or throw.

for more information, see you're missing point of promises.


Comments

Post a Comment

Popular posts from this blog

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m...
Read more

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

sql server - Why does Linq-to-SQL add unnecessary COUNT()? -

here tables: create table customer ( cid int primary key, name varchar(32) ) create table ord ( oid int primary key, cid int foreign key references customer, address varchar(20) ) here's linq-to-sql statement: var aaa = c in db.customer select new { c, o = c.ord.tolist() }; and here's inexplicable query generated linq-to-sql: select [t0].[cid] [cid], [t0].[name] [name], [t1].[oid] [oid], [t1].[cid] [cid2], [t1].[address] [address], (select count(*) [dbo].[ord] [t2] [t2].[cid] = [t0].[cid]) [value] [dbo].[customer] [t0] left outer join [dbo].[ord] [t1] on [t1].[cid] = [t0].[cid] order [t0].[cid], [t1].[oid] i want understand how rid of count(*) part. it's entirely uncalled for! linq needs count(*) in order determine number of entries returned o = c.ord.tolist() .
Read more