How to pass a null value in the clancats hydrahon update()?

Question

I need to pass a null value for a database column, according to the code below:

$birthdate = null;

User::update()
->set('name', $name)
->set('email', $email)
->set('birthdate', $birthdate)                
->where('id', $id)
->execute();

But the error is generated:

How to do this ?

Note: The update is a method of clancats hydrahon.

I believe that the problem occurs in that part of the library code.

public function set($param1, $param2 = null)
{
// do nothing if we get nothing
if (empty($param1))
{
   return $this;
}
        
// when param 2 is not null we assume that only one set is passed
// like: set( 'name', 'Lu' ); instead of set( array( 'name' => 'Lu' ) );
if ( !is_null( $param2 ) )
{
   $param1 = array( $param1 => $param2 );
}
        
// merge the new values with the existing ones.
$this->values = array_merge( $this->values, $param1 ); 
        
// return self so we can continue running the next function
return $this;
}

I added part of the lib code that I believe is the reason for the error. — Gato de Schrödinger, Oct 10 '20 at 18:30
can you share the values of param1 and param2 when the error is occuring ? — Al-Amin, Oct 10 '20 at 18:41
$param1 Array ( ). $param2 nameGato Schrödingeremailgatoschrodinger@gmail.combirthdate — Gato de Schrödinger, Oct 10 '20 at 18:48
When your $param1 is an empty array, then it does not reach to the array_merge function, it should return according to your code, your statement is conflicting — Al-Amin, Oct 10 '20 at 18:58
As well ? I'm using the library code. So does lib not support null in its update() method ? — Gato de Schrödinger, Oct 10 '20 at 19:07

Clément Baconnier · Accepted Answer · 2020-10-10T23:43:02.110

2

First of all; Never ever changes the files in the vendor directories. Vendor directories should not be versionned in your project and any changes will not be available in production or for someone else.

As answer; I think using an array should works. And also because it's wrote in comment

// when param 2 is not null we assume that only one set is passed
// like: set( 'name', 'Lu' ); instead of set( array( 'name' => 'Lu' ) );

$birthdate = null;

User::update()
->set('name', $name)
->set('email', $email)
->set(['birthdate' => $birthdate]) // like this               
->where('id', $id)
->execute();

Reason

if (empty($param1)) { // false: because it's not empty. We pass on
   return $this;
}

if ( !is_null( $param2 ) ) { // false: because it IS null. We pass on
   $param1 = array( $param1 => $param2 );
}

At this point we have $param1 = ['birthdate' => null] and $param2 = null But this time, we don't have a string. We have an array.

 // $this->values = array_merge( [], ['birthdate' => null] ); 
    $this->values = array_merge( $this->values, $param1 );

Going further, I think you can even do like this


User::update()
->set([
    'name'      => $name,
    'email'     => $email,
    'birthdate' => $birthdate,
 ])             
->execute();

edited Oct 10 '20 at 23:43

answered Oct 10 '20 at 23:37

Clément Baconnier

5,718
5
29
55

