18.0 add base_url: mixin tool for managing url and redirection without id#3591
Draft
sebastienbeau wants to merge 51 commits into
Draft
18.0 add base_url: mixin tool for managing url and redirection without id#3591sebastienbeau wants to merge 51 commits into
sebastienbeau wants to merge 51 commits into
Conversation
legalsylvain
reviewed
Apr 2, 2026
| This project was initial build for shopinvader as we need to have uniq url for the connected webshop. | ||
| But this concept can be reuse in other case and even replace the odoo url implementation. | ||
|
|
||
|
|
Contributor
There was a problem hiding this comment.
could you add a DEVELOP.md file to explain how to implement this mixin ?
thanks !
Member
Author
There was a problem hiding this comment.
Yes, thanks for the suggestion, I will created it and add example
arnaudlayec
reviewed
Apr 13, 2026
There was a problem hiding this comment.
(comment removed: see below commit "[FIX] Prevent raising AttributeError because of 'model_id' in 'url.url'")
sebastienbeau
force-pushed
the
18.0-add-url-2
branch
from
fd17684 to
3872648
Compare
This was referenced Apr 28, 2026
Moreover this generate a random issue.
If you want to reproduce this random issue you can make it not random
by forcing the processing of the todo. You can take the following code
diff --git a/odoo/api.py b/odoo/api.py
index 413a6b8..bb63ba8 100644
--- a/odoo/api.py
+++ b/odoo/api.py
@@ -937,6 +937,11 @@ class Environment(Mapping):
The field is such that none of its dependencies must be
recomputed.
"""
field = min(self.all.todo, key=self.registry.field_sequence)
+ if str(field) == 'shopinvader.category.index_id':
+ if len(self.all.todo) > 1:
+ for f in self.all.todo:
+ if str(f) != 'shopinvader.category.index_id':
+ field = f
return field, self.all.todo[field][0]
def check_cache(self):
… product should be able to do it even if the product is online
… safely allows the triggeer of field computation by concrete models
Computation was broken when the parent binding was archived. Moreover this commit eases overrides of depends for the computation.
performance issue, add index
Now we do not have binding anymore, need a new implementation
…t_url add fields to schemas
The dependency on base_sparse_field_list_support is required since we use a Serialized field to store list of values
This is required to avoid race condition when running tests if the base_sparse_field addon has not yet patch the odoo.fields namespace to register the Serialized field class
In a compute method, we must assign a value to the computed field only if it's different from the original one. Otherwise, the write method will be called every time.
paradoxxxzero
force-pushed
the
18.0-add-url-2
branch
from
606aa3c to
9b2d360
Compare
This was referenced May 11, 2026
Open
Dropped base_sparse_field_list_support dependency. Use standard odoo slugify. Tests the computed fields. Documentation
paradoxxxzero
force-pushed
the
18.0-add-url-2
branch
from
112211b to
f2ba676
Compare
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
15 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.
Add a base_url, generic module for
TODO