From 4eede86ce64dbee18c54928ed9a5ad22bbc34b3e Mon Sep 17 00:00:00 2001
From: Rohit Nirmal <rohitnirmal9@gmail.com>
Date: Sun, 9 Jun 2013 23:27:47 -0500
Subject: [PATCH] Moved journal open/close sound effect playback to
 JournalWindowImpl, and have opening sound play only if the player has the
 journal.

---
 apps/openmw/mwgui/journalwindow.cpp     | 10 +++++++---
 apps/openmw/mwinput/inputmanagerimp.cpp |  3 +--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/apps/openmw/mwgui/journalwindow.cpp b/apps/openmw/mwgui/journalwindow.cpp
index 5b84c99a5..e7cb9916f 100644
--- a/apps/openmw/mwgui/journalwindow.cpp
+++ b/apps/openmw/mwgui/journalwindow.cpp
@@ -183,7 +183,12 @@ namespace
             if (!MWBase::Environment::get().getWindowManager ()->getJournalAllowed ())
             {
                 MWBase::Environment::get().getWindowManager()->popGuiMode ();
-            } 
+            }
+            else
+            {
+				// play opening sound only if the player has the journal
+				MWBase::Environment::get().getSoundManager()->playSound ("book open", 1.0, 1.0);
+			}
             mModel->load ();
 
             setBookMode ();
@@ -213,6 +218,7 @@ namespace
 
         void close()
         {
+			MWBase::Environment::get().getSoundManager()->playSound ("book close", 1.0, 1.0);
             mModel->unload ();
 
             getPage (LeftBookPage)->showPage (Book (), 0);
@@ -432,8 +438,6 @@ namespace
 
         void notifyClose(MyGUI::Widget* _sender)
         {
-            MWBase::Environment::get().getSoundManager()->playSound ("book close", 1.0, 1.0);
-
             MWBase::Environment::get().getWindowManager ()->popGuiMode ();
         }
 
diff --git a/apps/openmw/mwinput/inputmanagerimp.cpp b/apps/openmw/mwinput/inputmanagerimp.cpp
index 00c520de9..e9a0e5089 100644
--- a/apps/openmw/mwinput/inputmanagerimp.cpp
+++ b/apps/openmw/mwinput/inputmanagerimp.cpp
@@ -658,12 +658,11 @@ namespace MWInput
 
         if(gameMode)
         {
-            MWBase::Environment::get().getSoundManager()->playSound ("book open", 1.0, 1.0);
             mWindows.pushGuiMode(MWGui::GM_Journal);
         }
         else if(mWindows.getMode() == MWGui::GM_Journal)
         {
-            MWBase::Environment::get().getSoundManager()->playSound ("book close", 1.0, 1.0);
+            
             mWindows.popGuiMode();
         }
         // .. but don't touch any other mode.