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()"
Shogo Kawahara さんが14年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Shogo Kawahara にセット
- 対象バージョン を OpenPNE 3.6beta1 にセット
Shogo Kawahara さんが14年以上前に更新
テスターへ(業務連絡)
以下の項目をよくチェックしてください
- URL中継ページ ( /proxy でセッションIDがURLに含まれていないか。特に Cookieの利用できない docomo端末で厳密にチェックしてください)
Shogo Kawahara さんが14年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 1e9b8854689b69361b60cf8c0345767f238c5cfb で適用されました。
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'
ですが、実際にハマりました。追加説明が必要だと思います。
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)
Shogo Kawahara さんが14年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 b9ce105421d6b17553f1ea94920eb1f129acc029 で適用されました。
Rimpei Ogawa さんが14年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
正規表現の修正ですが、 http://example.com/diary/search?keyword=hoge.php のような URL ではまだ上手くいきません。
Shogo Kawahara さんが14年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 b6f4812f02912e9d2726e14ec143470388c8645f で適用されました。
Rimpei Ogawa さんが14年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Kousuke Ebihara さんが14年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更