Ticket #1527 (closed 改善提案: 修正済)

Opened 9 years ago

Last modified 9 years ago

端末種別の判別ルーチンの統一

Reported by: shutta Owned by: shutta
Priority: 中 Milestone: EC-CUBE2.12.0beta
Component: その他 Version: 2.11.3
Keywords: Cc:
修正済み: yes

Description

EC-CUBE標準規約/リファクタリングガイドライン

の、 

端末種別の判別

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

r21302, r21303 にて修正。

SC_Mobile_UserAgent::isNonMobile()は、自クラスのisMobileメソッドの逆関数になっていたので、対象からは外し、コメントを記載しておいた。

これで、
SC_Mobile_UserAgent::isMobile()
SC_Mobile_UserAgent::isNonMobile()
のメソッドは不要なメソッドになったはず。
ただ、モジュールで利用されているかどうかが分からないし、
マイナーバージョンアップの指針ではメソッドを削除しないので、すぐには廃止はできない。

とりあえずは、メソッドのコメントに@deprecatedな説明を付加しておいた方がよいか？

comment:3 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

comment:4 Changed 9 years ago by h_yoshimoto

  • Priority changed from 低 to 中

comment:5 follow-up: ↓ 6 Changed 9 years ago by shutta

  • 修正済み set

r21438 により、懸念だった下記メソッドを削除。

SC_Mobile_UserAgent::isMobile()

SC_Mobile_UserAgent::isNonMobile()

comment:6 in reply to: ↑ 5 Changed 9 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

shutta への返信

r21438 により、懸念だった下記メソッドを削除。

SC_Mobile_UserAgent::isMobile()

SC_Mobile_UserAgent::isNonMobile()

ありがとうございました。

正常に動作している事を確認しましたのでクローズさせて頂きます。

Note: See TracTickets for help on using tickets.