Skip to content

[PORT] Port base_suspend_security to 9.0 - #333

Merged
pedrobaeza merged 2 commits into
OCA:9.0from
acsone:9.0-port-base_suspend_security-ape
Feb 26, 2016
Merged

[PORT] Port base_suspend_security to 9.0#333
pedrobaeza merged 2 commits into
OCA:9.0from
acsone:9.0-port-base_suspend_security-ape

Conversation

@adrienpeiffer

Copy link
Copy Markdown

No description provided.

@pedrobaeza pedrobaeza mentioned this pull request Jan 15, 2016
59 tasks
@gurneyalex

Copy link
Copy Markdown
Member

@adrienpeiffer can you also bump the version number in openerp.py ?

@gurneyalex gurneyalex added this to the 9.0 milestone Jan 21, 2016
@lmignon

lmignon commented Jan 22, 2016

Copy link
Copy Markdown
Contributor

@adrienpeiffer @gurneyalex @hbrunn IMO We should advertise that this module must be used with caution in a multi companies environment. Indeed, when you suspend the security you also suspend record rules and in some cases I've observed that the value of some fields is not linked to the current company.

@pedrobaeza

Copy link
Copy Markdown
Member

And a workaround/solution is to include in the context the key force_company with the id of the company that must be treated.

@lmignon

lmignon commented Jan 22, 2016

Copy link
Copy Markdown
Contributor

@pedrobaeza I'm not sure it's a solution since the record rules are not run at all...

@pedrobaeza

Copy link
Copy Markdown
Member

Well, it's not a complete solution for example for many2many records (like the taxes in the product), but it works with property/company_dependent fields.

@hbrunn

hbrunn commented Jan 22, 2016

Copy link
Copy Markdown
Member

@lmignon how is sudo()'s behavior here? I think we should do what this function does for consistency

@lmignon

lmignon commented Jan 29, 2016

Copy link
Copy Markdown
Contributor

@gurneyalex version bumped

@hbrunn

hbrunn commented Feb 1, 2016

Copy link
Copy Markdown
Member

👍

2 similar comments
@sebalix

sebalix commented Feb 26, 2016

Copy link
Copy Markdown
Contributor

👍

@pedrobaeza

Copy link
Copy Markdown
Member

👍

pedrobaeza added a commit that referenced this pull request Feb 26, 2016
@pedrobaeza
pedrobaeza merged commit ffdee61 into OCA:9.0 Feb 26, 2016
StefanRijnhart pushed a commit to StefanRijnhart/server-tools that referenced this pull request Feb 26, 2017
…efactor

[RFR] in order to work properly, we need to use FieldOne2Many's caching logic
@sbidoul
sbidoul deleted the 9.0-port-base_suspend_security-ape branch January 7, 2020 17:13
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (11.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants