Attachment #125965 for bug #166595





	private CCombo combo;

	private boolean ignoreNotification;

	private Text text;

	public SingleSelectionAttributeEditor(TaskDataModel manager, TaskAttribute taskAttribute) {
		super(manager, taskAttribute);
	}

	@Override
	public void createControl(Composite parent, FormToolkit toolkit) {
		if (isReadOnly()) {
			text = new Text(parent, SWT.FLAT | SWT.READ_ONLY);
			text.setFont(EditorUtil.TEXT_FONT);
			toolkit.adapt(text, false, false);
			text.setData(FormToolkit.KEY_DRAW_BORDER, Boolean.FALSE);
			refresh();
			if ("".equals(label)) { //$NON-NLS-1$
				// if set to the empty string the label will use 64px on GTK 
				text.setText(" "); //$NON-NLS-1$
			} else {
				text.setText(label);
			}
			setControl(text);
		} else {
			combo = new CCombo(parent, SWT.FLAT | SWT.READ_ONLY);

			combo.setFont(EditorUtil.TEXT_FONT);
			combo.setData(FormToolkit.KEY_DRAW_BORDER, FormToolkit.TEXT_BORDER);

			refresh();
			if (labelByValue != null) {
				values = labelByValue.keySet().toArray(new String[0]);
				for (String value : values) {
					combo.add(labelByValue.get(value));
				}
			}

			select(getValue(), getValueLabel());

			if (values != null) {
				combo.addSelectionListener(new SelectionAdapter() {
					@Override
					public void widgetSelected(SelectionEvent event) {
						if (!ignoreNotification) {
							int index = combo.getSelectionIndex();
							if (index > -1) {
								Assert.isNotNull(values);
								Assert.isLegal(index >= 0 && index <= values.length - 1);
								setValue(values[index]);
							}
						}
					}
				});

	}

	public void setValue(String value) {
		String oldValue = getAttributeMapper().getValue(getTaskAttribute());
		if (!oldValue.equals(value)) {
			getAttributeMapper().setValue(getTaskAttribute(), value);
			attributeChanged();
		}
	}

	void selectDefaultValue() {

		}
	}

	@Override
	public void refresh() {
		try {
			ignoreNotification = true;
			if (text != null) {
				String label = getValueLabel();
				if ("".equals(label)) { //$NON-NLS-1$
					// if set to the empty string the label will use 64px on GTK 
					text.setText(" "); //$NON-NLS-1$
				} else {
					text.setText(label);
				}
			} else {
				combo.removeAll();
				Map<String, String> labelByValue = getAttributeMapper().getOptions(getTaskAttribute());
				if (labelByValue != null) {
					values = labelByValue.keySet().toArray(new String[0]);
					for (String value : values) {
						combo.add(labelByValue.get(value));
					}
				}
				select(getValue(), getValueLabel());
				combo.redraw();
			}
		} finally {
			ignoreNotification = false;
		}
	}
}

Lines 26-46 Link Here

(-)src/org/eclipse/mylyn/internal/tasks/ui/editors/BooleanAttributeEditor.java (-3 / +18 lines)
26	*/	26	*/
27	public class BooleanAttributeEditor extends AbstractAttributeEditor {	27	public class BooleanAttributeEditor extends AbstractAttributeEditor {
28		28
		29	private Button button;
		30
		31	private boolean ignoreNotification;
		32
29	public BooleanAttributeEditor(TaskDataModel manager, TaskAttribute taskAttribute) {	33	public BooleanAttributeEditor(TaskDataModel manager, TaskAttribute taskAttribute) {
30	super(manager, taskAttribute);	34	super(manager, taskAttribute);
31	}	35	}
32		36
33	@Override	37	@Override
34	public void createControl(Composite parent, FormToolkit toolkit) {	38	public void createControl(Composite parent, FormToolkit toolkit) {
35	final Button button = toolkit.createButton(parent, super.getLabel(), SWT.CHECK);	39	button = toolkit.createButton(parent, super.getLabel(), SWT.CHECK);
36	button.setEnabled(!isReadOnly());	40	button.setEnabled(!isReadOnly());
37	button.setSelection(getValue());
38	button.addSelectionListener(new SelectionAdapter() {	41	button.addSelectionListener(new SelectionAdapter() {
39	@Override	42	@Override
40	public void widgetSelected(SelectionEvent e) {	43	public void widgetSelected(SelectionEvent e) {
41	setValue(button.getSelection());	44	if (!ignoreNotification) {
		45	setValue(button.getSelection());
		46	}
42	}	47	}
43	});	48	});
		49	refresh();
44	setControl(button);	50	setControl(button);
45	}	51	}
46		52
Lines 58-61 Link Here
58	attributeChanged();	64	attributeChanged();
59	}	65	}
60		66
		67	@Override
		68	public void refresh() {
		69	try {
		70	ignoreNotification = true;
		71	button.setSelection(getValue());
		72	} finally {
		73	ignoreNotification = false;
		74	}
		75	}
