Bug 801399

Summary: Designer blocked after viewing process diff if the diff pop-up is blocked
Product: [JBoss] JBoss Enterprise BRMS Platform 5 Reporter: Jiri Locker <jlocker>
Component: BRM (Guvnor)Assignee: manstis
Status: VERIFIED --- QA Contact: Jiri Locker <jlocker>
Severity: medium Docs Contact:
Priority: unspecified    
Version: BRMS 5.3.0.GACC: lpetrovi
Target Milestone: ER5   
Target Release: BRMS 5.3.0.GA   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Firefox 3.6; 8
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
blockedDesigner.png
none
updateddiffwindow none

Description Jiri Locker 2012-03-08 12:56:15 UTC
Created attachment 568615 [details]
blockedDesigner.png

Description of problem:
In Firefox, pop-ups are blocked by default. In case the user has not yet allowed pop-ups for Guvnor's host and tries to view a process version diff, error message is displayed and designer's workspace is blocked forever. There is no way to cancel the operation. The only way to continue editing the process is to close the process and open it again. Fortunately, any unsaved work can be saved.

Version-Release number of selected component (if applicable):
BRMS 5.3.0 ER4

How reproducible:
if pop-ups are blocked for Guvnor's host (is default in Firefox)

Steps to Reproduce:
1. make sure pop-ups are blocked and exception for localhost has not been added yet
2. access designer by opening any BPMN2 process or by creating a new one in Guvnor
3. if you have just created the process, save it first so that at least one version is checked in
4. try viewing any version diff, the result should match the attached screenshot
  
Actual results:
"Creating diff..." progress message never disappears and blocks the designer workspace forever

Expected results:
option 1) In case of failure to view the diff (popup is blocked) the progress message should disappear so that user can dismiss the error message and continue working with designer

option 2) don't use pop-ups to view the diff, use HTML the same way it is used to view BPMN2 or JSON source of the process

Additional info:

Comment 1 Tihomir Surdilovic 2012-03-15 01:09:47 UTC
Diff no longer creates a popup and displays the diff inline (see attachment). Fix is checked into Designer master and 2.1.x branch. let me know if you need a war for testing provided.

Comment 2 Tihomir Surdilovic 2012-03-15 01:10:15 UTC
Created attachment 570138 [details]
updateddiffwindow

Comment 3 Jiri Locker 2012-03-15 09:48:21 UTC
I will test with designer in ER5.

Comment 4 Jiri Locker 2012-05-04 15:19:22 UTC
Verified in ER6. Much better, thanks!