An error has now occurred in another file: "Notice: Array to string conversion in C:\wamp64\www\devsbook\core\Model.php on line 27". That would be in that code : https://paste.laravel.io/9d01fdca-afd0-4596-a0ab-759db4cda2f4 – Gato de Schrödinger Oct 11 '20 at 00:19
I don't know what else to do. This was supposed to be a simple thing – Gato de Schrödinger Oct 11 '20 at 00:25
What the content of the variables ? The line you mentioned `$statement->execute($queryParameters);` [takes an array](https://www.php.net/manual/en/pdostatement.execute.php) if you're using PDOStatement. The error *Array to string conversion* makes no sense for this line – Clément Baconnier Oct 11 '20 at 00:27
I'm starting to seriously think about doing an improvisation (I don't know what the correct English word would be). – Gato de Schrödinger Oct 11 '20 at 00:35
please make a `var_dump($variable);` and `die;` Based on the line 52, it looks like it happens *before* the update with something unrelated. – Clément Baconnier Oct 11 '20 at 00:35
sorry, I didn't understand where I would have to put this var_dump – Gato de Schrödinger Oct 11 '20 at 00:37
Line 19. Before `$statement = $connection->prepare($queryString);` `var_dump($queryString);` ``var_dump($queryParameters);`. – Clément Baconnier Oct 11 '20 at 00:39
*Array to string conversion* Is most often related when you try to echo an array like this `echo $myArray;` instead of `echo $myArray['key']`. Are you certain you didn't add something that could cause that somewhere? – Clément Baconnier Oct 11 '20 at 00:42
Notice: Undefined variable: queryString in C:\wamp64\www\devsbook\core\Model.php on line 19... Notice: Undefined variable: queryParameters in C:\wamp64\www\devsbook\core\Model.php on line 20 – Gato de Schrödinger Oct 11 '20 at 00:42
Line 19 of the link you gave me – Clément Baconnier Oct 11 '20 at 00:43
C:\wamp64\www\devsbook\core\Model.php:20:null – Gato de Schrödinger Oct 11 '20 at 00:43
Sorry for the mistakes. I'm starting my career. – Gato de Schrödinger Oct 11 '20 at 00:45
Does the values are updated in the database though? – Clément Baconnier Oct 11 '20 at 00:53
No. Before I added the set('birthdate', ['birthdate' => $ birthdate]) it updated. It just wouldn't update if I sent the birthdate as null. – Gato de Schrödinger Oct 11 '20 at 00:58
1

Like this `->set(['birthdate' => $birthdate])` – Clément Baconnier Oct 11 '20 at 00:59
1

It worked. Thank you very much. You helped me a lot. All the best for you. – Gato de Schrödinger Oct 11 '20 at 01:03
Did you edit the answer or did I really copy it wrong? – Gato de Schrödinger Oct 11 '20 at 01:04
1

Brazil now loves you :) – Gato de Schrödinger Oct 11 '20 at 01:05
Hahaha best wishes from Switzerland – Clément Baconnier Oct 11 '20 at 01:08
If you want you can answer here too: https://pt.stackoverflow.com/questions/476103/como-passar-um-valor-null-no-m%c3%a9todo-update-do-clancats-hydrahon – Gato de Schrödinger Oct 11 '20 at 01:11
Can you help me with that question ?: https://stackoverflow.com/questions/64356022/how-to-pass-values-by-bind-to-the-same-suffix-that-is-repeated-in-the-querie – Gato de Schrödinger Oct 14 '20 at 15:28

Jsowa · Answer 2 · 2020-10-10T20:01:42.480

1

This is because you pass null as $birthdate. That code is designed for these inputs:

$param1 = 'name';
$param2 = 'value';

or an array.

$param1 = [ 'name' => 'value' ];
$param2 = null;

You must send something as a value.

You should do it either

User::update()
->set('name', $name)
->set('email', $email)
->where('id', $id)
->execute();

or

$birthdate = 'YYYY-MM-DD'; //some date, can't be null

User::update()
->set('name', $name)
->set('email', $email)
->set('birthdate', $birthdate)                
->where('id', $id)
->execute();

or

$params = [
     'name' => $name,
     'email' => $email
];

if(!is_null($birthdate)) {
    $params['birthdate'] = $birthdate;
}

User::update()
->set($params)              
->where('id', $id)
->execute();

edited Oct 10 '20 at 20:01

answered Oct 10 '20 at 19:25

Jsowa

9,104
5
56
60

But I did not understand. Wouldn't Null be a value? How would I get my code to work? – Gato de Schrödinger Oct 10 '20 at 19:41
I edited my answer. I hope it clarifies your doubts. – Jsowa Oct 10 '20 at 19:46
I appreciate the idea. However, this does not solve the null problem. That doesn't seem to be very feasible, does it? – Gato de Schrödinger Oct 10 '20 at 19:49
You can define `$birthdate` as empty string `''`, so then your code will work, but field birthdate will be cleared in database. – Jsowa Oct 10 '20 at 19:51
You have many solutions. It depends exactly what do you need field birthdate for. – Jsowa Oct 10 '20 at 19:52
Empty string does not work. I've tried to pass. I need the user to be able to inform the date of birth field, but he can stop informing if he wants to. – Gato de Schrödinger Oct 10 '20 at 19:56
I added a new way. And there plenty of other methods how you can implement it. You only must choose it and if `$birthdate` is null you can't set that field in database. – Jsowa Oct 10 '20 at 20:02
Your answer had also been useful to me. I only realized it now. I gave an upvote. Thank you – Gato de Schrödinger Oct 11 '20 at 02:56

How to pass a null value in the clancats hydrahon update()?

2 Answers2