61	}	76	}




		this.readOnly = readOnly;
	}

	/**
	 * Refreshes the state of the widget from the data model. The default implementation throws
	 * <code>UnsupportedOperationException</code>.
	 * 
	 * <p>
	 * Subclasses should overwrite this method.
	 * 
	 * @since 3.1
	 * @throws UnsupportedOperationException
	 *             if this method is not supported by the editor
	 */
	public void refresh() {
		throw new UnsupportedOperationException();
	}

}

Lines 582-587 Link Here

(-)src/org/eclipse/mylyn/internal/bugzilla/core/RepositoryConfiguration.java (+7 lines)
582	if (bugzillaVersion == null) {	582	if (bugzillaVersion == null) {
583	bugzillaVersion = BugzillaVersion.MIN_VERSION;	583	bugzillaVersion = BugzillaVersion.MIN_VERSION;
584	}	584	}
		585
		586	if (bugzillaVersion.compareTo(BugzillaVersion.BUGZILLA_3_0) < 0) {
		587	// Product change is only supported for Versions >= 3.0 without verify html page
		588	TaskAttribute productAttribute = bugReport.getRoot().getMappedAttribute(BugzillaAttribute.PRODUCT.getKey());
		589	productAttribute.getMetaData().setReadOnly(true);
		590	}
		591
