This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 229109 - Under IE, Problems with Register/DeRegister Configuration items
Summary: Under IE, Problems with Register/DeRegister Configuration items
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Cosmos (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Leonard Richardson CLA
QA Contact:
URL:
Whiteboard: stability
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-28 12:15 EDT by Srinivas Reddy CLA
Modified: 2012-01-03 13:46 EST (History)
3 users (show)

See Also:


Attachments
Patch for 229109 (3.35 KB, patch)
2008-05-27 13:21 EDT, Sheldon Lee-Loy CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Srinivas Reddy CLA 2008-04-28 12:15:16 EDT
Build ID: COSMOS-1.0.0-200804270312

Steps To Reproduce:
1.Open http://localhost:8080/COSMOSUI under IE
2.select "Sample Repository with Registration",right click and select "Register Configuration items..."
3.Under Registration window, select "Sample Repository with Query"
4.After registration is successful, look into IE status bar for error message
5.Try to re-do registration, it won't show registration window

Result:
- From IE status bar it throws an error "this.domNode.style' is null or not an object"

Also, on successful registration/deregistration, closing the window using 'X'-close button instead of OK button will throw some error message. This is observed on FireFox browser as well.


More information:
Comment 1 Sheldon Lee-Loy CLA 2008-04-28 12:33:44 EDT
This is also the case for the deregister scenario.

The workaround is the user will have to refresh the page.

Pushing to i11 since there is a workaround.

Leonard can you investigate this problem.
Comment 2 Sheldon Lee-Loy CLA 2008-05-27 13:21:53 EDT
Created attachment 102197 [details]
Patch for  229109

Hi Leonard,

I look at this problem and it seems that the dialog is not properly destroyed in the destroy method.  For some reason I need to call the onCancel method on the dialog box and then in the create method I have to check if the dialog object is there.  If it is there then I destroy the dialog.  

Can you try out the patch and see if it fixes the problem.   If it does you can check in the patch and mark this bug as fixed.
Comment 3 Sheldon Lee-Loy CLA 2008-05-29 12:50:54 EDT
Hi Leonard,

I checked in my patch with another fix by accident.  But I believe the patch fixes the problem.  You can take the next development build and verify if it works for you.
Comment 4 Leonard Richardson CLA 2008-06-04 05:38:38 EDT
The steps to reproduce are no longer relevant as the dialog has changed and has cleared up the problems here.

But the problem of closing the dialog with the X button still remains.

I tested the integration build Build ID: COSMOS-1.0.0-200806031210 and it has not fix this problem but the behaviour is slightly altered. Although the error message is no longer thrown, if you then try and open the dialog again it fails to show.

I will investigate more.
Comment 5 Leonard Richardson CLA 2008-06-05 12:03:36 EDT
The X close button was not connected to the onclick event.

This has now been fixed and the dialog now closes correctly with either the cancel or X button.
Comment 6 Srinivas Reddy CLA 2008-06-25 00:27:35 EDT
Verified under build - COSMOS-1.0.0-200806231121, registration/deregistration,close-X button works fine.
Comment 7 Sheldon Lee-Loy CLA 2008-07-28 13:10:48 EDT
Marked as closed based on previous comments.