CodeDredd / pinia-orm

The Pinia plugin to enable Object-Relational Mapping access to the Pinia Store.
https://pinia-orm.codedredd.de/
MIT License
444 stars 38 forks source link

fix(pinia-orm): `saving`, `updating` hooks second param had wrong values #1826

Closed CodeDredd closed 4 months ago

CodeDredd commented 4 months ago

🔗 Linked issue

closes #1707

❓ Type of change

📚 Description

The hooks saving, creating, updating, saved, created & updated had a second param record which was wrongly already the new data instead of the old one. The new data is already saved in the model param.

📝 Checklist

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 75.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 99.71%. Comparing base (34348a5) to head (d573aec).

Files Patch % Lines
packages/pinia-orm/src/query/Query.ts 75.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1826 +/- ## ========================================== + Coverage 99.67% 99.71% +0.03% ========================================== Files 87 87 Lines 5934 5934 Branches 508 509 +1 ========================================== + Hits 5915 5917 +2 + Misses 17 14 -3 - Partials 2 3 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.