Bug 202378 - Error dialogue pops up twice when delete the chart name [13]
Summary: Error dialogue pops up twice when delete the chart name [13]
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: BIRT (show other bugs)
Version: 2.2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.3.0 M1   Edit
Assignee: Chen Chao CLA
QA Contact:
URL:
Whiteboard: Non-Auto
Keywords:
Depends on: 203427
Blocks:
  Show dependency tree
 
Reported: 2007-09-06 01:55 EDT by Xiaodan Wang CLA
Modified: 2009-07-09 21:29 EDT (History)
4 users (show)

See Also:

Attachments
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 Xiaodan Wang CLA 2007-09-06 01:55:42 EDT 
Description:
Error dialogue pops up twice when delete the chart name.

Build number: 
2.2.1.v20070906-0630

Steps to reproduce:
1. New a chart.
2. In Property Editor->Advanced, delete chart name then click on other white 
   space.

Expected result:
Error dialogue pops up only once.

Actual result:
Error dialogue pops up twice.

Error log:
A BIRT exception occurred.
  Plug-in Provider:Eclipse.org
  Plug-in Name:BIRT Model
  Plug-in ID:org.eclipse.birt.report.model
  Version:2.2.1.r22x_v20070831
  Error Code:Error.NameException.NAME_REQUIRED
  Error Message:The name of Chart("NewChart") is required.
Comment 1 xingjie he CLA 2007-09-06 05:42:49 EDT 
Please check that there are two events to call 'setProperty' method of DesignElementHandle, one is from CellEditor and another is from MouseDown. So there are two message dialogs. Reassign GUI. Thx!
Comment 2 Chen Chao CLA 2007-10-15 04:51:56 EDT 
Fixed it.
Comment 3 Xiaodan Wang CLA 2007-10-26 01:44:48 EDT 
Exception is still thrown out twice in build (2.3.0.v20071026-1107), reopen.
Comment 4 Chen Chao CLA 2007-10-26 03:33:12 EDT 
Fixed it.

Maybe the TextCellEditor was Changed in the new version Eclipse. Last time I fixed it under eclipse 3.3, but it's still incorrect under eclipse 3.4. And this time, the error cause is different from the last time.
Comment 5 Xiaodan Wang CLA 2007-10-29 01:17:12 EDT 
Verified in build (2.3.0.v20071029-0800), closed.
Comment 6 Chen Chao CLA 2007-10-29 04:54:36 EDT 
Rollback the code because of the bug 207127.

This bug is depended on eclipse bug 203427, please wait for fixing it.