プロジェクト

全般

プロフィール

Enhancement(機能追加・改善) #1160

完了

Implement auto link in the mobile_frontend. (モバイル用の自動リンク機能)

Shogo Kawahara さんが14年以上前に追加. 14年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-06-11
期日:
進捗率:

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&lt;/a>

http://google.com -> <a href="http://sns.example.com/default/linkProxy?url=http%3a%2f%2fgoogle%2ecom&lt;/a>

Way to Enhancement

Add helper function to opUtil.

It name "op_auto_link_for_mobile()"


関連するチケット 2 (1件未完了1件完了)

関連している opDiaryPlugin - Enhancement(機能追加・改善) #1323: モバイル自動リンクに対応Fixed(完了)Shogo Kawahara2010-07-13

操作
関連している OpenPNE 3 - Bug(バグ) #1641: モバイル自動リンクが半角英数以外が含まれるURLやメールアドレスを正しくリンクに変換しないNew(新規)2010-10-04

操作

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 で適用されました。

Shogo Kawahara さんが14年以上前に更新

更新履歴 bba9401dc094bcbd9d1e1e2cd8fe407b3d524712 で適用されました。

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' ですが、実際にハマりました。追加説明が必要だと思います。

Shogo Kawahara さんが14年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

Shogo Kawahara さんが14年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

修正しました

Rimpei Ogawa さんが14年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

Rimpei Ogawa さんが14年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

Rimpei Ogawa さんが14年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

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 に変更

他の形式にエクスポート: Atom PDF