Add after save callback

[it-mob]

When an API returns a different result than the submitted data, we use "AfterSave" callback to catch this result

[ADmad]

Tests needed for the new feature.

[it-mob]

Where can I find the .sqlite file ?

[ADmad]

What sqlite file? If you are referring to the test suite it uses an in memory sqlite db by default, no file.

[it-mob]

Thank you for the information.
I changed the URL of the sqlite file "webservice_test.sqlite" in the boostrap file with ":memory:" so that the tests start on Windows

[saeideng]

@it-mob
Admad meant you have to add testCase like these https://github.com/UseMuffin/Webservice/tree/master/tests/TestCase
so that this helps to ensure that this change works well now and in future

[it-mob]

@saeideng
Yes sure. I'm working on it.

[it-mob]

I added the tests but I don't understand the error of PHPSTAN during the CI tests

[saeideng]

probably something that fixed in phpstan and now should not be this error in ignore list
you can remove this error from /phpstan.neon file

[it-mob]

Thank you for the answer. I pushed my changes

[it-mob]

Hello,
Do you expect something else from me so that my PR can be validated ?

[ADmad]

Sorry for the delay, I'll review the changes soon.

[ADmad]

Please change this to $event->getResult().

[it-mob]

Thank you for the answer.
I just realized that the callbacks beforeDelete/afterDelete were missing.
I will add them and update my PR soon

[it-mob]

Hello,

I just made a new commit with before & after delete callbacks but I have errors with Travis.
I think the problem is the line break format (CRLF & CF)
Can you help me solve this problem? (I work on Windows)

Thank you.

[it-mob]

Hello,
I wanted to know if you're still expecting something from me about that PR.
I tried again to fix the PHPCS problem but without success.
Thank you

[ADmad]

Sorry, I had forgotten about this. NVM the phpcs issue. I can take care of it.

But I am now leaning towards targeting this enhancement to the cake-4.x branch (which is compatible with CakePHP 4) instead of master.