Attachment #77403 for bug #197199

View | Details | Raw Unified | Return to bug 197199
Collapse All | Expand All

Lines 13-18 Link Here

(-)src/org/eclipse/rse/internal/core/model/SystemProfileManager.java (-3 / +8 lines)
13	* Contributors:	13	* Contributors:
14	* David Dykstal (IBM) - created and used RSEPreferencesManager	14	* David Dykstal (IBM) - created and used RSEPreferencesManager
15	* - moved SystemPreferencesManager to a new plugin	15	* - moved SystemPreferencesManager to a new plugin
		16	* Kevin Doyle (IBM) - [197199] Renaming a Profile doesn't cause a save
16	********************************************************************************/	17	********************************************************************************/
17		18
18	package org.eclipse.rse.internal.core.model;	19	package org.eclipse.rse.internal.core.model;
Lines 173-181 Link Here
173	boolean isActive = isSystemProfileActive(profile.getName());	174	boolean isActive = isSystemProfileActive(profile.getName());
174	String oldName = profile.getName();	175	String oldName = profile.getName();
175	profile.setName(newName);	176	profile.setName(newName);
176	if (isActive) RSEPreferencesManager.renameActiveProfile(oldName, newName);	177	if (isActive) {
177	// invalidateCache();	178	RSEPreferencesManager.renameActiveProfile(oldName, newName);
178	// FIXME RSEUIPlugin.getThePersistenceManager().save(this);	179	}
		180	// Commit the profile to reflect the name change
		181	RSECorePlugin.getThePersistenceManager().commitProfile(profile, 5000);
		182	// Delete the profile by the old name, which is done in a separate job.
		183	RSECorePlugin.getThePersistenceManager().deleteProfile(profile.getPersistenceProvider(), oldName);
179	}	184	}
180		185
181	/* (non-Javadoc)	186	/* (non-Javadoc)

Lines 14-19 Link Here

(-)src/org/eclipse/rse/internal/persistence/dom/RSEDOMExporter.java (+3 lines)
14	* Martin Oberhuber (Wind River) - [175262] IHost.getSystemType() should return IRSESystemType	14	* Martin Oberhuber (Wind River) - [175262] IHost.getSystemType() should return IRSESystemType
15	* Martin Oberhuber (Wind River) - [184095] Replace systemTypeName by IRSESystemType	15	* Martin Oberhuber (Wind River) - [184095] Replace systemTypeName by IRSESystemType
16	* Kevin Doyle (IBM) - [163883] Multiple filter strings are disabled	16	* Kevin Doyle (IBM) - [163883] Multiple filter strings are disabled
		17	* Kevin Doyle (IBM) - [197199] Renaming a Profile doesn't cause a save
17	********************************************************************************/	18	********************************************************************************/
18		19
19	package org.eclipse.rse.internal.persistence.dom;	20	package org.eclipse.rse.internal.persistence.dom;
Lines 104-109 Link Here
104	}	105	}
105		106
106	if (clean \|\| profile.isDirty() \|\| dom.isDirty()) {	107	if (clean \|\| profile.isDirty() \|\| dom.isDirty()) {
		108	// Doing a rename requires the dom to update it's name
		109	dom.setName(profile.getName());
107	dom.clearAttributes();	110	dom.clearAttributes();
108	dom.addAttribute(IRSEDOMConstants.ATTRIBUTE_DEFAULT_PRIVATE, getBooleanString(profile.isDefaultPrivate()));	111	dom.addAttribute(IRSEDOMConstants.ATTRIBUTE_DEFAULT_PRIVATE, getBooleanString(profile.isDefaultPrivate()));
109	dom.addAttribute(IRSEDOMConstants.ATTRIBUTE_IS_ACTIVE, getBooleanString(profile.isActive()));	112	dom.addAttribute(IRSEDOMConstants.ATTRIBUTE_IS_ACTIVE, getBooleanString(profile.isActive()));

Return to bug 197199