585	if (status == BUGZILLA_REPORT_STATUS.NEW \|\| status == BUGZILLA_REPORT_STATUS.ASSIGNED	592	if (status == BUGZILLA_REPORT_STATUS.NEW \|\| status == BUGZILLA_REPORT_STATUS.ASSIGNED
586	\|\| status == BUGZILLA_REPORT_STATUS.REOPENED \|\| status == BUGZILLA_REPORT_STATUS.UNCONFIRMED) {	593	\|\| status == BUGZILLA_REPORT_STATUS.REOPENED \|\| status == BUGZILLA_REPORT_STATUS.UNCONFIRMED) {
587	if (bugzillaVersion.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_3_0) <= 0) {	594	if (bugzillaVersion.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_3_0) <= 0) {

Lines 64-69 Link Here

(-)src/org/eclipse/mylyn/internal/bugzilla/core/BugzillaAttribute.java (-1 / +3 lines)
64		64
65	COMPONENT(Messages.BugzillaAttribute_Component, "component", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$	65	COMPONENT(Messages.BugzillaAttribute_Component, "component", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$
66		66
		67	CONFIRM_PRODUCT_CHANGE("confirm_product_change", "confirm_product_change", TaskAttribute.TYPE_BOOLEAN, true, false), //$NON-NLS-1$ //$NON-NLS-2$
		68
67	CREATION_TS(Messages.BugzillaAttribute_Opened, "creation_ts", TaskAttribute.TYPE_DATE, true, false), //$NON-NLS-1$	69	CREATION_TS(Messages.BugzillaAttribute_Opened, "creation_ts", TaskAttribute.TYPE_DATE, true, false), //$NON-NLS-1$
68		70
69	CTYPE(Messages.BugzillaAttribute_Content_Type, "ctype", TaskAttribute.TYPE_SHORT_TEXT, false, false), //$NON-NLS-1$	71	CTYPE(Messages.BugzillaAttribute_Content_Type, "ctype", TaskAttribute.TYPE_SHORT_TEXT, false, false), //$NON-NLS-1$
Lines 109-115 Link Here
109		111
110	PRIORITY(Messages.BugzillaAttribute_Priority, "priority", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$	112	PRIORITY(Messages.BugzillaAttribute_Priority, "priority", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$
111		113
112	PRODUCT(Messages.BugzillaAttribute_Product, "product", TaskAttribute.TYPE_SHORT_TEXT, false, true), //$NON-NLS-1$	114	PRODUCT(Messages.BugzillaAttribute_Product, "product", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$
113		115
114	REP_PLATFORM(Messages.BugzillaAttribute_Platform, "rep_platform", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$	116	REP_PLATFORM(Messages.BugzillaAttribute_Platform, "rep_platform", TaskAttribute.TYPE_SINGLE_SELECT, false, false), //$NON-NLS-1$
115		117

Lines 11-27 Link Here

(-)src/org/eclipse/mylyn/internal/bugzilla/ui/editor/BugzillaTaskEditorPage.java (-2 / +186 lines)
11		11
12	package org.eclipse.mylyn.internal.bugzilla.ui.editor;	12	package org.eclipse.mylyn.internal.bugzilla.ui.editor;
13		13
		14	import java.util.Collections;
		15	import java.util.HashMap;
		16	import java.util.List;
		17	import java.util.Map;
14	import java.util.Set;	18	import java.util.Set;
15		19
16	import org.eclipse.core.runtime.CoreException;	20	import org.eclipse.core.runtime.CoreException;
		21	import org.eclipse.core.runtime.IStatus;
		22	import org.eclipse.core.runtime.NullProgressMonitor;
17	import org.eclipse.jface.dialogs.IMessageProvider;	23	import org.eclipse.jface.dialogs.IMessageProvider;
		24	import org.eclipse.mylyn.commons.core.StatusHandler;
18	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaAttribute;	25	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaAttribute;
19	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaCorePlugin;	26	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaCorePlugin;
20	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaCustomField;	27	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaCustomField;
		28	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaTaskDataHandler;
		29	import org.eclipse.mylyn.internal.bugzilla.core.BugzillaVersion;
21	import org.eclipse.mylyn.internal.bugzilla.core.IBugzillaConstants;	30	import org.eclipse.mylyn.internal.bugzilla.core.IBugzillaConstants;
		31	import org.eclipse.mylyn.internal.bugzilla.core.RepositoryConfiguration;
		32	import org.eclipse.mylyn.internal.bugzilla.ui.BugzillaUiPlugin;
		33	import org.eclipse.mylyn.tasks.core.RepositoryStatus;
22	import org.eclipse.mylyn.tasks.core.data.TaskAttribute;	34	import org.eclipse.mylyn.tasks.core.data.TaskAttribute;
23	import org.eclipse.mylyn.tasks.core.data.TaskAttributeMetaData;	35	import org.eclipse.mylyn.tasks.core.data.TaskAttributeMetaData;
24	import org.eclipse.mylyn.tasks.core.data.TaskData;	36	import org.eclipse.mylyn.tasks.core.data.TaskData;
		37	import org.eclipse.mylyn.tasks.core.data.TaskDataModel;
		38	import org.eclipse.mylyn.tasks.core.data.TaskDataModelEvent;
		39	import org.eclipse.mylyn.tasks.core.data.TaskDataModelListener;
25	import org.eclipse.mylyn.tasks.ui.TasksUi;	40	import org.eclipse.mylyn.tasks.ui.TasksUi;
26	import org.eclipse.mylyn.tasks.ui.editors.AbstractAttributeEditor;	41	import org.eclipse.mylyn.tasks.ui.editors.AbstractAttributeEditor;
27	import org.eclipse.mylyn.tasks.ui.editors.AbstractTaskEditorPage;	42	import org.eclipse.mylyn.tasks.ui.editors.AbstractTaskEditorPage;
Lines 29-34 Link Here
29	import org.eclipse.mylyn.tasks.ui.editors.AttributeEditorFactory;	44	import org.eclipse.mylyn.tasks.ui.editors.AttributeEditorFactory;
30	import org.eclipse.mylyn.tasks.ui.editors.LayoutHint;	45	import org.eclipse.mylyn.tasks.ui.editors.LayoutHint;
31	import org.eclipse.mylyn.tasks.ui.editors.TaskEditor;	46	import org.eclipse.mylyn.tasks.ui.editors.TaskEditor;
		47	import org.eclipse.mylyn.tasks.ui.editors.TaskEditorInput;
32	import org.eclipse.mylyn.tasks.ui.editors.TaskEditorPartDescriptor;	48	import org.eclipse.mylyn.tasks.ui.editors.TaskEditorPartDescriptor;
33		49
34	/**	50	/**
Lines 41-48 Link Here
41		57
42	public static final String ID_PART_BUGZILLA_FLAGS = "org.eclipse.mylyn.bugzilla.ui.editors.part.flags"; //$NON-NLS-1$	58	public static final String ID_PART_BUGZILLA_FLAGS = "org.eclipse.mylyn.bugzilla.ui.editors.part.flags"; //$NON-NLS-1$
43		59
		60	private final Map<TaskAttribute, AbstractAttributeEditor> attributeEditorMap;
		61
		62	private TaskDataModelListener productListener;
		63
44	public BugzillaTaskEditorPage(TaskEditor editor) {	64	public BugzillaTaskEditorPage(TaskEditor editor) {
45	super(editor, BugzillaCorePlugin.CONNECTOR_KIND);	65	this(editor, BugzillaCorePlugin.CONNECTOR_KIND);
46	}	66	}
47		67
48	/**	68	/**
Lines 53-58 Link Here
53	*/	73	*/
54	public BugzillaTaskEditorPage(TaskEditor editor, String connectorKind) {	74	public BugzillaTaskEditorPage(TaskEditor editor, String connectorKind) {
55	super(editor, connectorKind);	75	super(editor, connectorKind);
		76	this.attributeEditorMap = new HashMap<TaskAttribute, AbstractAttributeEditor>();
56	}	77	}
57		78
58	@Override	79	@Override
Lines 138-144 Link Here
138	}	159	}
139	});	160	});
140	}	161	}
141		162	BugzillaTaskEditorPage.this.addToAttributeEditorMap(taskAttribute, editor);
142	return editor;	163	return editor;
143	}	164	}
144	};	165	};
Lines 193-196 Link Here
193	super.doSubmit();	214	super.doSubmit();
194	}	215	}
195		216
		217	@Override
		218	protected void createParts() {
		219	attributeEditorMap.clear();
		220	super.createParts();
		221	}
		222
		223	@Override
		224	protected TaskDataModel createModel(TaskEditorInput input) throws CoreException {
		225	TaskDataModel model = super.createModel(input);
		226	BugzillaVersion bugzillaVersion = null;
		227	RepositoryConfiguration repositoryConfiguration = BugzillaCorePlugin.getRepositoryConfiguration(
		228	input.getTaskRepository(), false, new NullProgressMonitor());
		229	if (repositoryConfiguration != null) {
		230	bugzillaVersion = repositoryConfiguration.getInstallVersion();
		231	} else {
		232	bugzillaVersion = BugzillaVersion.MIN_VERSION;
		233	}
		234	if (bugzillaVersion.compareTo(BugzillaVersion.BUGZILLA_3_0) >= 0) {
		235	productListener = new ProductSelectionListener();
		236	model.addModelListener(productListener);
		237	}
		238	return model;
		239	}
		240
		241	/**
		242	* @since 3.1
		243	*/
		244	private void addToAttributeEditorMap(TaskAttribute attribute, AbstractAttributeEditor editor) {
		245	if (attributeEditorMap.containsKey(attribute)) {
		246	attributeEditorMap.remove(attribute);
		247	}
		248	attributeEditorMap.put(attribute, editor);
		249	}
		250
		251	/**
		252	* @since 3.1
		253	*/
		254	private AbstractAttributeEditor getEditorForAttribute(TaskAttribute attribute) {
		255	return attributeEditorMap.get(attribute);
		256	}
		257
		258	// /**
		259	// * @since 3.1
		260	// */
		261	// public void refresh() {
		262	// try {
		263	// showEditorBusy(true);
		264	// for (AbstractAttributeEditor abstractAttributeEditor : attributeEditorMap.values()) {
		265	// try {
		266	// abstractAttributeEditor.refresh();
		267	// } catch (UnsupportedOperationException e) {
		268	// // ignore
		269	// }
		270	// }
		271	// } finally {
		272	// showEditorBusy(false);
		273	// }
		274	// }
		275
		276	public void refresh(TaskAttribute attributeComponent) {
		277	AbstractAttributeEditor editor = getEditorForAttribute(attributeComponent);
		278	if (editor != null) {
		279	try {
		280	editor.refresh();
281	} catch (UnsupportedOperationException e) {
282	// ignore
283	}
284	}
285	}
286
287	private class ProductSelectionListener extends TaskDataModelListener {
288	@Override
289	public void attributeChanged(TaskDataModelEvent event) {
290	TaskAttribute taskAttribute = event.getTaskAttribute();
291	if (taskAttribute != null) {
292	if (taskAttribute.getId().equals(BugzillaAttribute.PRODUCT.getKey())) {
293	RepositoryConfiguration repositoryConfiguration = null;
294	try {
295	repositoryConfiguration = BugzillaCorePlugin.getRepositoryConfiguration(
296	getModel().getTaskRepository(), false, new NullProgressMonitor());
297	} catch (CoreException e) {
298	StatusHandler.log(new RepositoryStatus(getTaskRepository(), IStatus.ERROR,
299	BugzillaUiPlugin.ID_PLUGIN, 0, "Failed to obtain repository configuration", e)); //$NON-NLS-1$
300	getTaskEditor().setMessage("Problem occured when updating attributes", IMessageProvider.ERROR); //$NON-NLS-1$
301	return;
302	}
303
304	TaskAttribute attributeComponent = taskAttribute.getTaskData().getRoot().getMappedAttribute(
305	BugzillaAttribute.COMPONENT.getKey());
306	if (attributeComponent != null) {
307	List<String> optionValues = repositoryConfiguration.getComponents(taskAttribute.getValue());
308	Collections.sort(optionValues);
309	attributeComponent.clearOptions();
310	for (String option : optionValues) {
311	attributeComponent.putOption(option, option);
312	}
313	if (optionValues.size() == 1) {
314	attributeComponent.setValue(optionValues.get(0));
315	} else {
316	attributeComponent.setValue(""); //$NON-NLS-1$
317	}
318	refresh(attributeComponent);
319	}
320
321	TaskAttribute attributeTargetMilestone = taskAttribute.getTaskData().getRoot().getMappedAttribute(
322	BugzillaAttribute.TARGET_MILESTONE.getKey());
323	if (attributeTargetMilestone != null) {
324	List<String> optionValues = repositoryConfiguration.getTargetMilestones(taskAttribute.getValue());
325	Collections.sort(optionValues);
326	attributeTargetMilestone.clearOptions();
327	for (String option : optionValues) {
328	attributeTargetMilestone.putOption(option, option);
329	}
330	if (optionValues.size() == 1) {
331	attributeTargetMilestone.setValue(optionValues.get(0));
332	} else {
333	attributeTargetMilestone.setValue("---"); //$NON-NLS-1$
334	}
335	refresh(attributeTargetMilestone);
336	}
337
338	TaskAttribute attributeVersion = taskAttribute.getTaskData().getRoot().getMappedAttribute(
339	BugzillaAttribute.VERSION.getKey());
340	if (attributeVersion != null) {
341	List<String> optionValues = repositoryConfiguration.getVersions(taskAttribute.getValue());
342	Collections.sort(optionValues);
343	attributeVersion.clearOptions();
344	for (String option : optionValues) {
345	attributeVersion.putOption(option, option);
346	}
347	if (optionValues.size() == 1) {
348	attributeVersion.setValue(optionValues.get(0));
349	} else {
350	attributeVersion.setValue("unspecified"); //$NON-NLS-1$
351	}
352	refresh(attributeVersion);
353	}
354
355	TaskAttribute attributeDefaultAssignee = taskAttribute.getTaskData().getRoot().getMappedAttribute(
356	BugzillaAttribute.SET_DEFAULT_ASSIGNEE.getKey());
357	if (attributeDefaultAssignee != null) {
358	attributeDefaultAssignee.setValue("1"); //$NON-NLS-1$
359	refresh(attributeDefaultAssignee);
360	}
361
362	/*
363	* add confirm_product_change to avoid verification page on submit
364	*/
365	TaskAttribute attributeConfirmeProductChange = taskAttribute.getTaskData()
366	.getRoot()
367	.getMappedAttribute(BugzillaAttribute.CONFIRM_PRODUCT_CHANGE.getKey());
368	if (attributeConfirmeProductChange == null) {
369	attributeConfirmeProductChange = BugzillaTaskDataHandler.createAttribute(
370	taskAttribute.getTaskData().getRoot(), BugzillaAttribute.CONFIRM_PRODUCT_CHANGE);
371	}
372	if (attributeConfirmeProductChange != null) {
373	attributeConfirmeProductChange.setValue("1"); //$NON-NLS-1$
374	}
375	}
376	}
377	}
378	}
379
196	}	380	}

