View Issue Details

IDProjectCategoryView StatusLast Update
0005903OXID eShop (all versions)1. ----- eShop frontend -----public2014-12-18 15:16
Reporterfrank Assigned To 
PrioritynormalSeveritytweakReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.8.0 / 5.1.0 
Target Version4.9.3 / 5.2.3Fixed in Version4.9.3 / 5.2.3 
Summary0005903: currency sign in front of amount still wrong
DescriptionI was very happy to see that with version 4.7, AFTER TEN YEARS, OXID finally added the feature to optionally put the currency sign in front of the amount.

... and in the documentation http://wiki.oxidforge.org/Downloads/4.8.0_5.1.0 it is stated correctly:

Example: "$17.00" instead of "17.00 $"


However, I got shocked when I saw, that it seems to be impossible to really get it displayed like that.

I always get a space in-between:

$ 17.00

The space is simply WRONG. For which countries was this programmed, if not for US, Canada and Australia. Show me any (serious) shop in one of these countries with the space in-between. It looks SO ODD, that it's still a killer for every American user.


Additionally there are spots were it's still displayed behind the amount - so far I found it in the small cart preview pop-up grand total. Compared to the main bug that's of course relatively very minor.

TagsCurrency
ThemeAzure
BrowserAll
PHP Versionany
Database Versionany

Activities

frank

2014-10-03 08:19

reporter   ~0010208

At least the fix is easy:

core/smarty/plugins/oxprice.php - line 63 change to:

$sOutput = ( isset($sSide) && $sSide == 'Front' ) ? $sCurrencySign . $sPrice : $sPrice . ' ' . $sCurrencySign;

frank

2014-10-03 15:40

reporter   ~0010209

... and please don't forget this:

Additionally there are spots were it's still displayed behind the amount - so far I found it in the small cart preview pop-up grand total (clicking on the shopping bag upper right corner)

jurate.baseviciene

2014-10-08 12:46

reporter   ~0010223

Reminder sent to: frank

Hi,

 As we saw, you already submitted a complete solution for this issue. If you feel fancy, you'd also have the possibility to contribute your changes directly to our GitHub repository on https://github.com/OXID-eSales/oxideshop_ce/. Please leave a note there with the bug number you fixed so we can close this issue in the bug tracker."

Best regards
Jurate

Linas Kukulskis

2014-12-18 15:16

reporter   ~0010560

removed space