Soft delete need the rest of the object to update

[dandelionmood]

Hi !

Thanks for your work, it's proven quite useful to me !
I just noticed a little bug in the way the soft delete is written, here's my quick and dirty fix !

Pierre

[leomarquine]

But why are you passing all the items to the query? When soft deleting the only column you should update is the deleted_at column with the current time.

[dandelionmood]

You're right, and that wasn't working either.
I updated the patch accordingly :) !

[leomarquine]

@dandelionmood any news on that?

[dandelionmood]

hi @leomarquine the branch « branch-1 » should be OK !

[leomarquine]

So we really need this? If you don't set the column it will be bull anyway. Do you have any errors without it?

if( $this->delete === 'soft' ) {
    $item['deleted_at'] = null;
}

[leomarquine]

@dandelionmood ?

[dandelionmood]

Sorry, I didn't have time to check it yet.

Pierre