Bug 390529 - [api] make taskData available in TaskJobListener methods
Summary: [api] make taskData available in TaskJobListener methods
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.11   Edit
Assignee: Endre Kovács CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, noteworthy
Depends on:
Blocks:
 
Reported: 2012-09-27 04:20 EDT by Endre Kovács CLA
Modified: 2014-03-03 21:47 EST (History)
0 users

See Also:


Attachments
patch cut against the master of task git repo (3.76 KB, patch)
2012-10-06 17:31 EDT, Endre Kovács CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Endre Kovács CLA 2012-09-27 04:20:47 EDT
so, it is very nice to be notified about task submition changes (TaskJobListener), how ever, so far i couldn't get a hold of the changed taskData object that is about to be posted to the server. 

This code will get only the old taskAttributes (and obviously adds dependency on the UI plugin)

TasksUi.getTaskDataManager().getTaskData(event.getTask()).getRoot();

that will be overwritten when the task is submitted.

so, my question is: is it possible / would it make sense to you too: 
to supply the taskData reference for the TaskJobListener (eg.: in the TaskJobEvent) which will be submited to the repository?

thank you for your consideration
Comment 1 Steffen Pingel CLA 2012-10-05 04:14:10 EDT
That sounds reasonable idea to me. Would you be able to push a change to Gerrit with the proposed implementation?
Comment 2 Endre Kovács CLA 2012-10-06 17:31:16 EDT
Created attachment 221988 [details]
patch cut against the master of task git repo

i am sorry but i couldn't set up this whole gerrit thing.
however here is my patch on top of commit
3f5048053279a0b69ec70deba5a3e4a06a4ee467
in the mylyn.tasks repo
can you please look at/push this to gerrit?
thanks
Endre
Comment 3 Sam Davis CLA 2012-10-10 18:52:43 EDT
I have pushed this to Gerrit: 8155: 390529: make taskData available in TaskJobListener methods [I67c2d79d]
https://git.eclipse.org/r/#change,8155
Comment 4 Steffen Pingel CLA 2012-10-30 06:56:30 EDT
Endre, what kind of problems did you run into when configuring Gerrit? Please post to mylyn-integrators or to this bug if there is anything we can help with. 

I have commented on the code review.
Comment 5 Endre Kovács CLA 2012-11-08 11:33:52 EST
i managed to set up my environment and able to push to gerrit.
i pushed the required changes.

please review it @ https://git.eclipse.org/r/#/c/8155/
thanks
Endre
Comment 6 Sam Davis CLA 2014-02-28 14:57:19 EST
The changes have been merged. Thanks very much for the contribution! Leaving this open to cherry-pick.
Comment 7 Sam Davis CLA 2014-03-03 20:43:56 EST
Closing and tagging with [backport].
Comment 8 Sam Davis CLA 2014-03-03 21:47:52 EST
Cherry-picked to 3.11 branch.