diff -r 5b238bc8ffb6 -r 01504893d9cb photosgallery/slideshow/settingsdialog/src/shwslideshowsettingslist.cpp --- a/photosgallery/slideshow/settingsdialog/src/shwslideshowsettingslist.cpp Wed Sep 15 12:13:06 2010 +0300 +++ b/photosgallery/slideshow/settingsdialog/src/shwslideshowsettingslist.cpp Wed Oct 13 14:32:09 2010 +0300 @@ -1203,26 +1203,25 @@ //----------------------------------------------------------------------------- void CShwSlideShowSettingsList::SetPlayDirectionL(TBool aNewPlayDirection) { - TRACER("CShwSlideShowSettingsList::SetPlayDirectionL"); - GLX_LOG_INFO1("CShwSlideShowSettingsList::SetPlayDirectionL(%d)", - aNewPlayDirection); - - // retrieve play direction setting - (*SettingItemArray())[EPlayDirectionItem]->EditItemL(EFalse); - - // updates play direction - iPlayDirection = aNewPlayDirection; - - // internalise new state to settings item - (*(SettingItemArray()))[EPlayDirectionItem]->StoreL(); - - // persist direction - iShwSettings.SavePlayOrderL(iPlayDirection); - - // redraw music on/off control - ListBox()->DrawItem(EPlayDirectionItem); - } - + TRACER("CShwSlideShowSettingsList::SetPlayDirectionL"); + GLX_LOG_INFO("CShwSlideShowSettingsList::SetMusicStateL"); + if (aNewPlayDirection != iPlayDirection) + { + // retrieve play direction setting + (*SettingItemArray())[EPlayDirectionItem]->EditItemL(EFalse); + // updates play direction + iPlayDirection = aNewPlayDirection; + // internalise new state to settings item + (*(SettingItemArray()))[EPlayDirectionItem]->StoreL(); + + // persist direction + iShwSettings.SavePlayOrderL(iPlayDirection); + //@TODO cenrep update + + // redraw music on/off control + ListBox()->DrawItem(EPlayDirectionItem); + } + } //----------------------------------------------------------------------------- // CShwSlideShowSettingsList::ConfirmationQueryL //-----------------------------------------------------------------------------