Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 324897 - [Wizards] WizardDialog behavior in 3.6 is not backward compatible to 3.2
Summary: [Wizards] WizardDialog behavior in 3.6 is not backward compatible to 3.2
Status: RESOLVED INVALID
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact: Prakash Rangaraj CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-09 15:27 EDT by Kihup Boo CLA
Modified: 2010-10-05 02:41 EDT (History)
1 user (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 Kihup Boo CLA 2010-09-09 15:27:37 EDT 
Build Identifier: I20100608-0911

When I call WizardDialog.showPage() method, I have a null argument exception. The same code works without any problems with WizardDialog in Eclipse 3.2.

!ENTRY org.eclipse.osgi 4 0 2010-09-09 15:14:03.971
!MESSAGE Application error
!STACK 1
org.eclipse.core.runtime.AssertionFailedException: null argument:
	at org.eclipse.core.runtime.Assert.isNotNull(Assert.java:85)
	at org.eclipse.core.runtime.Assert.isNotNull(Assert.java:73)
	at org.eclipse.jface.dialogs.PageChangingEvent.<init>(PageChangingEvent.java:57)
	at org.eclipse.jface.wizard.WizardDialog.doPageChanging(WizardDialog.java:906)
	at org.eclipse.jface.wizard.WizardDialog.showPage(WizardDialog.java:1197)
...

Reproducible: Always

Steps to Reproduce:
1. Create a WizardPage and a Wizard.
2. Add the page to the wizard.
3. Override createPageControls(Composite) method of the Wizard, and call wizard.getContainer().showPage(composite).
Comment 1 Kihup Boo CLA 2010-09-09 17:06:06 EDT 
Some correction to my report.

Actual call needs to be corrected as:
IWizardContainer.showPage(IWizardPage) and this call ends up in WizardDialog.showPage().
Comment 2 Remy Suen CLA 2010-09-09 17:37:25 EDT 
You can probably temporarily workaround the problem by overriding getCurrentPage() to return a non-null value. I suspect this has been broken since 3.3.

Could you please provide some simple code to reproduce the problem?
Comment 3 Kihup Boo CLA 2010-09-10 09:50:34 EDT 
Hi,

Thanks for the quick suggested workaround. It works.
I will try to write a test code that reproduces this problem and attach it to this bug when I have a chance.

Thanks again.
Comment 4 Prakash Rangaraj CLA 2010-10-05 02:41:42 EDT 
(In reply to comment #3)
> Thanks for the quick suggested workaround. It works.
> I will try to write a test code that reproduces this problem and attach it to
> this bug when I have a chance.

   Please reopen when you have the sample code to reproduce the issue