Ticket #1527 (closed 改善提案: 修正済)
端末種別の判別ルーチンの統一
Reported by: | shutta | Owned by: | shutta |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.12.0beta |
Component: | その他 | Version: | 2.11.3 |
Keywords: | Cc: | ||
修正済み: | yes |
Description
の、
端末種別の判別 Net_UserAgent_Mobile::isMobile() や MOBILE_SITE 定数など存在するが, SC_Display_Ex::detectDevice() に統一する
に準拠する。
Change History
comment:1 Changed 9 years ago by shutta
- Owner changed from somebody to shutta
- Status changed from new to assigned
- Component changed from フロント to その他
comment:2 Changed 9 years ago by shutta
SC_Mobile_UserAgent::isNonMobile()は、自クラスのisMobileメソッドの逆関数になっていたので、対象からは外し、コメントを記載しておいた。
これで、
SC_Mobile_UserAgent::isMobile()
SC_Mobile_UserAgent::isNonMobile()
のメソッドは不要なメソッドになったはず。
ただ、モジュールで利用されているかどうかが分からないし、
マイナーバージョンアップの指針ではメソッドを削除しないので、すぐには廃止はできない。
とりあえずは、メソッドのコメントに@deprecatedな説明を付加しておいた方がよいか?
Note: See
TracTickets for help on using
tickets.