#1968 - Calculatr addon results in error

By Guest
Added 16th Jun 2015, 8:59 PM
26 views

Identifier	#1968
Issue type	Feature request or suggestion
Title	Calculatr addon results in error
Status	Completed
Handling member	Chris Graham
Addon	General / Uncategorised
Description	When calculatr is added and calculate button is pressed it results in error 1: SyntaxError: missing ) after argument list
Steps to reproduce	Add the calculator addon enter the values Click on calculate button
Funded?	No

The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Rating

Unrated

Comments

By Guest, By Guest, posted 16th Jun 2015, 8:59 PM

By Guest posted 26th Jun 2015, 11:30 PM

I committed a fix to github for this error. It was due to uniqid('',true) returning an ID with a decimal point in it and Java didn't like that. uniqid() would probably be good enough, but I chose to wrap it in a str_replace to replace the decimal point with an underscore. The cache $map array key was causing an extra input field on the calculator form, so I added an additional $key check for cache.

#1968 - Calculatr addon results in error

Rating

Comments

Statistics