Return to bug 166595

Lines 34-39 Link Here

(-)src/org/eclipse/mylyn/internal/tasks/ui/editors/SingleSelectionAttributeEditor.java (-24 / +47 lines)
34		34
35	private CCombo combo;	35	private CCombo combo;
36		36
		37	private boolean ignoreNotification;
		38
		39	private Text text;
		40
37	public SingleSelectionAttributeEditor(TaskDataModel manager, TaskAttribute taskAttribute) {	41	public SingleSelectionAttributeEditor(TaskDataModel manager, TaskAttribute taskAttribute) {
38	super(manager, taskAttribute);	42	super(manager, taskAttribute);
39	}	43	}
Lines 41-57 Link Here
41	@Override	45	@Override
42	public void createControl(Composite parent, FormToolkit toolkit) {	46	public void createControl(Composite parent, FormToolkit toolkit) {
43	if (isReadOnly()) {	47	if (isReadOnly()) {
44	Text text = new Text(parent, SWT.FLAT \| SWT.READ_ONLY);	48	text = new Text(parent, SWT.FLAT \| SWT.READ_ONLY);
45	text.setFont(EditorUtil.TEXT_FONT);	49	text.setFont(EditorUtil.TEXT_FONT);
46	toolkit.adapt(text, false, false);	50	toolkit.adapt(text, false, false);
47	text.setData(FormToolkit.KEY_DRAW_BORDER, Boolean.FALSE);	51	text.setData(FormToolkit.KEY_DRAW_BORDER, Boolean.FALSE);
48	String label = getValueLabel();	52	refresh();
49	if ("".equals(label)) { //$NON-NLS-1$
50	// if set to the empty string the label will use 64px on GTK
51	text.setText(" "); //$NON-NLS-1$
52	} else {
53	text.setText(label);
54	}
55	setControl(text);	53	setControl(text);
56	} else {	54	} else {
57	combo = new CCombo(parent, SWT.FLAT \| SWT.READ_ONLY);	55	combo = new CCombo(parent, SWT.FLAT \| SWT.READ_ONLY);
Lines 59-83 Link Here
59	combo.setFont(EditorUtil.TEXT_FONT);	57	combo.setFont(EditorUtil.TEXT_FONT);
60	combo.setData(FormToolkit.KEY_DRAW_BORDER, FormToolkit.TEXT_BORDER);	58	combo.setData(FormToolkit.KEY_DRAW_BORDER, FormToolkit.TEXT_BORDER);
61		59
62	Map<String, String> labelByValue = getAttributeMapper().getOptions(getTaskAttribute());	60	refresh();
63	if (labelByValue != null) {
64	values = labelByValue.keySet().toArray(new String[0]);
65	for (String value : values) {
66	combo.add(labelByValue.get(value));
67	}
68	}
69
70	select(getValue(), getValueLabel());
71		61
72	if (values != null) {	62	if (values != null) {
73	combo.addSelectionListener(new SelectionAdapter() {	63	combo.addSelectionListener(new SelectionAdapter() {
74	@Override	64	@Override
75	public void widgetSelected(SelectionEvent event) {	65	public void widgetSelected(SelectionEvent event) {
76	int index = combo.getSelectionIndex();	66	if (!ignoreNotification) {
77	if (index > -1) {	67	int index = combo.getSelectionIndex();
78	Assert.isNotNull(values);	68	if (index > -1) {
79	Assert.isLegal(index >= 0 && index <= values.length - 1);	69	Assert.isNotNull(values);
80	setValue(values[index]);	70	Assert.isLegal(index >= 0 && index <= values.length - 1);
		71	setValue(values[index]);
		72	}
81	}	73	}
82	}	74	}
83	});	75	});
Lines 109-116 Link Here
109	}	101	}
110		102
111	public void setValue(String value) {	103	public void setValue(String value) {
112	getAttributeMapper().setValue(getTaskAttribute(), value);	104	String oldValue = getAttributeMapper().getValue(getTaskAttribute());
113	attributeChanged();	105	if (!oldValue.equals(value)) {
		106	getAttributeMapper().setValue(getTaskAttribute(), value);
		107	attributeChanged();
		108	}
114	}	109	}
115		110
116	void selectDefaultValue() {	111	void selectDefaultValue() {
Lines 120-123 Link Here
120	}	115	}
121	}	116	}
122		117
		118	@Override
		119	public void refresh() {
		120	try {
		121	ignoreNotification = true;
		122	if (text != null) {
		123	String label = getValueLabel();
		124	if ("".equals(label)) { //$NON-NLS-1$
		125	// if set to the empty string the label will use 64px on GTK
		126	text.setText(" "); //$NON-NLS-1$
		127	} else {
		128	text.setText(label);
		129	}
		130	} else {
		131	combo.removeAll();
		132	Map<String, String> labelByValue = getAttributeMapper().getOptions(getTaskAttribute());
		133	if (labelByValue != null) {
		134	values = labelByValue.keySet().toArray(new String[0]);
		135	for (String value : values) {
		136	combo.add(labelByValue.get(value));
		137	}
		138	}
		139	select(getValue(), getValueLabel());
		140	combo.redraw();
		141	}
		142	} finally {
		143	ignoreNotification = false;
		144	}
		145	}
123	}	146	}

Lines 213-216 Link Here

(-)src/org/eclipse/mylyn/tasks/ui/editors/AbstractAttributeEditor.java (+15 lines)
213	this.readOnly = readOnly;	213	this.readOnly = readOnly;
214	}	214	}
215		215
		216	/**
		217	* Refreshes the state of the widget from the data model. The default implementation throws
		218	* <code>UnsupportedOperationException</code>.
		219	*
		220	* <p>
		221	* Subclasses should overwrite this method.
		222	*
		223	* @since 3.1
		224	* @throws UnsupportedOperationException
		225	* if this method is not supported by the editor
		226	*/
		227	public void refresh() {
		228	throw new UnsupportedOperationException();
		229	}
		230
216	}	231	}