Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 39986 - Decorators acting funny
Summary: Decorators acting funny
Status: RESOLVED WORKSFORME
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P2 major with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 40458 40575 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-07-13 03:10 EDT by Wassim Melhem CLA
Modified: 2003-08-14 14:07 EDT (History)
3 users (show)

See Also:


Attachments
missing CVS decorators (5.27 KB, image/gif)
2003-07-13 03:12 EDT, Wassim Melhem CLA
no flags Details
Picture of missing decorators in Java Browsing perspective (164.64 KB, image/jpeg)
2003-07-29 10:09 EDT, Dani Megert CLA
no flags Details
pde.ui is lacking CVS decorators (4.40 KB, image/gif)
2003-08-14 13:56 EDT, Wassim Melhem CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wassim Melhem CLA 2003-07-13 03:10:57 EDT
Build: I-20030710

Decorators are behaving oddly in this build.
First, when I launch Eclipse, they do not appear.  I have to do an explicit 
action, such as open a file for them to appear.
Second, as you can see in the attached screenshot, after synchronizing files 
with the repository, some of the files lost the CVS label decorator.
Comment 1 Wassim Melhem CLA 2003-07-13 03:12:10 EDT
Created attachment 5436 [details]
missing CVS decorators
Comment 2 Tod Creasey CLA 2003-07-14 11:30:15 EDT
This was a problem with lost updates when there were no windows. Fixed for M2.
Comment 3 Tod Creasey CLA 2003-07-14 11:32:17 EDT
Marking fixed
Comment 4 Wassim Melhem CLA 2003-07-15 21:50:03 EDT
Build: I-20030715
The first problem (i.e. decorators do not automatically appear when I launch 
Eclipse) has been fixed.
The second problem though (i.e. some files lose the CVS decorator after 
synchronizing) is occurring much less frequenly, but still occurring 
nevertheless.
I have the Live Synchronize view open, the Package explorer view open, and one 
Java file open: every time I make a modification to the file and save.  The 
file in the Package explorer sometimes loses the decorator and sometimes it 
retains it.  It's pretty random.
Comment 5 Tod Creasey CLA 2003-07-17 13:48:40 EDT
I can't replicate this in 20030717 but I am going to keep this open as a 
precaution. Please let me know if the problem persists.

Also be aware that if using the Synch View it has a higher priority than 
decorators and they may not be run until after the Synchonize is done.
Comment 6 Wassim Melhem CLA 2003-07-18 14:24:22 EDT
I'm still encountering it in the I-20030717(2359) driver.
This time the synchronize view was not even open.  I saved a Java file and the 
CVS decorator vanished.
Once I opened the synchronize view, the UI refreshed, and the decorator 
reappeared.
Comment 7 Tod Creasey CLA 2003-07-22 09:56:21 EDT
Adding Jean-Michel to the list - this does not appear to be a general problem 
with the decorators mechanism but rather an issue with the CVS Decorator not 
getting a result in some cases.

I have been able to replicate this problem after a save which ends up sending 
updates from both the CVS decorator and the resource change listener in the 
packages view but all this should cause is a superfluous extra decoration.
Comment 8 Tod Creasey CLA 2003-07-22 10:46:25 EDT
I have replicated this without the CVS decorator so this is a general problem. 
Removing Jean-Michel from the cc list.
Comment 9 Tod Creasey CLA 2003-07-22 14:39:35 EDT
*** Bug 40575 has been marked as a duplicate of this bug. ***
Comment 10 Tod Creasey CLA 2003-07-22 14:40:48 EDT
This was due to updates being lost due to releasing the result lock before the 
job had finished scheduling. It was indeed a timing issue.

I have released a fix into build >20030722 but will keep this PR open as it 
has been a toughie thus far.
Comment 11 Tod Creasey CLA 2003-07-28 08:11:13 EDT
*** Bug 40458 has been marked as a duplicate of this bug. ***
Comment 12 Dani Megert CLA 2003-07-29 02:49:31 EDT
CVS decorators still don't show up for some items using I20030723
Comment 13 Tod Creasey CLA 2003-07-29 09:40:44 EDT
Dani do you have a specific set of steps to replicate this? The problem is 
usually timing specific so it tends to be certain actions that cause this to 
occur.
Comment 14 Tod Creasey CLA 2003-07-29 09:41:06 EDT
Added Dani in order to get more info
Comment 15 Dani Megert CLA 2003-07-29 10:08:51 EDT
I see this while using Java Browsing perspective. See attached picture.
Comment 16 Dani Megert CLA 2003-07-29 10:09:46 EDT
Created attachment 5578 [details]
Picture of missing decorators in Java Browsing perspective
Comment 17 Tod Creasey CLA 2003-07-29 10:21:32 EDT
This looks like the same problem with missing updates as we had before. I have 
made another pass over the code and found some more potential problems that I 
have fixed up but I would feel better if I could replicate the issue.


Where you doing any CVS related operations while doing this? Or just editing 
and saving?
Comment 18 Dani Megert CLA 2003-07-29 10:32:41 EDT
If I open a type using the setup in attached image, then edit and save, the
decorators disappear.
Comment 19 Tod Creasey CLA 2003-07-29 10:48:10 EDT
I have replicated this in the Java browsing perspective the same way - thanks.
Comment 20 Tod Creasey CLA 2003-08-14 10:48:47 EDT
Closing this report. The lost update problem has been handled by keeping track 
of whether or not the last request was served.
Comment 21 Wassim Melhem CLA 2003-08-14 13:49:53 EDT
Things have been fine for a couple of weeks.
But today as I launch I-20030813, my CVS decorators are not showing up, 
although the CVS checkbox is indeed selected in the preferences.
Comment 22 Wassim Melhem CLA 2003-08-14 13:56:03 EDT
Created attachment 5772 [details]
pde.ui is lacking CVS decorators
Comment 23 Wassim Melhem CLA 2003-08-14 14:07:41 EDT
I just noticed in my stack trace that the team cvs plug-in has failed to start 
due to some circularity errors.  So I'll open a defect against them, as this 
would explain the lack of decorators.  (I can hear Tod's sigh of relief all 
the way from T.O. :-)