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

日付に 2015/08/04 のように 08 が入ると動かない #14

Open
unok opened this issue Aug 4, 2015 · 2 comments
Open

日付に 2015/08/04 のように 08 が入ると動かない #14

unok opened this issue Aug 4, 2015 · 2 comments

Comments

@unok
Copy link

unok commented Aug 4, 2015

導入が失敗して調査していて気づいたのですが、開始日などに 2015/08/04 のように 08 があると DateUtils が正常に動かないようです。

Google Apps Script のバグなのか、parseInt('08') すると NaN が帰ってきてしまいます。
8進数処理をしてしまい、8進数として不正なために NaN が帰ってきてしまう?ようです。
parseInt('08', 10) とすれば 8 が帰ってきます。
ちなみに、Mac の npm テストで上記を処理すると NaN にならずに 8 が帰ってくるようです。
Google Apps Script では、全般的に 10 進数を想定している箇所には 10 を明示的に指定した方が良さそうです。
修正しようと思ったのですが、DateUtils 以外でも利用している場所がありそうなので、修正できていません。。。

http://stackoverflow.com/questions/17204249/why-does-parseint-return-nan-for-08-string-and-return-7-for-07-string

@YusukeKokubo
Copy link

同様の問題に当たりました。
とりあえず開始日を '2015/8/26 のようにして回避しました…。

@indigolain
Copy link

こちらでparseIntを明示的に10進数に変換する修正が加わたことによってこちらのissueが解決したと思われるので、ご確認いただければと思います 💡
#20

halsk added a commit to HackCamp/miyamoto that referenced this issue Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants