Odtnhj

I am trying to create a webpage where when a user logs in from the login page, my page that is the 'user management' page should display users belonging to the login user's company.

For eg: When a user belonging to company APPLE logs in, this webpage should only display users belonging to APPLE.

In the code, "currentUser" is the login user I get from the login page via localStorage getItem. "allUsers" array is the list of all users from different companies. Empty array "user" should display only display Current user's company's-users from the list of allUsers.

For that I have used FOR LOOP seen at the end of the JS file. I'm getting a error that the property 'company' is undefined. Looking for someone to help me. Thanks in advance.

 var myApp = angular.module("myApp", );

 myApp.controller("myController", function ($scope) 
 console.log("in controller...");
 $scope.newUser = ;
 $scope.info = "";

 if ($scope.users = JSON.parse(localStorage.getItem("users")) !== null) 

 $scope.users = JSON.parse(localStorage.getItem("users"));
 

 if (localStorage.getItem("currentUser") !== null) 

 var currentUser = JSON.parse(localStorage.getItem("currentUser"));
 console.log("Received");
 
 else 
 console.log("Not received");
 

 if (localStorage.getItem("allUsers") === null) 
 $scope.allUsers = [
 email: "John@yahoo.com", password:"John123", firstName: "John", lastName: "Doe", contact: "281-283-2480", role: "Supplier-Admin", company: "Apple" ,
 email: "Rick@yahoo.com", password: "Rick123", firstName: "Rick", lastName: "Fraiser", contact: "987-283-2489", role: "Supplier-User", company: "Apple" ,
 email: "Sam@yahoo.com", password: "Sam123", firstName: "Sam", lastName: "Tarly", contact: "456-786-2480", role: "BuyerAdmin", company: "Samsung" 
 ];
 localStorage.setItem("allUsers", JSON.stringify($scope.allUsers));
 // localStorage.setItem("users", JSON.stringify($scope.users));
 else 
 $scope.allUsers = JSON.parse(localStorage.getItem("allUsers"));
 

 //filter allUsers based on currentUser's role
 for (var i = 0; $scope.allUsers.length; i++) 
 $scope.users = [];
 if ($scope.allUsers[i].company === currentUser[0].company) 
 $scope.users.push($scope.allUser[i]);
 
 localStorage.setItem("users", JSON.stringify($scope.users));
 
 );

localStorage.getItem() returns string representation of null, if the key has null value in localStorage so your null check might not be behaving like expected. I'd recommend to wrap getItem in a function that would check that.

function(itemName) 
 const item = localStorage.getItem(itemName);
 return item === 'null'
 ? null
 : item;

Your problem is probably in the place where you set currentUser, maybe because of the same null check.

Also try declaring your i variable in for loop.

for(var i = 0; ...) ...

Edit

There seems to be an error in this line

if ($scope.allUsers[i].company === currentUser[0].company) 
 $scope.users.push($scope.allUser[i]);

You're trying to push $scope.allUser[I] which is not defined, you've probably meant to write $scope.allUsers[I].