Bug 999950

Summary: Business-central is single-user application
Product: [Retired] JBoss BRMS Platform 6 Reporter: Zuzana Krejčová <zkrejcov>
Component: Business CentralAssignee: Pedro Zapata <pzapataf>
Status: CLOSED CURRENTRELEASE QA Contact: Zuzana Krejčová <zkrejcov>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 6.0.0CC: etirelli, kverlaen, lpetrovi, manstis, pere.fernandez, rrajasek, rzhang
Target Milestone: CR1   
Target Release: 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 20:18:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 983634, 1001561, 1048265, 1049361, 1049368, 1049396    
Bug Blocks:    

Description Zuzana Krejčová 2013-08-22 12:24:56 UTC
Description of problem:
Similarly to bug 983634 that has been ignored, the whole business-central is single-user only. Collaboration does not work. The last user saving an asset of any type wins.

If two users edit an asset at the same time, the last commit/save overwrites changes made by the first commit/save. Users are in no way warned that someone else is making changes, there are no conflicts in git.

This is not the type of behaviour one would expect of production ready application using git.

I'd expect at least a warning. Warning before editing the asset, in the worst case upon saving the changes (optimistic locking). In that case, user shouldn't be allowed to save the changes, because that would overwrite the previous commit. This would still mean the loss of work, but at least the user would notice.
If both commits/saves are allowed, then there should be some way to merge them. This would be ideal.



Version-Release number of selected component (if applicable):
6.0 ER1

Comment 3 manstis 2013-09-12 13:38:33 UTC
More changes (to drools-wb):

(master) https://github.com/droolsjbpm/drools-wb/commit/4983117df

(6.0.x) https://github.com/droolsjbpm/drools-wb/commit/9208509e4

Other editors to follow.

Comment 6 Lukáš Petrovický 2013-10-23 11:40:47 UTC
Why is this "POST"? What does that even mean?

Is this issue resolved or not?

Comment 7 Edson Tirelli 2013-10-23 13:36:10 UTC
Porcelli, can you please clarify the status of this issue and if it is done, set it to MODIFIED?

Comment 10 Edson Tirelli 2013-11-18 19:59:06 UTC
Raising the priority to get it fixed for the product release.

Comment 13 Zuzana Krejčová 2014-01-03 14:56:22 UTC
I will definitely be looking into this more, but as of now this issue is not fixed. Consider this blocked by at least bug 1048265. If I find more issues, I will, of course, inform you.
As for bug 983634 which is supposedly blocked by this - shouldn't it be the other way? This being blocked by bug 983634?

Comment 14 Pedro Zapata 2014-01-10 15:21:03 UTC
The global issue has been implemented and solved. As agreed during the release status call, it's marked as modified.

There are a few dependent tickets still open should be targeted for 6.0.1 and will be tracked separately.

Comment 16 Zuzana Krejčová 2014-01-27 09:48:31 UTC
(In reply to Pedro Zapata from comment #14)
> The global issue has been implemented and solved. 

Issues with the solution were mostly fixed as well - see bugs that block this. 

Verified with CR2.