servala/servala-portal
3
0
Fork 0
Code Issues 21 Pull requests 5 Projects 2 Releases Packages 2 Activity Actions

Billing Entity Management #66

Merged
rixx merged 32 commits from 54-billing-entity-management into main 2025-06-22 15:42:14 +00:00
Conversation 8 Commits 32 Files changed 23 +10 -3
1 changed files with 10 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 4718a01bb0 - Show all commits

11
src/servala/core/odoo.py
View file

@ -130,10 +130,17 @@ def get_invoice_addresses(user):
odoo_users = CLIENT.search_read(
model="res.users",
domain=[("login", "=", email), ("active", "=", True)],
fields=["id"],
fields=["id", "share"],
tobru marked this conversation as resolved Outdated
tobru commented 2025-06-05 08:52:57 +00:00
Outdated
Copy link

Enhance the domain to include ('active','=',True) to make sure the user is active in Odoo.

Enhance the domain to include `('active','=',True)` to make sure the user is active in Odoo.
limit=1,
)
if odoo_users and (uid := odoo_users[0].get("id")):
if odoo_users:
odoo_user = odoo_users[0]
tobru marked this conversation as resolved Outdated
tobru commented 2025-06-05 08:56:57 +00:00
Outdated
Copy link

A user in the Servala Portal which matches an internal user in Odoo should be able to see all invoice addresses, like they do when they log in to the Odoo backend. I figured out that the "Internal Users" filter in Odoo uses the domain ('share', '=', False) (L342) so let's use the same to decide if the user can see all invoice addresses. There are also "Portal Users" in Odoo, they should not see all invoice addresses and behave like a normal Servala Portal user.

A user in the Servala Portal which matches an internal user in Odoo should be able to see all invoice addresses, like they do when they log in to the Odoo backend. I figured out that the "Internal Users" filter in Odoo uses the domain `('share', '=', False)` ([L342](https://github.com/odoo/odoo/blob/16.0/odoo/addons/base/views/res_users_views.xml#L342)) so let's use the same to decide if the user can see all invoice addresses. There are also "Portal Users" in Odoo, they should not see all invoice addresses and behave like a normal Servala Portal user.
if odoo_user.get("share") is False:
# An Odoo internal user (share=False) should see all invoice addresses.
or_conditions = []
elif uid := odoo_user.get("id"):
# For portal users or users not in Odoo, apply standard filters.
or_conditions.append(("create_uid", "=", uid))
odoo_contacts = CLIENT.search_read(