Angularjs CRUD delete issue (GET called in DELETE method)

Question

I am trying to consume my spring rest service using angularjs client following this link

Create,update and read parts are working. When I try to delete, its showing this error.

Error: [$resource:badcfg] Error in resource configuration for action get. Expected response to contain an object but got an array (Request: GET http://localhost:8080/SpringRestExample/employee)

Why i am getting GET request in DELETE method?

employee_service.js

'use strict';

App.factory('Employee', ['$resource', function ($resource) {
    return $resource(
            'http://localhost:8080/SpringRestExample/employee/:id', 
            {id: '@employeeId'},
            {
                update: {
                      method: 'PUT' 
                }

            }
    );
}]);

employee_controller.js

'use strict';

App.controller('EmployeeController', ['$scope', 'Employee', function($scope, Employee) {
          var self = this;
          self.employee= new Employee();

          self.employees=[];

          self.fetchAllEmployees = function(){
              self.employees = Employee.query();
          };

          self.createEmployee = function(){
              self.employee.$save(function(){
                  self.fetchAllEmployees();
              });
          };

          self.updateEmployee = function(){
              self.employee.$update(function(){
                  self.fetchAllEmployees();
              });
          };

         self.deleteEmployee = function(identity){
             var employee = Employee.get({employeeId:identity}, function() {
                 employee.$delete(function(){
                      console.log('Deleting employee with id ', identity);
                      self.fetchAllEmployees();
                  });
             });
          };

          self.fetchAllEmployees();

          self.submit = function() {
              if(self.employee.employeeId==null){
                  console.log('Saving New Employee', self.employee);    
                  self.createEmployee();
              }else{
                  console.log('Updating employee with id ', self.employee.employeeId);
                  self.updateEmployee();
                  console.log('Employee updated with id ', self.employee.employeeId);
              }
              self.reset();
          };

          self.edit = function(employeeId){
              console.log('id to be edited', employeeId);
              for(var i = 0; i < self.employees.length; i++){
                  if(self.employees[i].employeeId === employeeId) {
                     self.employee = angular.copy(self.employees[i]);
                     break;
                  }
              }
          };

          self.remove = function(employeeId){
              console.log('id to be deleted', employeeId);
              if(self.employee.employeeId === employeeId) {//If it is the one shown on screen, reset screen
                 self.reset();
              }
              self.deleteEmployee(employeeId);
          };


          self.reset = function(){
              self.employee= new Employee();
              $scope.myForm.$setPristine(); //reset Form
          };

      }]);

Sounds like your backend is returning an array for the .get request. Nothing to do with DELETE. Check the response data for http://localhost:8080/SpringRestExample/employee/someEmployeeId — Phil
– Phil, Commented Oct 12, 2016 at 11:03
@Phil the url will return employee details(id,firstname,lastname and age) of the employeeId. — user3785322
– user3785322, Commented Oct 12, 2016 at 12:33
That's not what I meant. Pick a valid employee ID (let's say it's "1") and use curl or similar to request http://localhost:8080/SpringRestExample/employee/1. If the response starts with [ and not {, then it is returning an array where Angular is expecting an object — Phil
– Phil, Commented Oct 12, 2016 at 23:36

lyschoening · Accepted Answer · 2016-10-12 11:06:52Z

1

Your issue could be when you call Employee.get({employeeId:identity}, ...) prior to deleting the employee. This will load the employee before deletion and it will do a GET request on 'http://localhost:8080/SpringRestExample/employee/:id'.

For this query to work properly, you need to provide id, which you haven't done, so it might just be leaving out that part of the URL. You provided employeeId, which is only used for mapping the id parameter to the Employee objects. Try replacing the query above with {id: identity}.

answered Oct 12, 2016 at 11:06

lyschoening

18.8k11 gold badges48 silver badges55 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

user3785322 Over a year ago

@Phil so i have got some other issue as you suggested in the first comment?

lyschoening Over a year ago

@Phil I think that is only for mapping to the Employee.employeeId property.

Phil Over a year ago

@lyschoening you are correct. From the documentation ~ "If the parameter value is prefixed with @, then the value for that parameter will be extracted from the corresponding property on the data object (provided when calling a "non-GET" action method)."

Collectives™ on Stack Overflow

Angularjs CRUD delete issue (GET called in DELETE method)

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related