Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314475 - Advanced button not working again
Summary: Advanced button not working again
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6 RC3   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 314417 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-05-26 10:47 EDT by Szymon Brandys CLA
Modified: 2010-05-27 05:47 EDT (History)
7 users (show)

See Also:
serge: review+
bokowski: review+
pawel.pogorzelski1: review+


Attachments
Fix v01 (1.26 KB, patch)
2010-05-26 10:49 EDT, Szymon Brandys CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Szymon Brandys CLA 2010-05-26 10:47:06 EDT
This is caused by the fix for bug 312888. See also bug 311599, comment 19 and the following comments.
Comment 1 Szymon Brandys CLA 2010-05-26 10:49:48 EDT
Created attachment 170007 [details]
Fix v01

Serge, could you verify that the fix is ok? Thanks.
Comment 2 Serge Beauchamp CLA 2010-05-26 10:59:42 EDT
That looks good, thanks Szymon
Comment 3 Boris Bokowski CLA 2010-05-26 11:30:01 EDT
Do you have reproducible steps for this? The patch looks safe (only adding a null check) but I would like to be able to see the problem and that this fix actually solves the problem.
Comment 4 Szymon Brandys CLA 2010-05-26 11:51:59 EDT
Sure, the steps are as follows:

1) Install Data Tools from the Helios update site to you Eclipse SDK
2) Start Eclipse and create a project (it may be a Java project)
3) Try to create a new SQL file in the project, New > Other > SQL Development > SQL File
4) Click 'Advanced' on the opened New SQL File wizard

You should see NPE logged

5) Resize the dialog and see multiple Advanced sections, see https://bugs.eclipse.org/bugs/attachment.cgi?id=169892
Comment 5 Szymon Brandys CLA 2010-05-26 12:25:18 EDT
Committed to HEAD.
Comment 6 Serge Beauchamp CLA 2010-05-27 05:44:37 EDT
*** Bug 314417 has been marked as a duplicate of this bug. ***