Bug 1018173

Summary: I18NTextImpl doesn't update shortText when text is updated
Product: [Retired] JBoss BPMS Platform 6 Reporter: Toshiya Kobayashi <tkobayas>
Component: jBPM CoreAssignee: Maciej Swiderski <mswiders>
Status: CLOSED CURRENTRELEASE QA Contact: Ivo Bek <ibek>
Severity: medium Docs Contact:
Priority: high    
Version: 6.0.0CC: kverlaen
Target Milestone: ER6   
Target Release: 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1026838 (view as bug list) Environment:
Last Closed: 2014-08-06 20:12:49 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:
Bug Depends On:    
Bug Blocks: 1008824, 1026838    
Attachments:
Description Flags
the process definition with a single human task none

Description Toshiya Kobayashi 2013-10-11 11:15:13 UTC 
Description of problem:

Platform BZ for https://issues.jboss.org/browse/JBPM-4148

Steps to Reproduce:
1. Get a Task with taskService.getTaskById()
2. Modify task name of the Task with InternalI18NText.setText()
3. Get a TaskSummary with taskService.getTasksAssignedAsPotentialOwner()

Test case in https://github.com/droolsjbpm/jbpm/pull/181 would explain well.

Actual results:

TaskSummary.getName() returns old task name.

Expected results:

TaskSummary.getName() returns updated task name.
Comment 1 JBoss JIRA Server 2013-11-18 18:58:24 UTC 
Maciej Swiderski <swiderski.maciej> updated the status of jira JBPM-4148 to Resolved
Comment 2 JBoss JIRA Server 2013-11-18 18:58:24 UTC 
Maciej Swiderski <swiderski.maciej> made a comment on jira JBPM-4148

pull request has been merged already in master, thanks Toshiya
Comment 3 Kris Verlaenen 2013-11-29 05:18:32 UTC 
Maciej, has this been merged in master and 6.0.x?
Comment 4 Maciej Swiderski 2013-11-29 11:30:27 UTC 
yes, Toshiya's pull request was merged and back ported to 6.0.x

jbpm
master:
https://github.com/droolsjbpm/jbpm/commit/4e106e9ab7724c13c7a7908c5d4d01f540b7dcd7

6.0.x:
https://github.com/droolsjbpm/jbpm/commit/3ec9b8c0b41350c47eca436a08d2d6e979ad6d17

marking it as modified
Comment 5 Ivo Bek 2014-01-03 12:20:57 UTC 
I return this bug because the name of task will not change when I update it. Here is my test case http://pastebin.com/CYH5QRhu and I will attach the process definition.
Comment 6 Ivo Bek 2014-01-03 12:21:48 UTC 
Created attachment 844935 [details]
the process definition with a single human task
Comment 7 Maciej Swiderski 2014-01-03 12:34:55 UTC 
Ivo,

this is expected as you need to remember that to change entity and if you change it outside of the transaction changes won't be saved in db.
Take a look here for both in transaction and outside of transaction tests for this case:
https://github.com/droolsjbpm/jbpm/blob/master/jbpm-human-task/jbpm-human-task-core/src/test/java/org/jbpm/services/task/TaskQueryServiceBaseTest.java#L934
Comment 8 Ivo Bek 2014-01-03 13:09:16 UTC 
Hi Maciej,

thank you for the explanation, it makes sense and after I used the transactions, it works, so I change the status to verified.