Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BugFix: Must specify base with to integer: to_integer(value, 10) #305

Closed
wants to merge 1 commit into from

Conversation

connorsml
Copy link
Contributor

Otherwise months/days 08 and 09 will not validate.

Otherwise months/days 08 and 09 will not validate.
@mworrell
Copy link
Member

mworrell commented Apr 3, 2012

Also, a bit later in the code the function to_integer is called as to_integer(date_components[0]).

I guess the parseInt function should only have the additional ,10 ?

(Good catch though, this kind of radix guessing is a source of mysterious bugs)

@connorsml
Copy link
Contributor Author

Yes, sorry. I will do it again properly.

On 3 April 2012 16:20, Marc Worrell <
reply@reply.github.com

wrote:

Also, a bit later in the code the function to_integer is called as
to_integer(date_components[0]).

I guess the parseInt function should only have the additional ,10 ?

(Good catch though, this kind of radix guessing is a source of mysterious
bugs)


Reply to this email directly or view it on GitHub:
#305 (comment)

Michael Connors
Alencon,
France

Tel Ireland: 076 615 1147
Tel France: 061 460 1909
International: 00 353 76 615 1147
http://www.michaelconnors.net

@connorsml connorsml closed this Apr 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants