Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343563 - task list does not show bug as closed if custom transitions are used
Summary: task list does not show bug as closed if custom transitions are used
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.6   Edit
Assignee: Frank Becker CLA
QA Contact: Frank Becker CLA
URL:
Whiteboard:
Keywords:
: 341831 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-04-21 10:54 EDT by Severin Gehwolf CLA
Modified: 2011-06-08 13:13 EDT (History)
3 users (show)

See Also:


Attachments
my transition file (1.02 KB, text/plain)
2011-05-10 16:01 EDT, Frank Becker CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Severin Gehwolf CLA 2011-04-21 10:54:07 EDT
Mylyn usually shows closed bugs crossed off. This isn't the case for closed bugs on a Bugzilla instance using custom transitions.

Steps to reproduce:
1.) Create a Red Hat Bugzilla task repo. URL: https://bugzilla.redhat.com/
2.) Use custom transition file: http://fedorapeople.org/~jerboaa/mylyn_rh_bugzilla_file.txt
3.) "Open Repository Task...", enter bug id 696813 (this bug is CLOSED, resolution NEXTRELEASE).

Expected results:
Bug is shown crossed off in Task List view.

Actual results:
Bug not crossed off.

The following screenshot illustrates the issue:
http://fedorapeople.org/~jerboaa/eclipse-mylyn-not-showing-bug-closed-custom-transitions.png
Comment 1 Steffen Pingel CLA 2011-04-21 11:27:42 EDT
Severin, which version of Mylyn are you using? We had a similar report a while back and some fixes were applied:  bug 324349: custom closed statuses are never used.
Comment 2 Severin Gehwolf CLA 2011-04-21 11:45:34 EDT
It's version 3.5.0. Looking at bug 324349, shouldn't this be fixed? Is there a problem with the custom transition file we are using?
Comment 3 Frank Becker CLA 2011-05-10 14:10:59 EDT
I found the reason for your problem.

You need to add all other close status to the transition file

If you add

ClosedCustomStatus=CLOSED

your problem is fixed.
Comment 4 Severin Gehwolf CLA 2011-05-10 14:57:20 EDT
(In reply to comment #3)
> I found the reason for your problem.
> 
> You need to add all other close status to the transition file
> 
> If you add
> 
> ClosedCustomStatus=CLOSED
> 
> your problem is fixed.

Hmm, I don't seem to get this working. Have you tried this? It does not seem to work for me: Red Hat Bugzilla #698380 is an example. Here is the content of my current transition file:

CustomStatusNames=true
DuplicateStatus=CLOSED
ClosedCustomStatus=CLOSED
ClosedCustomStatus=POST
<transitions>
name :UNCONFIRMED:, can_change_to :NEW,ASSIGNED,RESOLVED,MODIFIED,ON_DEV,ON_QA,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :NEW:, can_change_to :ASSIGNED,RESOLVED,MODIFIED,ON_DEV,ON_QA,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :ASSIGNED:, can_change_to :MODIFIED,ON_DEV,ON_QA,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :MODIFIED:, can_change_to :NEW,ASSIGNED,ON_DEV,ON_QA,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :ON_DEV:, can_change_to :NEW,ASSIGNED,MODIFIED,ON_QA,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :ON_QA:, can_change_to :NEW,ASSIGNED,MODIFIED,ON_DEV,VERIFIED,RELEASE_PENDING,POST,CLOSED:
name :RELEASE_PENDING:, can_change_to :NEW,ASSIGNED,MODIFIED,ON_DEV,ON_QA,VERIFIED,POST,CLOSED:
name :VERIFIED:, can_change_to :NEW,ASSIGNED,MODIFIED,ON_DEV,ON_QA,RELEASE_PENDING,POST,CLOSED:
name :POST:, can_change_to :NEW,ASSIGNED,MODIFIED,ON_DEV,ON_QA,VERIFIED,RELEASE_PENDING,CLOSED:
name :CLOSED:, can_change_to :ASSIGNED,VERIFIED:

Does this work for you?
Comment 5 Frank Becker CLA 2011-05-10 16:01:39 EDT
Created attachment 195269 [details]
my transition file

here is my transition file.

For me this work in both cases 

1)

ClosedCustomStatus=CLOSED
ClosedCustomStatus=POST

2)

ClosedCustomStatus=POST
ClosedCustomStatus=CLOSED
Comment 6 Severin Gehwolf CLA 2011-05-10 16:29:32 EDT
(In reply to comment #5)
> Created attachment 195269 [details]
> my transition file
> 
> here is my transition file.
> 
> For me this work in both cases 
> 
> 1)
> 
> ClosedCustomStatus=CLOSED
> ClosedCustomStatus=POST
> 
> 2)
> 
> ClosedCustomStatus=POST
> ClosedCustomStatus=CLOSED

Ok, confimed. It works, thanks! This stuff (transition file) is really fragile, though. Not sure what was wrong with my transition file. Maybe that's just me :)
Comment 7 Frank Becker CLA 2011-05-11 00:21:00 EDT
(In reply to comment #6)
> (In reply to comment #5)
> > Created attachment 195269 [details] [details]
> > my transition file
> > 
> > here is my transition file.
> > 
> > For me this work in both cases 
> > 
> > 1)
> > 
> > ClosedCustomStatus=CLOSED
> > ClosedCustomStatus=POST
> > 
> > 2)
> > 
> > ClosedCustomStatus=POST
> > ClosedCustomStatus=CLOSED
> 
> Ok, confimed. It works, thanks! This stuff (transition file) is really fragile,
> though. Not sure what was wrong with my transition file. Maybe that's just me
> :)

Or you have not update the configuration after your changes!
Comment 8 Severin Gehwolf CLA 2011-05-11 09:45:28 EDT
(In reply to comment #7)
> Or you have not update the configuration after your changes!

Thanks, but I'm pretty sure I did that.
Comment 9 Steffen Pingel CLA 2011-05-16 11:36:06 EDT
*** Bug 341831 has been marked as a duplicate of this bug. ***
Comment 10 Steffen Pingel CLA 2011-05-16 11:37:39 EDT
Can we close this? Severin, please feel free to update the documentation on the Wiki if there is anything that would help others to understand the format more easily.
Comment 11 Severin Gehwolf CLA 2011-05-16 11:54:30 EDT
(In reply to comment #10)
> Can we close this? Severin, please feel free to update the documentation on the
> Wiki if there is anything that would help others to understand the format more
> easily.

Steffen, feel free to close this bug.

I would update the documentation, but I'm not feeling knowledgeable enough to do so. Maybe Frank could elaborate on what each line does. Examples usually help.
Comment 12 Frank Becker CLA 2011-05-29 14:12:13 EDT
(In reply to comment #11)
> (In reply to comment #10)
> > Can we close this? Severin, please feel free to update the documentation on the
> > Wiki if there is anything that would help others to understand the format more
> > easily.
> 
> Steffen, feel free to close this bug.
> 
> I would update the documentation, but I'm not feeling knowledgeable enough to
> do so. Maybe Frank could elaborate on what each line does. Examples usually
> help.

I update the documentation.

Please see http://wiki.eclipse.org/Mylyn/Bugzilla_Connector#Custom_Transitions
Comment 13 Steffen Pingel CLA 2011-06-08 13:12:08 EDT
Reopening to fix assignment.
Comment 14 Steffen Pingel CLA 2011-06-08 13:13:09 EDT
Closing.