プロジェクト

全般

プロフィール

Backport(バックポート) #1439

#975の修正で正規表現のマッチングが不十分である

Mutsumi Imamura13年以上前に追加. 13年以上前に更新.

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

100%


説明

現象

#1227のテスト中に下記の再現方法のように文章を入力したところ、文字装飾タグの補完が行われずにフッターなどに文字粧飾が反映されてしまう。

再現環境

発生バージョン
  • OpenPNE3.2.7-dev
  • OpenPNE3.4.6-dev
  • OpenPNE3.5.2-dev
再現ブラウザ
  • IE8
  • IE7(IE Tester)
  • IE6(IE Tester)

再現方法

日記本文に

<op:b>てててて:b>
<op:u>てててて</u>
<op:s>てててて</p:s>
<op:i>てててて<op:i
<op:font size="5">てててて</opnt>
<op:font size="1">てててて</oont>
<op:font color="#FF0000">ててててp:font>
てててて

などと入力し日記を作成する。

作成した日記をIEで確認する。

原因

#975での修正で追加された正規表現のマッチングに漏れがある

修正内容


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1438: #975の修正で正規表現のマッチングが不十分である Fixed(完了) 2010-07-27
関連している OpenPNE 3 - Bug(バグ) #975: 日記作成時に文字装飾の閉じタグがない場合、IEでフッターなどにも文字装飾が反映されてしまう Fixed(完了) 2010-04-06
関連している OpenPNE 3 - Backport(バックポート) #1452: 壊れたopタグの後ろに改行が含まれると意図しないタグが作成される Fixed(完了) 2010-07-29

関係しているリビジョン

リビジョン cbd534a4 (差分)
Shinichi Urabe13年以上前に追加

(fixes #1439, BP from #1438) add html tag followup method.

リビジョン b6c85bd5 (差分)
Shinichi Urabe13年以上前に追加

Dealing with variables not defined.Fixed regular expression. (fixes #1439, BP from #1438)

履歴

#1 Shinichi Urabe13年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#1227 の件をこちらで対応します

#2 Shinichi Urabe13年以上前に更新

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

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

#3 Rimpei Ogawa13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 正規表現の (?:<) (?:>) の括弧と修飾子の i が不要です
  • foreach ($countStartTags as $k => $v) の中で、 $countEndTags[$k] が定義されていない場合に Notice となります。また、未定義かどうかを検証した場合には int へのキャストも不要です
  • Description にある再現方法の例で、改行前の <op:i が正しく変換されないのは今回の変更点とは関係ないですが別のバグとして修正の必要があるでしょう

#4 Shinichi Urabe13年以上前に更新

壊れたopタグの後に改行を含む状態で正しく変換されない問題は #1452 で修正します

#5 Shinichi Urabe13年以上前に更新

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

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

#6 Rimpei Ogawa13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#7 Mutsumi Imamura13年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

確認しました。テストOKです。

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