[Moneymanagerex-commits] SF.net SVN: moneymanagerex:[1757] trunk/mmex/src/splittransactionsdialog. cpp

[<ste...@us...>]

Revision: 1757
          http://moneymanagerex.svn.sourceforge.net/moneymanagerex/?rev=1757&view=rev
Author:   stef145g
Date:     2012-02-07 01:21:02 +0000 (Tue, 07 Feb 2012)
Log Message:
-----------
Fix: Prevent user from accessing edit dialogue when being used as a View Screen

Modified Paths:
--------------
    trunk/mmex/src/splittransactionsdialog.cpp

Modified: trunk/mmex/src/splittransactionsdialog.cpp
===================================================================
--- trunk/mmex/src/splittransactionsdialog.cpp	2012-02-06 14:30:33 UTC (rev 1756)
+++ trunk/mmex/src/splittransactionsdialog.cpp	2012-02-07 01:21:02 UTC (rev 1757)
@@ -260,10 +260,9 @@
 
 void SplitTransactionDialog::EditEntry()
 {
-    // locate the selected item in the list
+    // locate the selected item in the list and exit if not found
     long item = this->GetSelectedItem();
-    if (item == wxNOT_FOUND)
-        return;
+    if (item == wxNOT_FOUND) return;
 
     int categID    = split_->entries_[item]->categID_;
     int subCategID = split_->entries_[item]->subCategID_;
@@ -279,17 +278,11 @@
         DataToControls();
         UpdateSplitTotal();
     }
-
-    //category = core_->categoryList_.GetFullCategoryString(categID,subCategID);
-    //wxString d_msg = wxString()
-    //    << wxT("Selected Item: ") << item << wxT("\n\n")
-    //    << wxT("Category: ") << category << wxT("\n");
-    //wxLogMessage(d_msg); 
 }
 
 void SplitTransactionDialog::OnListDblClick(wxListEvent& /*event*/)
 {
-    EditEntry();
+    if (itemButtonEdit_->IsShown()) EditEntry();
 }
 
 void SplitTransactionDialog::SetDisplaySplitCategories()

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.