Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

deleted_by not set to null on restoring #6

Open
nstapelbroek opened this issue Aug 20, 2018 · 1 comment
Open

deleted_by not set to null on restoring #6

nstapelbroek opened this issue Aug 20, 2018 · 1 comment

Comments

@nstapelbroek
Copy link
Owner

nstapelbroek commented Aug 20, 2018

Original: https://github.com/nstapelbroek/culpa/issues/12

Hi could you please add

public function restoring(Model $model) {
    $model->{$this->getColumn($model, 'deleted')} = null;
}

to the BlameableObserver, in order to also restore the deleted_by column when restoring a soft-deleted model

Thank you

@nstapelbroek
Copy link
Owner Author

Hi @BraunLukas ,

Thank you for your suggestion. Feel free to submit a Pull Request with the change and a small test to proof your issue is fixed.

I'm happy to assist with this. I have some time left at the end of the week if you want to.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant