Bug 293029 - Properties dialog doesn't remember selected configuration when changing page
Summary: Properties dialog doesn't remember selected configuration when changing page
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 6.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 6.0.2   Edit
Assignee: James Blackburn CLA
QA Contact: Doug Schaefer CLA
URL:
Whiteboard:
Keywords: contributed
Depends on: 248341
Blocks:
  Show dependency tree
 
Reported: 2009-10-22 08:42 EDT by James Blackburn CLA
Modified: 2009-10-22 09:05 EDT (History)
3 users (show)

See Also:

Attachments
Patch 1 (869 bytes, patch)
2009-10-22 08:53 EDT, James Blackburn CLA 		jamesblackburn+eclipse: 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 James Blackburn CLA 2009-10-22 08:42:50 EDT 
+++ This bug was initially created as a clone of Bug #248341 +++

The patch on bug 248431, taking AbstractPage from 1.64 -> 1.65 causes the Configuration drop down to forget the current selected configuration when changing page.

1) In Project Properties select a configuration other than active
2) change page
3) configuration reset to active
Comment 1 James Blackburn CLA 2009-10-22 08:53:42 EDT 
Created attachment 150233 [details]
Patch 1

Previously populateConfiguration used to use the index of the last selected configuration. It would only select the active configuration if the last selected index was -1.

Since the patch was applied #populateConfigurations cfgIndex is set to -1. The fix seems to be trivial - preset the index to the index of the last selected config.
Comment 2 James Blackburn CLA 2009-10-22 08:57:22 EDT 
Committed to HEAD.
Comment 3 James Blackburn CLA 2009-10-22 09:05:58 EDT 
Committed to 60x as well.