Enhancement（機能追加・改善） #1160
closed
Implement auto link in the mobile_frontend. (モバイル用の自動リンク機能）
100%
Description
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()"
Updated by Shogo Kawahara over 14 years ago
- Status changed from New（新規） to Accepted（着手）
- Assignee set to Shogo Kawahara
- Target version set to OpenPNE 3.6beta1
Updated by Shogo Kawahara over 14 years ago
テスターへ（業務連絡）
以下の項目をよくチェックしてください
- URL中継ページ ( /proxy でセッションIDがURLに含まれていないか。特に Cookieの利用できない docomo端末で厳密にチェックしてください）
Updated by Shogo Kawahara over 14 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
更新履歴 1e9b8854689b69361b60cf8c0345767f238c5cfb で適用されました。
Updated by Shogo Kawahara over 14 years ago
更新履歴 bba9401dc094bcbd9d1e1e2cd8fe407b3d524712 で適用されました。
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Pending Review（レビュー待ち） to 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' ですが、実際にハマりました。追加説明が必要だと思います。
Updated by Shogo Kawahara over 14 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
Updated by Shogo Kawahara over 14 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
修正しました
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
_op_auto_links_urls()
- script_name の抽出には basename() が使えると思います
- 正規表現の
(?:\/.*\.php)?の部分ですが、スクリプト名よりも後ろの部分に.phpという文字列が出てくると意図通りにマッチしないので修正してください（例： http://example.com/index.php/diary/search?keyword=hoge.php）
Updated by Shogo Kawahara over 14 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
更新履歴 b9ce105421d6b17553f1ea94920eb1f129acc029 で適用されました。
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
正規表現の修正ですが、 http://example.com/diary/search?keyword=hoge.php のような URL ではまだ上手くいきません。
Updated by Shogo Kawahara over 14 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
更新履歴 b6f4812f02912e9d2726e14ec143470388c8645f で適用されました。
Updated by Rimpei Ogawa over 14 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
Updated by Kousuke Ebihara over 14 years ago
- Status changed from Pending Testing（テスト待ち） to Fixed（完了）
- % Done changed from 70 to 100