[Moneymanagerex-commits] SF.net SVN: moneymanagerex:[2801] branches/0.9.9.0/mmex/src/mmex.cpp

[<vo...@us...>]

Revision: 2801
          http://moneymanagerex.svn.sourceforge.net/moneymanagerex/?rev=2801&view=rev
Author:   vomikan
Date:     2012-07-23 18:59:19 +0000 (Mon, 23 Jul 2012)
Log Message:
-----------
fix

Modified Paths:
--------------
    branches/0.9.9.0/mmex/src/mmex.cpp

Modified: branches/0.9.9.0/mmex/src/mmex.cpp
===================================================================
--- branches/0.9.9.0/mmex/src/mmex.cpp	2012-07-23 14:38:33 UTC (rev 2800)
+++ branches/0.9.9.0/mmex/src/mmex.cpp	2012-07-23 18:59:19 UTC (rev 2801)
@@ -3679,8 +3679,6 @@
 
 void mmGUIFrame::OnOnlineUpdateCurRate(wxCommandEvent& /*event*/)
 {
-    wxString site = wxT("http://download.finance.yahoo.com/d/quotes.csv?s=%s&f=sl1n&e=.csv");
-
     const int currencyID = mmDBWrapper::getBaseCurrencySettings(m_core->db_.get());
     const wxString base_symbol = mmDBWrapper::getCurrencySymbol(m_core->db_.get(), currencyID);
     if(base_symbol.IsEmpty())
@@ -3689,15 +3687,15 @@
         return;
     }
 
-    wxString temp;    
+    wxString site;    
     for (int idx = 0; idx < (int)m_core->currencyList_.currencies_.size(); idx++)
     {
         const wxString symbol = m_core->currencyList_.currencies_[idx]->currencySymbol_.Upper();
         if (symbol != base_symbol)
-            temp << symbol << base_symbol << wxT("=X+");
+            site << symbol << base_symbol << wxT("=X+");
     }
-    site.RemoveLast(1);
-    site = wxString::Format(site, temp);
+    if (site.Right(1).Contains(wxT("+"))) site.RemoveLast(1);
+    site = wxString::Format(wxT("http://download.finance.yahoo.com/d/quotes.csv?s=%s&f=sl1n&e=.csv"), site.c_str());
 
     wxString rates;
     int err_code = site_content(site, rates);

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