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

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#1505 closed defect (fixed)

サイドバナーで<cmd>形式の小窓が動作していない

Reported by: kiwa Owned by: ebihara
Priority: critical Milestone: OpenPNE2.10.2
Component: pne-cmd Version: 2.8.x & 2.10.x & 2.11.x
Keywords: OpenPNE2.8.7 OpenPNE2.11.2 Cc:

Description (last modified by ebihara)

■現象

サイドバナーにCMDが表示されない(URLCMDは表示される)

■原因

smarty:nodefaultsが有効な箇所ではt_cmdの以下の正規表現にマッチしなくなるのが原因。

$regexp = '/&lt;cmd\s+src=&quot;(\w+)&quot;(?:\s+args=&quot;([a-zA-Z0-9_\-+%]+(,[a-zA-Z0-9_\-+%]+)*)?&quot;)?\s*&gt;/i';

■修正内容

smarty:nodefaultsが有効な箇所でもマッチするよう、HTMLエンティティに変換されたものと変換されていないものの両方をチェックするように変更。

$regexp = '/(&lt;|<)cmd\s+src=(&quot;|"|\')(\w+)(&quot;|"|\')(?:\s+args=(&quot;|"|\')([a-zA-Z0-9_\-+%]+(,[a-zA-Z0-9_\-+%]+)*)(&quot;|"|\'))?\s*(&gt;|>)/i'; 

■関連情報

Change History (15)

comment:1 Changed 11 years ago by kiwa

Priority: minorcritical

comment:2 Changed 11 years ago by kiwa

Milestone: OpenPNE2.10.2

comment:3 Changed 11 years ago by kiwa

Description: modified (diff)
Keywords: OpenPNE2.8.7 added

comment:4 Changed 11 years ago by ogawa

Summary: サイドバナーで小窓が動作していないサイドバナーで<cmd>形式の小窓が動作していない

comment:5 Changed 11 years ago by yamaji

Owner: changed from nobody to yamaji
Status: newassigned

バグ取り始めます

comment:6 Changed 11 years ago by kunitada

Owner: changed from yamaji to nobody
Status: assignednew

すいません、手放します。

comment:7 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:8 Changed 11 years ago by ebihara

Description: modified (diff)
Keywords: 確認中 added
Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンで修正しました。smarty:nodefaultsが有効な箇所ではt_cmdの正規表現にマッチしなくなるのが原因でした。

  • 2.8.x - r4681
  • 2.10.x - r4680
  • trunk - r4684 (なぜか取り消されているr4331の変更も取り込みました)

Descriptionの「■原因」「■修正内容」を更新しました。

ご確認ください。

comment:9 Changed 11 years ago by ogawa

Keywords: 確認中 removed
Owner: changed from ogawa to ebihara

&quot;|"|\'

だと中途半端です。

comment:10 Changed 11 years ago by ebihara

Status: newassigned

既存のコードにあわせて、ダブルクォーテーションのみの対応とします。

comment:11 Changed 11 years ago by ebihara

Keywords: 確認中 added
Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンで修正しました。

comment:12 Changed 11 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed
Owner: changed from ogawa to ebihara

テスト待ち。

comment:13 Changed 11 years ago by kiwa

Resolution: fixed
Status: newclosed

動作テストしました。OKです。

comment:14 Changed 11 years ago by kiwa

Keywords: テスト待ち removed

comment:15 Changed 10 years ago by kiwa

Keywords: OpenPNE2.11.2 added
Note: See TracTickets for help on using tickets.