You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by ri...@apache.org on 2006/05/19 12:58:41 UTC

svn commit: r407770 - /incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java

Author: rineholt
Date: Fri May 19 03:58:40 2006
New Revision: 407770

URL: http://svn.apache.org/viewvc?rev=407770&view=rev
Log:
TUSCANY-408
http://issues.apache.org/jira/browse/TUSCANY-408
Bigbank currency conversion is not working because property setter methods are not actually invoked

Modified:
    incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java

Modified: incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java?rev=407770&r1=407769&r2=407770&view=diff
==============================================================================
--- incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java (original)
+++ incubator/tuscany/java/sampleapps/bigbank/account/src/main/java/bigbank/account/services/account/AccountServiceImpl.java Fri May 19 03:58:40 2006
@@ -56,25 +56,31 @@
         AccountServiceImpl.tsformatXSDDateTime.setTimeZone(TimeZone.getTimeZone("UTC"));
     }
 
-    private float currencyConversion = 1.0f; 
+    private float currencyConversion = 0.0f; 
     private String currency= "USD";
 
     @Property
     public void setCurrency(final String currency) {
-        this.currency= currency == null ? this.currency:  currency.trim();
-
-        if ("USD".equals(this.currency))
-            currencyConversion= 1.0f;
-        else if ("EURO".equals(this.currency))
-            currencyConversion= 0.8f;
-        else{
-            try {
-                currencyConversion = Float.parseFloat(this.currency);
-            } catch (Exception e) {
+        this.currency= currency;
+    }
+    float getCurrencyConversion(){
+        if(currencyConversion == 0.0F){
+            if ("USD".equals(this.currency))
                 currencyConversion= 1.0f;
-            }            
-           
+            else if ("EURO".equals(this.currency))
+                currencyConversion= 0.8f;
+            else{
+                try {
+                    currencyConversion = Float.parseFloat(this.currency);
+                } catch (Exception e) {
+                    currencyConversion= 1.0f;
+                }            
+               
+            }
+
+            
         }
+        return currencyConversion;
         
     }
 
@@ -163,13 +169,13 @@
     }
 
     private float fromUSDollarToCurrency(float value) {
-        return value * currencyConversion ;
+        return value * getCurrencyConversion() ;
 
    
     }
     private float toUSDollarfromCurrency(float value) {
 
-        return  value /currencyConversion ; 
+        return  value /getCurrencyConversion() ; 
     }
     
 



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org