Skip to content

[FIX] The duplicated name check compares root Directories with non-root Directories - #526

Open
SirPyTech wants to merge 1 commit into
OCA:18.0from
PyTech-SRL:18.0-fix-dms-storage_root_directories
Open

SirPyTech wants to merge 1 commit into
OCA:18.0from
PyTech-SRL:18.0-fix-dms-storage_root_directories

Conversation

@SirPyTech

Copy link
Copy Markdown

Steps:

  1. Inside a directory, create a directory "Test"
  2. Create a root directory "Test"

Current:
WARNING devel odoo.http: A directory with the same name already exists.

Expected:
The directory can be created.

Additional Notes:
The root cause is that the storage's root directories are all the storage directories and the commit focuses on that.

Comment thread dms/models/storage.py Outdated
@SirPyTech
SirPyTech requested a review from victoralmau October 15, 2026 08:45
Comment thread dms/models/storage.py Outdated
@SirPyTech
SirPyTech force-pushed the 18.0-fix-dms-storage_root_directories branch from 7243818 to a43982e Compare October 15, 2026 08:55
@SirPyTech
SirPyTech requested a review from victoralmau October 15, 2026 08:57

@HekkiMelody HekkiMelody left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review, LGTM

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.

4 participants