Bug 1195145
| Summary: | Unable to download document attachments in proc.inst.perspective | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Retired] JBoss BPMS Platform 6 | Reporter: | Jan Hrcek <jhrcek> | ||||||||||
| Component: | Business Central | Assignee: | Pere Fernàndez <pere.fernandez> | ||||||||||
| Status: | CLOSED EOL | QA Contact: | Lukáš Petrovický <lpetrovi> | ||||||||||
| Severity: | urgent | Docs Contact: | |||||||||||
| Priority: | high | ||||||||||||
| Version: | 6.1.0 | CC: | kverlaen, pere.fernandez, rrajasek | ||||||||||
| Target Milestone: | CR2 | ||||||||||||
| Target Release: | 6.1.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 20:02:03 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: | |||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Jan Hrcek
2015-02-23 08:58:56 UTC
Fixed in 6.2.x jbpm and jbpm-console-ng: http://github.com/droolsjbpm/jbpm/commit/73bc3e802 http://github.com/droolsjbpm/jbpm-console-ng/commit/4346240b5 Checking with BPM Suite 6.1.0 ER6. I can see the download link in the Process Instance Details > Documents Tab, but this link is now working ONLY for processes instances that are completed. For running process instances that have some document uploaded (e.g. a process for which document was uploaded via process start form), the document link is there, but it has invalid URL (of the form "http://localhost:8080/business-central/org.kie.workbench.KIEWebappnull"). Clicking this link naturally causes HTTP 404. Interesting is, that when I COMPLETE that process, the document link becomes valid. See the behavior in the video I attach and please review the fix. Created attachment 1000322 [details]
VIdeo illustrating new issue
Fixed changes Document creation in order to make the MarshallingStrategy generate it with the link & identifier. Commits (master): https://github.com/droolsjbpm/jbpm/commit/65cbaec173ccf489bd02d237a356e3588bb76f1e https://github.com/droolsjbpm/jbpm-form-modeler/commit/3fcf705cb0131ec6a8c220b9bbae0f65f5122635 https://github.com/droolsjbpm/droolsjbpm-knowledge/commit/8b35faf712020767c4a10b330aece7d0777783c7 Commits (6.2.x): https://github.com/droolsjbpm/jbpm/commit/0b286b7e0390aeec65cf3e1161532e707ad08a74 https://github.com/droolsjbpm/jbpm-form-modeler/commit/b40bde60730d55da3204bc5befca2120ac1e4485 https://github.com/droolsjbpm/droolsjbpm-knowledge/commit/8d48d6c919e734be8e55dd6bed3bee0cfb695376 Checking with BPM Suite 6.1 CR1. This is still not working as expected. Now the document links in Process Instance Details > Documents tab are NEVER displayed (not for running process, nor for completed process). Please see the current behavior in the 2 videos attached: 1) DocProcess - document uploaded on process start. After that checking Process Instance Details > Documents tab, there is no link to download document (even thought the document link is present in Process Variables tab - but just as a text, not as clickable link) 2) HTDocUpload - document uploaded in task list as a part of human task. After that the process completes. Agan, Process Instance Details > Documents tab does not list any documents (but it's present in the Proc. Variables tab - as in case 1) Please see the videos attached for steps to reproduce. The sources of the 2 processes used & their corresponding forms can be found in our repo of testing artifacts: http://git.app.eng.bos.redhat.com/git/bpms-assets.git/tree/formmodeler/src/main/resources/docfield Created attachment 1006678 [details]
Video: CR1 still has the issue
Created attachment 1006679 [details]
Video: CR1 still has the issue 2
There was an error related with date parsing and the computer locale. Forced a pattern to store the document last modified date. Commits (master): https://github.com/droolsjbpm/jbpm/commit/a00a6da14b7f5d125364a8de29175323e166e272 https://github.com/droolsjbpm/jbpm-console-ng/commit/8dcae2a91ce99ae39b0eebe317a0e31c40525068 Commits (6.2.x): https://github.com/droolsjbpm/jbpm/commit/dd7f970385fa910a1cead8b15279ae392debf7c9 https://github.com/droolsjbpm/jbpm-console-ng/commit/094a237074679331c3da417c9b14b317cf6b9494 Reopened based on https://github.com/droolsjbpm/jbpm/commit/dd7f970385fa910a1cead8b15279ae392debf7c9#commitcomment-10436193 Changed the properties separator for documents from "," to a more complex expression "####" to avoid errors on with the file naming. Commits(master): https://github.com/droolsjbpm/jbpm/commit/286518af64f23ae33e310589d238fe3513073236 https://github.com/droolsjbpm/jbpm-console-ng/commit/8f0847b720519348bb2127bd1b93e05a12630c6a It requires to specify in the documentation that the uploaded files shouldn't have the #### token on it's name. Now it's working fine. Verified with BPM Suite 6.1.0 CR2 |