Bug 1297383

Summary: Share Process Image / PDF doesn't work
Product: [Retired] JBoss BPMS Platform 6 Reporter: Jeremy Lindop <jlindop>
Component: jBPM DesignerAssignee: Tihomir Surdilovic <tsurdilo>
Status: CLOSED EOL QA Contact: Kirill Gaevskii <kgaevski>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.2.0   
Target Milestone: DR2   
Target Release: 6.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-03-27 19:44:59 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:
Attachments:
Description Flags
ProcessImageData
none
ProcessPDFData none

Description Jeremy Lindop 2016-01-11 11:13:04 UTC
Created attachment 1113592 [details]
ProcessImageData

Description of problem:
Share Process Image / Share Process PDF generates invalid html

Version-Release number of selected component (if applicable):
bpms 6.2.0.GA


Steps to Reproduce:
1. Open a BP
2. Run the button-bar menu options "Share Process Image" / "Share Process PDF" 
3. The popup dialog contains a very short html string, without the Image or PDF, e.g.
Share Process Image:
<img src="data:image/png;base64,[B@435f416b">

Share Process PDF:
<object data="data:application/pdf;base64,[B@3c4d6fb3" type="application/pdf"></object>


Actual results:
Invalid image / pdf in the html

Expected results:
Valid base63 image / pdf data should be in the html.
See attachments for examples. These can be placed in an html page to display the process image / pdf.

Additional info:

Comment 1 Jeremy Lindop 2016-01-11 11:13:29 UTC
Created attachment 1113593 [details]
ProcessPDFData

Comment 3 Jeremy Lindop 2016-01-26 16:36:42 UTC
Fixed
master: https://github.com/droolsjbpm/jbpm-designer/commit/4302587ec9f56e74f14b8da4eff103400cf532b9

This commit was made before the 6.4.x branch was created, and is therefore in the 6.4.x branch.

Comment 4 Kirill Gaevskii 2016-03-02 11:28:52 UTC
Hi Jeremy,

Good job! But I have two marks:

1. I guess it will be better if we can add something like style="height:100%;width:100%" for pdf object. It is too small by default and needs some addition settings.

2. The second but more important: IE11 not support PDF objects. We have to create new Documentation issue to reflect this aspect, new issue about it or store PDF file on server side and include it with iFrame HTML tag. What do you think?

Comment 5 Jeremy Lindop 2016-03-21 17:07:17 UTC
1. The height and width have been added to the pdf object.

master: https://github.com/droolsjbpm/jbpm-designer/commit/3db10a970d28b2fab32dac7da314ac8203216d65

6.4.x: https://github.com/droolsjbpm/jbpm-designer/commit/48219691cb2ec3742fd0c915c75718fda799c3b6

2. I've created a new Documentation BZ https://bugzilla.redhat.com/show_bug.cgi?id=1319863
Given that this section of the docs is very brief, I've suggested that a brief note about IE11 be added.

Comment 6 Kirill Gaevskii 2016-04-01 13:57:12 UTC
Hi Jeremy,

thank you for your efforts, it is looks good now.