ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#2634 closed defect (fixed)

OPENPNE_URL にポート番号が含まれている場合、PC版URLを携帯版URLに変換する機能が動作しない

Reported by: ebihara Owned by: ebihara
Priority: major Milestone: OpenPNE2.12.1
Component: 指定しない Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.7 OpenPNE2.13.1 Cc:

Description (last modified by kudo)

■現象

OPENPNE_URL にポート番号が含まれている場合、t_url2a_ktai の変換機能が動作しない。

■原因

正規表現によるURLのマッチングにおいて、ポート番号を考慮していない。

■修正内容

正規表現によるURLのマッチングをおこなう前に OPENPNE_URL もしくは OPENPNE_SSL_URL ではじまるURLかどうかをチェックする。

■関連情報

Change History (18)

comment:1 Changed 11 years ago by ebihara

Description: modified (diff)

comment:2 Changed 11 years ago by ebihara

Summary: OPENPNE_URL にポート番号が含まれている場合、t_url2a_ktai と t_url2cmd(SNS内のURLのみ)の変換機能が動作しないOPENPNE_URL にポート番号が含まれている場合、PC版URLを携帯版URLに変換する機能とアルバム・アルバム画像URLの小窓変換機能が動作しない

comment:3 Changed 11 years ago by ebihara

Description: modified (diff)

comment:4 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.7 added
Summary: OPENPNE_URL にポート番号が含まれている場合、PC版URLを携帯版URLに変換する機能とアルバム・アルバム画像URLの小窓変換機能が動作しないOPENPNE_URL にポート番号が含まれている場合、PC版URLを携帯版URLに変換する機能が動作しない
Version: 2.12.x & 2.13.x2.10.x & 2.12.x & 2.13.x

t_url2a_ktai 内のポート番号を考慮していないコードは OpenPNE-2.10.x にも存在するので、チケットを分けて対処することにしましょう。

comment:5 Changed 11 years ago by ogawa

Type: enhancementdefect

comment:6 Changed 11 years ago by ebihara

Priority: minormajor

comment:7 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

comment:8 Changed 11 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:9 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed
  • OPENPNE_USE_PARTIAL_SSL が false の場合は、OPENPNE_SSL_URL は見ないようにする
  • (%s|%s) の部分のパフォーマンスが悪そうなので改善したい

comment:10 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

comment:11 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

$host が定義されないまま使われています。

comment:12 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:13 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

PCでSSLだった場合に携帯でもSSLであるとは限らないので、$opnepne_url固定なのはあまりよくないと思います。

comment:14 Changed 11 years ago by ebihara

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:15 Changed 11 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:16 Changed 11 years ago by ebihara

Description: modified (diff)

comment:17 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題ありません。

comment:18 Changed 10 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.