[18.0][MIG] dms_field: Migration to 18.0#416
Merged
OCA-git-bot merged 44 commits intoOCA:18.0from May 8, 2025
Merged
Conversation
... so to speak. @Tecnativa TT29847
Co-authored-by: luismalta <luis.malta@kmee.com.br>
Approved Enric Tobella OCA#169
Currently translated at 100.0% (75 of 75 strings) Translation: dms-15.0/dms-15.0-dms_field Translate-URL: https://translation.odoo-community.org/projects/dms-15-0/dms-15-0-dms_field/it/
Currently translated at 100.0% (75 of 75 strings) Translation: dms-15.0/dms-15.0-dms_field Translate-URL: https://translation.odoo-community.org/projects/dms-15-0/dms-15-0-dms_field/it/
The root directory can't be opened using the contextual action menu (`open`)
Impacted versions:
12.0 and above
Steps to reproduce:
Go to "DMS directories" in the partner form view
Create a new root directory
Right-click and choose `Open`
Current behavior:
Odoo raises an error `TypeError: record is null`
Expected behavior:
Open the directory like if we click on the `Open` button in the right section (Document preview)
- Create dms.field.template to set model, groups and directory structure. - Remove the wizard to create a directory from a record. - Directory structure is created when record is created (partner for example). - Change the information on the right side. - Have directory/file tree open when loading. TT46804 TT46804
…cut several files and paste them into another directory) TT47482
…n as default state Before this change, when lauching the reload, the child directories of the reloaded one are opened too. By doing at this step and not setting the default state, the reloads are done just for the implied nodes. Furthermore, the code is improved to just reload the directories that have being changed.
Currently translated at 100.0% (75 of 75 strings) Translation: dms-15.0/dms-15.0-dms_field Translate-URL: https://translation.odoo-community.org/projects/dms-15-0/dms-15-0-dms_field/es/
With hr_dms_field The steps to reproduce the problem are: 1. Open Employee 2. Go to Documents Page 3. Drop a file on a directory 4. Go to another Employee 5. Try to drop a file The file can not be dropped.
In the migration, the view type is changed from dms_tree to dms_list. That's because in this point https://github.com/odoo/odoo/blob/16.0/addons/web/static/src/views/fields/field.js#L282 the 'tree' string is replaced by 'list' what is causing problems trying to set the view type as it was before.
A parent directory can be set in templates, if set, the directory that is created linked to a record will be a "child" of that directory. TT48650
…ors when creating directories from a template. Example of use case: - Create a partner. - Create a template linked to partners. - Create a directory through the template. - An access group linked to the partner is created. - Delete the partner directory. - Creates a new directory through the template. - No error will occur and the previous access group will be re-used.
…ring Now the process will be as follows: - The auto-generated access group will be created. - The user will be defined in explicit_user_ids of the group created in the previous step. In the directory created (from the corresponding record) the auto-generated group (for the user to have access) + the access groups that the template had will be used.
Currently translated at 100.0% (62 of 62 strings) Translation: dms-16.0/dms-16.0-dms_field Translate-URL: https://translation.odoo-community.org/projects/dms-16-0/dms-16-0-dms_field/it/
We need to avoid applying a template except when testing functionality with dms_field* modules to avoid the error that a directory with the same name already exists (example: create partner). Related to OCA#378
… if the user does not have access to the linked directory. Example use case: - Define a dms template with an access group and set only Admin as the explicit user (without groups). - Create a "Test employee" employee. - Modify the employee with Marc Demo and link him a new user (test-user) - The auto-generated access group of test-employee will have test-user defined as explicit user. TT55368
CarlosRoca13
force-pushed
the
18.0-MIG-dms_field
branch
from
b131324 to
5f16b4b
Compare
victoralmau
suggested changes
Apr 29, 2025
Member
victoralmau
left a comment
victoralmau
left a comment
There was a problem hiding this comment.
Can you check the following?
Error when trying to create a template.
Error when trying to access a partner's file.
Member
|
/ocabot migration dms_field |
CarlosRoca13
force-pushed
the
18.0-MIG-dms_field
branch
from
5f16b4b to
68deb80
Compare
Contributor
Author
|
@victoralmau the problems should be solved 😄 |
Member
CarlosRoca13
force-pushed
the
18.0-MIG-dms_field
branch
from
68deb80 to
3fc1902
Compare
Contributor
Author
|
Changed @pilarvargas-tecnativa |
pilarvargas-tecnativa
approved these changes
May 8, 2025
Contributor
|
This PR has the |
Member
|
/ocabot merge nobump |
Contributor
|
This PR looks fantastic, let's merge it! |
Contributor
|
Congratulations, your PR was merged at a0de801. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cc @Tecnativa TT55507
ping @victoralmau @pilarvargas-tecnativa