Givewp: Donation total not respect to the number of decimal

Created on 6 Dec 2017  ยท  8Comments  ยท  Source: impress-org/givewp

Issue Overview

  • Donation total does not respect to the number of decimal while switching on donation amount value.
  • Same with Custom amount
  • Issue exist on latest Give 1.8.17

nimbus-record-video-2017-12-06-11-07-12

Expected Behavior


  • The number of decimal should remain same when switching to the donation amount or gateway.

Current Behavior


  • Currently, It does not remain same and change from $10.00 to $10 while switching to the amount.
  • Also with the Custom amount, it does not work.

Possible Solution


Steps to Reproduce (for bugs)


  1. Create a donation form with Multi-level donation with various amount
  2. Switching among the amount
  3. As you can see that decimal value does not remain same when switching

Related Issues and/or PRs

Todos

  • [ ] Tests

WordPress Environment

### WordPress Environment ###

Home URL: http://givewp.dev
Site URL: http://givewp.dev
WP Version: 4.9.1
WP Multisite: โ€“
WP Memory Limit: 256 MB
WP Debug Mode: โœ”
WP Cron: โœ”
Language: en_US
Permalink Structure: /%year%/%monthnum%/%day%/%postname%/
Show on Front: posts
Table Prefix Length: 3
Table Prefix Status: Acceptable
Admin AJAX: Inaccessible
Registered Post Statuses: publish, future, draft, pending, private, trash, auto-draft, inherit, refunded, failed, revoked, cancelled, abandoned, processing, preapproval

### Server Environment ###

Hosting Provider: DBH: localhost, SRV: givewp.dev
TLS Connection: Connection uses TLS 1.2
TLS Connection: Probably Okay
Server Info: nginx/1.10.1
PHP Version: โŒ 5.3.29 - We recommend a minimum PHP version of 5.6. See: PHP Requirements in Give
PHP Post Max Size: 1,000 MB
PHP Time Limit: 900
PHP Max Input Vars: 3000
PHP Max Upload Size: 1,000 MB
cURL Version: โŒ 7.38.0, OpenSSL/1.0.2h - We recommend a minimum cURL version of 7.40.
SUHOSIN Installed: โ€“
MySQL Version: โŒ 5.5.55 - We recommend a minimum MySQL version of 5.6. See: WordPress Requirements
Default Timezone is UTC: โœ”
fsockopen/cURL: โœ”
SoapClient: โœ”
DOMDocument: โœ”
gzip: โœ”
GD Graphics Library: โœ”
Multibyte String: โœ”
Remote Post: โœ”
Remote Get: โœ”

### Give Configuration ###

Give Version: 1.8.17
Upgraded From: 1.8.17
Test Mode: Enabled
Currency Code: INR
Currency Position: Before
Decimal Separator: .
Thousands Separator: ,
Success Page: http://givewp.dev/donation-confirmation/
Failure Page: http://givewp.dev/donation-failed/
Donation History Page: http://givewp.dev/donation-history/
Give Forms Slug: /donations/
Enabled Payment Gateways: Test Donation, Offline Donation, PayPal Standard
Default Payment Gateway: Test Donation
PayPal IPN Verification: Enabled
PayPal IPN Notifications: N/A
Admin Email Notifications: Disabled
Donor Email Access: Enabled

### Session Configuration ###

Give Use Sessions: Enabled
Session: Disabled

### Active Give Add-ons ###


### Other Active Plugins ###


### Inactive Plugins ###

Give - Currency Switcher: by WordImpress โ€“ 1.0.0
Give - Fee Recovery: by WordImpress โ€“ 1.3.2
Give - Form Field Manager: by WordImpress โ€“ 1.2.3
Give - Gift Aid: by WordImpress โ€“ 1.0.0
Give - PDF Receipts: by WordImpress โ€“ 2.1
Give - Recurring Donations: by WordImpress โ€“ 1.4
Give - Setting API Fields Example: by WordImpress โ€“ 1.0
Give - Stripe Gateway: by WordImpress โ€“ 1.5
Give - Tributes: by WordImpress โ€“ 1.2

### Active MU Plugins ###

Local by Flywheel Relative URL (for Live Links): by Flywheel โ€“ 1.0

### Theme ###

Name: Twenty Seventeen
Version: 1.4
Author URL: https://wordpress.org/
Child Theme: No โ€“ If you're modifying Give on a parent theme you didn't build personally, then we recommend using a child theme. See: How to Create a Child Theme

All 8 comments

I think it's okay as .00 is not so important

@raftaar1191 it is important but @jaydeeprami I am unable to reproduce. Can you try on a fresh install and create a video? I'm suspect that this is an conflict with another add-on or plugin and not a core issue.

Sure @DevinWalker , I will create a video of fresh install.

@DevinWalker, I have just checked with the Fresh install and cannot reproduce this issue. Also, In my Previous install, there is a no any Add-on activated.

@DevinWalker, I can able to reproduce this issue with Indian Rupee.

@jaydeeprami Can you share the GIve Setting from the old sites by export it from the Tools > Export > Export Give Settings

@jaydeeprami this appears resolved now after merging #2462 - can you pull down the latest and confirm?

@DevinWalker, Yes, it's resolved.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

marutim picture marutim  ยท  4Comments

ravinderk picture ravinderk  ยท  3Comments

DevinWalker picture DevinWalker  ยท  4Comments

DevinWalker picture DevinWalker  ยท  3Comments

mathetos picture mathetos  ยท  4Comments