At user/1/subscription/signup there is an error:
Notice: Undefined variable: thousands_separator in recurly_format_currency() (line 783 of C:\wamp\www\commerce\sites\all\modules\recurly\recurly.module).
Notice: Undefined variable: decimal_separator in recurly_format_currency() (line 783 of C:\wamp\www\commerce\sites\all\modules\recurly\recurly.module).
Notice: Undefined variable: thousands_separator in recurly_format_currency() (line 783 of C:\wamp\www\commerce\sites\all\modules\recurly\recurly.module).
Notice: Undefined variable: decimal_separator in recurly_format_currency() (line 783 of C:\wamp\www\commerce\sites\all\modules\recurly\recurly.module).
I have created a test subscription on Recurly called Test. It is 7.99 per month. This is the output of the subscription. It is missing the 7.
Test
.99 per month
7 day trial
Contact us to sign up
This is just a test plan for this system
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | recurly_currency_notices.patch | 3.12 KB | quicksketch |
Comments
Comment #1
quicksketchThanks, I hadn't noticed this problem because I don't have Commerce installed (Recurly does not currently depend upon it). Really we could ditch the usage of Commerce's format_currency entirely, since Recurly only supports a certain set of currencies anyway. In any case the problem is fixed with this patch, which I've committed to the project.