krishnakumar
krishnakumar

Reputation: 118

Can't delete records from a table in angular JS

$scope.submitForm = function(){ 
        $scope.users.push({
            fname       : $scope.fname,
            lname       : $scope.lname,
            email       : $scope.email,
            password    : $scope.password,
        });
        $scope.fname = ''; //clear the input after adding
        $scope.lname = '';
        $scope.email = '';
        $scope.password = '';
        localStorage.setItem('users', JSON.stringify($scope.users));
    };

I am facing some issues while delete a record from a table using angular js. At the time of deletion its working fine but while refreshing the page it came back.

$scope.deleteUser = function(idx){
        console.log(idx);
        deleteUser = $window.confirm('Are you sure you want to delete the User?');
        if(deleteUser){
            $scope.users.splice(idx, 1);
            console.log($scope.users);
        }

    };

Upvotes: 2

Views: 81

Answers (2)

Mihail Petkov
Mihail Petkov

Reputation: 1545

You should update your local storage values when you delete an user.

$scope.deleteUser = function(idx){
    console.log(idx);
    deleteUser = $window.confirm('Are you sure you want to delete the User?');
    if(deleteUser){
        $scope.users = $scope.users.splice(idx, 1);
        localStorage.setItem('users', JSON.stringify($scope.users));
        console.log($scope.users);
    }

};

Upvotes: 1

Thalaivar
Thalaivar

Reputation: 23632

$scope.deleteUser = function(idx){
        console.log(idx);
        deleteUser = $window.confirm
             ('Are you sure you want to delete the User?');
        if(deleteUser){
            $scope.users.splice(idx, 1);
            console.log($scope.users);
        }           
    };

Here you are not deleting it from localStorage, i am guessing $scope.users contains data from the API.What you can do is after deleting, you can update the localStorage with $scope.users again

localStorage.setItem('users', JSON.stringify($scope.users));

Upvotes: 1

Related Questions