Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 334336 - [diagram] Image with ID 'bookImage' is already registered
Summary: [diagram] Image with ID 'bookImage' is already registered
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sapphire (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Shenxue Zhou CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-13 20:42 EST by Konstantin Komissarchik CLA
Modified: 2021-11-19 09:21 EST (History)
0 users

See Also:

Attachments
Patch v1 (1.06 KB, patch)
2011-01-31 18:58 EST, Shenxue Zhou CLA 		konstantin: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin Komissarchik CLA 2011-01-13 20:42:28 EST 
Opening and closing library sample editor results in multiple instances of log messages like the following:

Image with ID 'bookImage' is already registered
Comment 1 Konstantin Komissarchik CLA 2011-01-28 20:49:15 EST 
This might not be reproducible any more with the demise of the library sample, but the underlying issue should still exist...
Comment 2 Shenxue Zhou CLA 2011-01-31 10:40:15 EST 
The root cause is this Graphiti bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=334402
Comment 3 Konstantin Komissarchik CLA 2011-01-31 12:25:47 EST 
Is there a way to check if the image is already registered before registering? Your code should work whether or not Graphiti chooses to cache images between editor sessions. If there isn't an API to check this ahead of time or a way to register an image without causing problems to be logged, I would recommend opening a bug on that instead of dispose-on-close problem.
Comment 4 Shenxue Zhou CLA 2011-01-31 18:58:18 EST 
Created attachment 188018 [details]
Patch v1

Before registering images we first check whether they have already been cached.
Comment 5 Konstantin Komissarchik CLA 2011-01-31 21:24:47 EST 
Patch released.
Comment 6 Konstantin Komissarchik CLA 2011-06-09 16:33:49 EDT 
This problem has not been seen since the fix, so closing as verified.