[16.0][FIX] fs_attachment: Avoid local variable redefine

OCA / storage

GNU Affero General Public License v3.0

66 stars 148 forks source link

[16.0][FIX] fs_attachment: Avoid local variable redefine #306

Closed lmignon closed 7 months ago

lmignon commented 7 months ago

When manipulating the file system api through a local variable named fs, we observed some strange behavior when it was wrongly redefined in an enclosing scope as in the following example: with fs.open(...) as fs. This PR fixes this issue by renaming the local variable and therefore avoiding the name clash.

lmignon commented 7 months ago

/ocabot merge patch

OCA-git-bot commented 7 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-306-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 7 months ago

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