Overrides getAttributesForInsert & getDirtyForUpdate instead of performInsert & performUpdate to

[RomainMazB]

This fix #87 by decreasing the code override from the Eloquent Model class.

• Instead of overriding the whole performInsert method, we now override the getAttributesForInsert method which only cares about getting the attributes to be inserted into the database.
• Instead of overriding the whole performUpdate method, we now override the getDirtyForUpdate method which only cares about getting the modified attributes to be updated into the database.

[saibotk]

Wow, thank you very very much for the quick response on this! Also this looks very good and much better.

Can we maybe add a simple test case to prevent regressions in the future? Something like a quick model event test? Also a small entry in the changelog would be nice.

Thank you so much!

[saibotk]

I just saw, we do not have a proper eloquent test ready, so see this as optional if it is too much for now, we will do it later anyway.

[RomainMazB]

I'll write a test to avoid future regression, it's not a problem.

Probably during the weekend or on Monday.

[RomainMazB]

Just re-thinking about it.

Adding database tests means that github actions will now need to run postgres + postgis to valid the tests.

I am not familiar with github actions so I can't provide a PR on that. What do you recommend?

[saibotk]

Dont worry i can add those later thank you 🙏

[saibotk]

Oh @RomainMazB, can you quickly run composer fix and add an entry to the CHANGELOG.md about this fix?

If not i can also do that :)

[maximepvrt]

@saibotk can you release this fix ?

[saibotk]

done @maximepvrt sry should have done that sooner!