Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] fs_image: remove rawCacheKey invalidation #305

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

sbejaoui
Copy link
Contributor

@sbejaoui sbejaoui commented Nov 29, 2023

Fix view crash when uploading an image

The rawCacheKey is appropriately managed by the base class and reflects the
record's last update datetime (write_date).
Since it lacks a setter, attempting to invalidate its value results in a view crash.
Nevertheless, the value will automatically be updated upon saving the record.

@OCA-git-bot
Copy link
Contributor

Hi @lmignon,
some modules you are maintaining are being modified, check this out!

@sbejaoui sbejaoui changed the title [FIX] fs_image: rawCacheKey is properly managed by STD [FIX] fs_image: remove rawCacheKey invalidation Nov 29, 2023
@lmignon
Copy link
Sponsor Contributor

lmignon commented Nov 29, 2023

@sbejaoui can you add a fragment to include into the addon history... as it's done here https://github.com/OCA/storage/pull/304/files#diff-0c0256bf7a5f02140833fcc325442a42553aac6aaf8b013258f347d6ce2d7e8c

@sbejaoui sbejaoui force-pushed the 16.0-fs_image-sbj branch 2 times, most recently from 0f57a90 to 7b3d5b6 Compare December 2, 2023 08:51
@sbejaoui
Copy link
Contributor Author

sbejaoui commented Dec 2, 2023

@sbejaoui can you add a fragment to include into the addon history... as it's done here https://github.com/OCA/storage/pull/304/files#diff-0c0256bf7a5f02140833fcc325442a42553aac6aaf8b013258f347d6ce2d7e8c

done

fs_image/readme/newsfragments/305.bugfix Outdated Show resolved Hide resolved
rawCacheKey is properly managed by STD
@lmignon
Copy link
Sponsor Contributor

lmignon commented Dec 2, 2023

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-305-by-lmignon-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit adbb461 into OCA:16.0 Dec 2, 2023
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at bccc50c. Thanks a lot for contributing to OCA. ❤️

@lmignon lmignon deleted the 16.0-fs_image-sbj branch December 2, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants