PHP json array to delete a line in JSON file

Question

Having a bit of an issue with some PHP code. I have an contacts list angular app to add and remove contacts. I have the add part done but the delete is giving me some grief. On ng-click I'm sending data with a $http request to remove a specific line in a JSON file. It's sending the data fine but I think something is wrong with my php...

delete.php

    $jsonString = file_get_contents("contacts.json");
$contactData = json_decode($jsonString, true);

$getArray = array(
    "firstname" => $_POST['firstname'],
    "lastname"  => $_POST['lastname'],
    "number"    => $_POST['number'],
    "hobbies"   => $_POST['hobbies']
);

foreach ($contactData as $key => $value) {
    if($value->number == $getArray) {
        unset($contactData[$key]);
    }
}

$json = json_encode($contactData, JSON_PRETTY_PRINT);
//Add to JSON contents
file_put_contents("contacts.json", $json);

app.js

$scope.deleteRow = function(contact){
    console.log(contact);
    //var index = $scope.number.indexOf(item);
    $http({
        method   : 'POST',
        url      : 'delete.php',
        data     : $.param(contact),
        headers  : {'Content-Type': 'application/x-www-form-urlencoded'}
    })
    .success(function(data){
        updateService.listWasUpdated();
    });
};

html

<button type="button" class="btn btn-default" aria-label="Bold" ng-click="deleteRow(contact)">

Any help would be greatly appreciated

Uri Goren · Accepted Answer · 2016-11-26 21:27:31Z

1

Because json_decode is being called with assc==true the returning array is associative, and you should use $value['number'] instead of $value->number

Here is a corrected version with the array_filter function for brevity:

//load
$jsonString = file_get_contents("contacts.json");
$contactData = json_decode($jsonString, true);
//filter
$contactData = array_filter($contactData, function ($v) {return $v['number']!=$_POST['number'];});
//save
$json = json_encode($contactData, JSON_PRETTY_PRINT);
file_put_contents("contacts.json", $json);

edited Nov 26, 2016 at 21:27

answered Nov 26, 2016 at 18:06

Uri Goren

13.8k8 gold badges62 silver badges113 bronze badges

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

Comments

Winter Faulk · Accepted Answer · 2016-11-26 18:08:37Z

1

The issue is that you are not specifying the number key under the $getArray in the If statement, it needs to be replace with $getArray['number'].

if($value->number == $getArray['number']) {
    unset($contactData[$key]);
}

Is there a reason you are not using SQLite to store your contact info? It would make it a lot easier to handle storing and manipulating of your contacts.

answered Nov 26, 2016 at 18:08

Winter Faulk

4134 silver badges9 bronze badges

2 Comments

Uri Goren Over a year ago

voting up for the suggestion to use a SQLite

Braydon Harris Over a year ago

Yeah I know a database is the correct way to go, but this is just a learning app for angular and only working with like 10 lines of JSON.

Collectives™ on Stack Overflow

PHP json array to delete a line in JSON file

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related