diff -r 6cadd6867c17 -r 67f2ed48ad91 idlefw/plugins/mcsplugin/publisher/src/mcsplugindata.cpp
--- a/idlefw/plugins/mcsplugin/publisher/src/mcsplugindata.cpp	Wed Sep 15 12:32:36 2010 +0300
+++ b/idlefw/plugins/mcsplugin/publisher/src/mcsplugindata.cpp	Wed Oct 13 14:53:46 2010 +0300
@@ -111,9 +111,9 @@
 // 
 // ---------------------------------------------------------------------------
 //
-const TDesC* CMCSData::Name()
+TDesC& CMCSData::Name()
     {
-    return iName;
+    return *iName;
     }
     
 // ---------------------------------------------------------------------------
@@ -131,9 +131,9 @@
 // 
 // ---------------------------------------------------------------------------
 //
-const TDesC* CMCSData::Value()
+TDesC& CMCSData::Value()
     {
-    return iValue;
+    return *iValue;
     }
     
 // ---------------------------------------------------------------------------
@@ -265,20 +265,20 @@
             
             // id of all bookmarks is zero so name has to be check
             // in case of bookmark has changed
-            if( data->MenuItem().Id() != id ||
-                ( id == 0 && data->Name() != NULL && iData[i]->Name() != NULL && 
-                  data->Name()->CompareF( *iData[i]->Name() ) != 0 ) )
+            if ( id >= 0 && ( data->MenuItem().Id() != id ||
+                    ( id == 0 && data->Name().CompareF(
+                            iData[ i ]->Name() ) != 0 ) ) )
                 {
                 data->SetDirty( ETrue );
                 CMCSData* oldData = iData[i];
                 iData.Remove( i );
                 delete oldData;
+                
                 iData.InsertL( data, i );
-                CleanupStack::Pop( data );            
+                CleanupStack::Pop( data );
                 }
             else
                 {
-                // keep old data
                 CleanupStack::PopAndDestroy( data );
                 }
             }
@@ -572,8 +572,8 @@
                 }
             }
         }
-    // EFalse tells that modified settings are not stored to plugin reference
-    iPluginSettings->SetSettingsL( iInstanceUid, settingItems, EFalse );
+    // ETrue tells that modified settings are stored also to plugin reference
+    iPluginSettings->SetSettingsL( iInstanceUid, settingItems, ETrue );
     CleanupStack::PopAndDestroy(); // settingItems
     }