OpenPNE 3

• config/OpenPNE.yml.sample の is_use_mobile_cookie の設定ですが、コメントインしてください。この状態ではデフォルト値が false である（Cookie を使いたい場合はコメントアウトして使う）と誤解されかねません。ここでの設定値は真偽値なのですから、 is_use_mobile_cookie: true をコメントなしで設定させておいて、必要が生じた場合に is_use_mobile_cookie: false に変更させるようになっていたほうが直感的でわかりやすいです。

• もともとあった if 文の端末判定のための条件式のなかに $isUseMobileCookie 変数のチェックを追加していますが、これはあまり直感的ではないと思います。もともとの条件式はまあ複雑ではあるといえ端末判定であるということがある程度明確でしたが、ここに $isUseMobileCookie の判定が加わったことでその意味が失われました。このメソッドにおいて、 $isUseMobileCookie が false なのであれば、端末判定などする必要もなく false を返却して構わないので、それを示すために、もともとあった端末判定の if 文の前、 op_is_use_mobile_cookie の設定値を取得した直後のタイミングで別の if 文として $isUseMobileCookie の判定をおこなうことで、 $isUseMobileCookie の判定と端末の判定それぞれに別々の意図が生まれ、可読性が向上すると思いますので修正を検討してください。

更新履歴 3a9c129b3d19f88bef5019b3c33219358270c5ef で適用されました。

テスト完了致しました。
問題ありませんのでステータスをFixedに致します。