Num::format() should not include the currency symbol, money_format() …

…should be used for this purpose
kohana · Nov 22, 2009 · 990e31d · 990e31d
1 parent f922e84
commit 990e31d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/classes/kohana/num.php b/classes/kohana/num.php
@@ -36,11 +36,11 @@ public static function ordinal($number)
 	}
 
 	/**
-	 * Format a number.
+	 * Locale-aware number formatting.
 	 *
 	 * @param   float    number to format
 	 * @param   integer  decimal places
-	 * @param   boolean  monetary format?
+	 * @param   boolean  monetary formatting?
 	 * @return  string
 	 */
 	public static function format($number, $places, $monetary = FALSE)
@@ -51,16 +51,14 @@ public static function format($number, $places, $monetary = FALSE)
 		{
 			$decimal   = $info['mon_decimal_point'];
 			$thousands = $info['mon_thousands_sep'];
-			$currency  = $info['currency_symbol'];
 		}
 		else
 		{
 			$decimal   = $info['decimal_point'];
 			$thousands = $info['thousands_sep'];
-			$currency  = NULL;
 		}
 
-		return $currency.number_format($number, $places, $decimal, $thousands);
+		return number_format($number, $places, $decimal, $thousands);
 	}
 
 	final private function __construct()