1. Mailing Lists
  2. Contributors
  3. Re: Problem when 2 person click same button on the same state at the same time.

Contributors mailing list archives

contributors@odoo-community.org

Browse archives

l10n-estonia status and proposing myself as PSC

Re: Problem when 2 person click same button on the same state at the same time.

Avatar

Re: Problem when 2 person click same button on the same state at the same time.

by
Ecosoft Co. Ltd., Kitti Upariphutthiphong
- 10/06/2022 06:32:12
Thanks Raph for confirming. 

I am also testing, def _check_concurrency(self): without success

Seems to be useless without passing __last_update from the client side. May we we keep our hope on v16 OWL.


On Fri, Jun 10, 2022 at 10:46 AM Raphaël Valyi <rvalyi@akretion.com> wrote:
Yes an old one... A few years ago, some Odoo employee told me they planned to push live changes to all Odoo open sessions (server push). Might be a way to make such possibility extremely unlikely enough. But so far it seems they didn't developp the idea further. I guess the web client JS/OWL overhaul would come first anyway.

Well in the places were this is extremely important like accounting move posting or stock reservation, we should ensure we have some sort of checking. In the other places, cross fingers, detect and fix later, try to partition the records processing over the users in ways that tend to avoid concurrent updates...

May be leaving tabs open forever should also be avoided. May be some minimal autoresfresh could help and cost little dev...

On Thu, Jun 9, 2022, 3:42 PM Kitti U. <kittiu@ecosoft.co.th> wrote:
Found this problem discussed wayed back by our OCA people, https://github.com/odoo/odoo/issues/6737

After reading, sound not quite possible, does it?

On Fri, Jun 10, 2022 at 1:02 AM Kitti U. <kittiu@ecosoft.co.th> wrote:
OK, I think it called the page is "Dirty"

I found from ERPNext on the same scenario, the 2nd person attempt to update, it will show warning like following,

Error: Document has been modified after you have opened it (2022-06-10 00:59:10.014101, 2022-06-10 00:59:24.963271). Please refresh to get the latest document.

This would be good too.

On Fri, Jun 10, 2022 at 12:45 AM Kitti U. <kittiu@ecosoft.co.th> wrote:
Yes Tom, but with that we need to write checking code everywhere. I wonder if there are better way (or yet some base server module in OCA).

I know that in other big name ERP., such an issue was prevented at the framework level, no way it can happen at all. Such things, when occur in accounting documents, can be a serious one.

Another thing (of the same technical shallowness of Odoo) I experienced iwas, user somehow click reverse document twice (because slow response), and it produce reverse entries twice!

On Fri, Jun 10, 2022 at 12:26 AM Tom Blauwendraat <tom@sunflowerweb.nl> wrote:

Regular way is just to check if the sale order is already confirmed, and then don't increase the sequence again.

On 6/9/22 19:22, Kitti U. wrote:
Dear community,

I don't know technical word to describe this. As we knew, Odoo's view is not refreshed in real time as data got updated backend. And this is problematic.

For example,
  • 2 users open the same document at state Draft, and a Confirm Button visible.
  • 1st user clicks Confirm, and document sequence set to 1001
  • 2nd user still see the same Confirm button (as page not refreshed), and click it, sequence now change to 1002 without consent of the first person.
Because Odoo don't have mechanism to prevent this by the framework, and this can happen easily on less mature code.

Anyone have good techniques to prevent this, rather than writing the blocking everywhere?

Thanks!

_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe

_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe

_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe

_______________________________________________
Mailing-List: https://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: https://odoo-community.org/groups?unsubscribe

Reference