Update Fails from Home controller

Hello Team

Please any help on this

Hello Mr Jayraj

This is my query for fetching or binding the data which is perfectly working

  public JsonResult UpdateItem(int? ItemId, int? quantity)
        {

            var dataList = objRestaurantDBEntities.tblItems.Include("tblCategory").Include("tblQuantities").ToList();
            var modifiedData = dataList.AsEnumerable().Where(x => x.ItemId == ItemId).Select(d => new bindItem
                {
                    
                    ItemId = d.ItemId,
                    CategoryId = d.CategoryId,
                    PCode = d.PCode,
                    CategoryName = d.tblCategory.CategoryName,
                    QuantityId = d.tblQuantities.FirstOrDefault().QuantityId,
                    Quantity = d.tblQuantities.FirstOrDefault().Quantity,
                    ItemName = d.ItemName,
                    CostPrice = d.CostPrice,
                    ItemPrice = d.ItemPrice,
                    Profit = d.Profit,
                    Active = d.Active
                }).FirstOrDefault();
            return Json(modifiedData, JsonRequestBehavior.AllowGet); 
   
 }

 function getItemforUpdate($ItemId)
                {
                    
                    var itemModel = new Object();
                    $("#itemId").val($ItemId);
                    if ($ItemId != 0) {
                        jQuery.ajax({
                            url: "@Url.Action("UpdateItem", "Home")" + "?ItemId=" + $ItemId + "&quantity=abc",                          
                            data:({
                                ItemId: $ItemId
                            }),
                            type: "Get",
                            dataType: "json",
                            contentType: "application/json; charset=utf-8",
                            data: itemModel,
                            success: function (itemModel) {                    
                                if (itemModel != null) {                                  
                                    $("#pCode").val(itemModel.PCode);
                                    $("#itemId").val(itemModel.ItemId)                                 
                                    $("#categoryId").val(itemModel.CategoryId)
                                    $("#itemName").val(itemModel.ItemName)
                                    $("#txtCostPrice").val( itemModel.CostPrice);
                                    $("#quantity").val(itemModel.Quantity);
                                    $("#itemPrice").val(itemModel.ItemPrice)
                                    $("#status").val(itemModel.Active)
                                    
                                }
                            }
                        });
                    
                    }
                    else {
                        $("#itemId").val("");
                        $("#categoryId").val("");
                        $("#pCode").val("");
                        $("#itemName").val("");
                        $("#txtCostPrice").val("");                      
                        $("#quantity").val("");
                        $("#itemPrice").val("");
                        $("#status").val("");                       
                    }
                    
                    $("#UpdateData").show();
                    $("#SubmitData").hide();
                    $("#exampleModal").modal('show');
                }

The issue with the update functionality in the provided code seems to be related to how the UpdateforProduct function is being called and the data being passed to the SaveItem action method in the Home controller.

In the UpdateforProduct function, the ItemId parameter is being passed, but it is not being used to fetch the existing item from the database for updating. Instead, the function is creating a new itemModel object and populating it with values from various HTML elements.

To fix this issue, you need to modify the UpdateforProduct function to fetch the existing item from the database using the ItemId parameter. You can then update the fetched item with the new values from the HTML elements and send it to the SaveItem action method for updating.

function UpdateforProduct(ItemId) {
    // Fetch the existing item from the database
    $.ajax({
        type: 'GET',
        url: '@Url.Action("GetItem", "Home")',
        data: { itemId: ItemId },
        success: function (response) {
            if (response.success) {
                var itemModel = response.itemModel;
                // Update the item with new values from HTML elements
                itemModel.PCode = $("#pCode").val();
                itemModel.CategoryId = $("#categoryId").val();
                itemModel.QuantityId = $("#qtyId").val();
                itemModel.Quantity = $("#quantity").val();
                itemModel.ItemName = $("#itemName").val();
                itemModel.CostPrice = $("#txtCostPrice").val();
                itemModel.ItemPrice = $("#itemPrice").val();
                itemModel.Active = $("#status").val();

                // Send the updated item to the SaveItem action method for updating
                $.ajax({
                    contentType: 'application/json; charset=utf-8',
                    dataType: 'JSON',
                    type: 'POST',
                    url: '@Url.Action("SaveItem", "Home")',
                    data: JSON.stringify({
                        itemModel: itemModel
                    }),
                    success: function (response) {
                        if (response.success) {
                            toastr.success(response.message = "Product successfully updated!");
                            DataTable.ajax.reload();
                            $("#exampleModal").modal('hide');
                        }
                    },
                    error: function (msg) {
                        toastr.danger(msg.responseText);
                    }
                });
            }
        },
        error: function (msg) {
            toastr.danger(msg.responseText);
        }
    });
}