puプルリクを送ったらライブラリを壊してしまった話

1. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
プルリク送ったら
ライブラリ壊し
たった
鍋太郎

2. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
自己紹介
プログラマー✓
Debian JP 事務局✓
小市若者連 委員長 ←New✓
https://about.me/nabetaro✓

3. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
Phony
電話番号を扱う✓
世界中が目標✓
E164準拠✓

4. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
使い方
Phonyモジュールのメソッド
normalize 正規化✓
format 整形✓
etc...✓
✓
Phony['国番号']
その国の電話番号が使える…はず✓
✓

5. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
使ってみる
Phony.normalize('+81 3-1234-5678')
=> "81312345678"
Phony.normalize('03-1234-5634', cc: '81')
=> "81312345634"
Phony["81"].normalize('+81 3-1234-5634')
=> "81312345634"
Phony["81"].normalize('03-1234-5634')
=> "312345634" # !?

6. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
使ってみる
Phony.format('81312345634', cc: '81', format: :international)
=> "+81-3-1234-5634"
Phony.format('81312345634', cc: '81', format: :national)
=> "03-1234-5634"
Phony.format('81312345634', cc: '81', format: :local)
=> "1234-5634"
Phony['81'].format('0312345634', format: :international)
=> "+81-03-1234-5634" # !?
Phony['81'].format('0312345634', format: :national)
=> "003-1234-5634" # !?
Phony['81'].format('0312345634', format: :local)
=> "1234-5634"

7. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
犯行動機
国内で利用している電話番号も
使えるようにしたかった。
'0312345634' を 03-1234-5634 に✓
✓

8. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
犯行内容
テストケースを書いてプルリク
送った
✓
https://github.com/floere/phony/pull/453
2日で修正された

9. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
結果
Phony.format('81312345634', cc: '81', format: :international)
=> "+81-3-1234-5634"
Phony.format('81312345634', cc: '81', format: :national)
=> "03-1234-5634"
Phony.format('81312345634', cc: '81', format: :local)
=> "1234-5634"
Phony['81'].format('0312345634', format: :international)
=> "+81-3-1234-5634"
Phony['81'].format('0312345634', format: :national)
=> "03-1234-5634"
Phony['81'].format('0312345634', format: :local)
=> "1234-5634"
ええやん

10. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
結果
Phony.normalize('+81 3-1234-5678')
=> "81312345678"
Phony.normalize('03-1234-5634', cc: '81')
=> "810312345634" # !!!
Phony["81"].normalize('+81 3-1234-5634')
=> "81312345634"
Phony["81"].normalize('03-1234-5634')
=> "0312345634"
まって、まって、まって

11. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
影響
phony_rails
rails向けライブラリ✓
電話番号の正規化✓
電話番号のvalidation✓
✓
https://github.com/joost/phony_rails
おかしくなっとる!!!

12. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
自分の取った対応
追加テストケースのプルリク
https://github.com/floere/phony/
pull/456
✓
未マージ✓
✓
revertのプルリク
https://github.com/floere/phony/
pull/457
✓
✓

13. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
おすすめ対応
2.18.10 で止めておく✓
プルリク #457 を適用✓

14. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
考察
Phony["81"].normalizeの結
果の先頭に国番号を付加
✓
日本の電話番号は 国内プレ
フィックス「0」を取って国番
号を付加
✓

15. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
余談
市外局番には国内プレフィック
ス(trunk)は含まれない
https://en.wikipedia.org/wiki/
Telephone_numbers_in_Japan
Talkページで先頭の0について議論してる✓
わかりにくい…✓
✓
✓

16. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0
まとめ
オチはありません。✓
良かれと思ってもうまくいかな
いこともあります。
✓
適切な改善案を出しましょう。✓
おちこんだりもしたけれど、私
はげんきです。
✓