Bug 1018173 - I18NTextImpl doesn't update shortText when text is updated
Summary: I18NTextImpl doesn't update shortText when text is updated
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BPMS Platform 6
Classification: Retired
Component: jBPM Core
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ER6
: 6.0.0
Assignee: Maciej Swiderski
QA Contact: Ivo Bek
URL:
Whiteboard:
Depends On:
Blocks: 1008824 1026838
TreeView+ depends on / blocked
 
Reported: 2013-10-11 11:15 UTC by Toshiya Kobayashi
Modified: 2018-12-03 20:15 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
: 1026838 (view as bug list)
Environment:
Last Closed: 2014-08-06 20:12:49 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
the process definition with a single human task (5.54 KB, application/xml)
2014-01-03 12:21 UTC, Ivo Bek
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1171134 0 high CLOSED Changed task name is not propagated to TaskSummary 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker JBPM-4148 0 Major Resolved I18NTextImpl doesn't update shortText when text is updated 2016-07-19 02:53:41 UTC

Internal Links: 1171134

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.


Note You need to log in before you can comment on or make changes to this bug.