php - Check if update happened in put request

Solution:

You can use Model Events, i.e.afterUpdate andnotSaved, like:

use Phalcon\Mvc\Model;
use Phalcon\Http\Response;

class ModelBase extends Model
{
    public function afterUpdate()
    {
        $response = new Response();
        $response->setJsonContent([
           'success' => true,
           'message' => "Record updated"
        ])->send();
    }

    public function notSaved()
    {
        $response = new Response();
        $response->setJsonContent([
           'success' => false,
           'message' => 'Record not saved'
        ])->send();
    }
}
Copy code

TheProduct and all other models will extendModelBase. Then your code could be:

public function put()
{ 
  $id = $this->getParam('id');

  $input = $this->getRawData();

  $product = Product::findFirst([
    'conditions' => 'id = :id:',
    'bind' => ['id' => $id]
  ]);

  if ($product === null){
    throw new NotFoundException();
  }

  $product->assign($input);
  $product->update();
}
Copy code

And Phalcon event will respond if the model was updated or not. If you prefer, you can also use custom http response codes forupdate ornotSaved. More information aboutModel Events in the documentation

Solution:

You are calling $product->update() three times. You do it once after the assign, then again for your if test, which is why it's always returning TRUE there I believe, and once inside the toArray() which may not actually return anything since the second and third updates don't have any data to update (not sure about that though).

I would code this as follows:

$product->assign($input);
$results = $product->update();

if (!results) {
    $this->errorResponse($product->getMessages());
} else {
    $this->successResponse($results->toArray());
}
Copy code

I am assuming that the $product->assign($input); statement is working as expected to update the $product data for you. I don't use that. I prefer to do direct assignments for updates so nothing is left to chance, ie. $product->whatever = $input['whatever'];.

Give this a try and hopefully it will work as expected for you.