mirror of
https://github.com/eclipse-cdt/cdt
synced 2025-07-03 07:05:24 +02:00
Bug 367391 - [Generate Getters and Setters] Going back and deselecting a
getter or setter doesn't affect subsequent code generations.
This commit is contained in:
parent
084ef2d664
commit
0381475f16
1 changed files with 26 additions and 40 deletions
|
@ -11,9 +11,6 @@
|
|||
*******************************************************************************/
|
||||
package org.eclipse.cdt.internal.ui.refactoring.gettersandsetters;
|
||||
|
||||
import java.util.Set;
|
||||
import java.util.SortedSet;
|
||||
|
||||
import org.eclipse.core.runtime.preferences.IEclipsePreferences;
|
||||
import org.eclipse.core.runtime.preferences.IEclipsePreferences.IPreferenceChangeListener;
|
||||
import org.eclipse.core.runtime.preferences.IEclipsePreferences.PreferenceChangeEvent;
|
||||
|
@ -112,17 +109,10 @@ public class GenerateGettersAndSettersInputPage extends UserInputWizardPage impl
|
|||
@Override
|
||||
public void widgetSelected(SelectionEvent e) {
|
||||
Object[] items = context.getElements(null);
|
||||
SortedSet<GetterSetterInsertEditProvider> checkedFunctions = context.selectedFunctions;
|
||||
for (Object treeItem : items) {
|
||||
variableSelectionView.setChecked(treeItem, true);
|
||||
Object[] childs = context.getChildren(treeItem);
|
||||
for(Object currentElement : childs){
|
||||
if (currentElement instanceof GetterSetterInsertEditProvider) {
|
||||
GetterSetterInsertEditProvider editProvider = (GetterSetterInsertEditProvider) currentElement;
|
||||
checkedFunctions.add(editProvider);
|
||||
}
|
||||
}
|
||||
}
|
||||
updateSelectedFunctions();
|
||||
}
|
||||
});
|
||||
|
||||
|
@ -131,11 +121,10 @@ public class GenerateGettersAndSettersInputPage extends UserInputWizardPage impl
|
|||
deselectAll.addSelectionListener(new SelectionAdapter(){
|
||||
@Override
|
||||
public void widgetSelected(SelectionEvent e) {
|
||||
Object[] items = context.getElements(null);
|
||||
for (Object treeItem : items) {
|
||||
for (Object treeItem : context.getElements(null)) {
|
||||
variableSelectionView.setChecked(treeItem, false);
|
||||
}
|
||||
context.selectedFunctions.clear();
|
||||
updateSelectedFunctions();
|
||||
}
|
||||
});
|
||||
|
||||
|
@ -161,23 +150,21 @@ public class GenerateGettersAndSettersInputPage extends UserInputWizardPage impl
|
|||
}
|
||||
|
||||
private void selectMethods(AccessorKind type) {
|
||||
Object[] items = context.getElements(null);
|
||||
Set<GetterSetterInsertEditProvider> checked = context.selectedFunctions;
|
||||
for (Object treeItem : items) {
|
||||
for (Object treeItem : context.getElements(null)) {
|
||||
if (treeItem instanceof FieldWrapper) {
|
||||
FieldWrapper field = (FieldWrapper) treeItem;
|
||||
Object[] funtions = context.getChildren(field);
|
||||
for (Object funct : funtions) {
|
||||
if (funct instanceof GetterSetterInsertEditProvider) {
|
||||
GetterSetterInsertEditProvider getSet = (GetterSetterInsertEditProvider) funct;
|
||||
Object[] accessors = context.getChildren(field);
|
||||
for (Object accessor : accessors) {
|
||||
if (accessor instanceof GetterSetterInsertEditProvider) {
|
||||
GetterSetterInsertEditProvider getSet = (GetterSetterInsertEditProvider) accessor;
|
||||
if (getSet.getType() == type) {
|
||||
checked.add(getSet);
|
||||
variableSelectionView.setChecked(getSet, true);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
updateSelectedFunctions();
|
||||
}
|
||||
|
||||
private void createTree(Composite comp) {
|
||||
|
@ -198,27 +185,26 @@ public class GenerateGettersAndSettersInputPage extends UserInputWizardPage impl
|
|||
}
|
||||
}
|
||||
}
|
||||
Set<GetterSetterInsertEditProvider> checkedFunctions = context.selectedFunctions;
|
||||
for (Object currentElement : variableSelectionView.getCheckedElements()) {
|
||||
if (currentElement instanceof GetterSetterInsertEditProvider) {
|
||||
GetterSetterInsertEditProvider editProvider = (GetterSetterInsertEditProvider) currentElement;
|
||||
checkedFunctions.add(editProvider);
|
||||
}
|
||||
}
|
||||
variableSelectionView.addCheckStateListener(new ICheckStateListener() {
|
||||
|
||||
updateSelectedFunctions();
|
||||
|
||||
variableSelectionView.addCheckStateListener(new ICheckStateListener() {
|
||||
public void checkStateChanged(CheckStateChangedEvent event) {
|
||||
Set<GetterSetterInsertEditProvider> checkedFunctions = context.selectedFunctions;
|
||||
for (Object currentElement : variableSelectionView.getCheckedElements()) {
|
||||
if (currentElement instanceof GetterSetterInsertEditProvider) {
|
||||
GetterSetterInsertEditProvider editProvider = (GetterSetterInsertEditProvider) currentElement;
|
||||
checkedFunctions.add(editProvider);
|
||||
}
|
||||
}
|
||||
updateSelectedFunctions();
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
private void updateSelectedFunctions() {
|
||||
context.selectedFunctions.clear();
|
||||
for (Object currentElement : variableSelectionView.getCheckedElements()) {
|
||||
if (currentElement instanceof GetterSetterInsertEditProvider) {
|
||||
context.selectedFunctions.add((GetterSetterInsertEditProvider) currentElement);
|
||||
}
|
||||
}
|
||||
setPageComplete(!context.selectedFunctions.isEmpty());
|
||||
}
|
||||
|
||||
public void preferenceChange(PreferenceChangeEvent event) {
|
||||
if (variableSelectionView.getTree().isDisposed()) {
|
||||
return;
|
||||
|
|
Loading…
Add table
Reference in a new issue