Enhancement（機能追加・改善） #1160
Implement auto link in the mobile_frontend. (モバイル用の自動リンク機能）
100%
説明
Overview¶
Implement auto link in the mobile_frontend.
However, when auto linking url is outer sns, it url link proxy page to prevent a session hijack.
(This measures are especially necessary for DoCoMo.)
For example:
http://sns.example.com/member/1 -> <a href="/member/1">http://sns.example.com/member/1</a>
http://google.com -> <a href="http://sns.example.com/default/linkProxy?url=http%3a%2f%2fgoogle%2ecom</a>
Way to Enhancement¶
Add helper function to opUtil.
It name "op_auto_link_for_mobile()"
関連するチケット
関係しているリビジョン
added op_auto_link_text_for_mobile() (fixes #1160)
fixed NOTICE of op_auto_link_text_for_mobile() (refs #1160)
fixed a template of member/profile on the mobile_frontend for autolink (fixes #1160)
fixed op_auto_link_text_for_mobile() doesn't link phone-number correctly (refs #1160)
fixed op_auto_link_text_for_mobile() as default_mobile_auto_link_type accepted ['all'] (refs #1160)
fixed op_auto_link_text_for_mobile(), because href attribute is not correct (refs #1160)
fixed _op_auto_links_urls() to use basename() (refs #1160)
fixed _op_auto_links_urls(), bacause the regular expression is not correct (fixes #1160)
fixed _op_auto_links_urls(), because regular expression for script name is not correct (fixes #1160)
履歴
#1
Shogo Kawahara がほぼ14年前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Shogo Kawahara にセット
- 対象バージョン を OpenPNE 3.6beta1 にセット
#2
Shogo Kawahara がほぼ14年前に更新
テスターへ（業務連絡）
以下の項目をよくチェックしてください
- URL中継ページ ( /proxy でセッションIDがURLに含まれていないか。特に Cookieの利用できない docomo端末で厳密にチェックしてください）
#3
Shogo Kawahara がほぼ14年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 1e9b8854689b69361b60cf8c0345767f238c5cfb で適用されました。
#4
Shogo Kawahara がほぼ14年前に更新
更新履歴 bba9401dc094bcbd9d1e1e2cd8fe407b3d524712 で適用されました。
#5
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
3点フィードバックします。
1. 内部リンクのURLのパス部分がなくなってしまう¶
例: http://example.com/hoge/fuga/member/search
<a href="/member/search">/member/search</a>（"/hoge/fuga" の部分がなくなっている）
.
2. 電話番号の末尾に不要な < が表示される¶
例: 06-6012-3456
<a href="tel:0660123456"> 06-6012-3456<</a>
.
3. OpenPNE.yml の設定で default_mobile_auto_link_type: ['all'] とすると動かない¶
正しくは、 default_mobile_auto_link_type: 'all' ですが、実際にハマりました。追加説明が必要だと思います。
#6
Shogo Kawahara がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
#8
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
#9
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
#10
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
#11
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
_op_auto_links_urls()
- script_name の抽出には basename() が使えると思います
- 正規表現の
(?:\/.*\.php)?の部分ですが、スクリプト名よりも後ろの部分に.phpという文字列が出てくると意図通りにマッチしないので修正してください（例： http://example.com/index.php/diary/search?keyword=hoge.php）
#12
Shogo Kawahara がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 b9ce105421d6b17553f1ea94920eb1f129acc029 で適用されました。
#13
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
正規表現の修正ですが、 http://example.com/diary/search?keyword=hoge.php のような URL ではまだ上手くいきません。
#14
Shogo Kawahara がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 b6f4812f02912e9d2726e14ec143470388c8645f で適用されました。
#15
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
#16
Kousuke Ebihara がほぼ14年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更