プロジェクト

全般

プロフィール

Bug(バグ) #2497

未完了

opWidgetFormRichTextareaを利用するフォームにおいてHTML特殊文字を実体参照で入力すると変換されてしまう

Maki Takahashi さんが約13年前に追加. ほぼ5年前に更新.

ステータス:
Pending Fixing(修正待ち)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
2011-10-14
期日:
進捗率:

0%

予定工数:
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

Overview (現象)

opWidgetFormRichTextareaおよび、opWidgetFormRichTextareaを継承するwidgetを利用するフォームにおいて
HTML特殊文字(例:©)を実体参照( © )で入力すると変換されてしまう。

Causes (原因)

sfWidget::escapeOnce() から呼ばれる sfWidget::fixDoubleEscape() が原因。

  static public function escapeOnce($value)
  {
    return self::fixDoubleEscape(htmlspecialchars((string) $value, ENT_QUOTES, self::getCharset()));
  }

  static public function fixDoubleEscape($escaped)
  {
    return preg_replace('/&([a-z]+|(#\d+)|(#x[\da-f]+));/i', '&$1;', $escaped);
  }

確認バージョン

OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
OpenPNE 3.4.17 (stable-3.4.x)


関連するチケット 3 (3件未完了0件完了)

関連している OpenPNE 3 - Bug(バグ) #2407: 管理画面ガジェット設定で&などのエスケープ文字を入力すると元の文字で表示されるNew(新規)2011-09-17

操作
コピー先 OpenPNE 3 - Backport(バックポート) #3641: opWidgetFormRichTextareaを利用するフォームにおいてHTML特殊文字を実体参照で入力すると変換されてしまうNew(新規)2011-10-14

操作
コピー先 OpenPNE 3 - Backport(バックポート) #3665: opWidgetFormRichTextareaを利用するフォームにおいてHTML特殊文字を実体参照で入力すると変換されてしまうNew(新規)2014-07-15

操作

Yuya Watanabe さんが約13年前に更新

詳しく調査していませんが, #2407 の現象と類似していると思ったので関連するチケットに追加しておきます.

Maki Takahashi さんが約13年前に更新

  • 説明 を更新 (差分)
  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット

Maki Takahashi さんが約13年前に更新

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

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

Maki Takahashi さんが約13年前に更新

_partsForm.phpを使わない管理画面では変換されないようになりますが、_partsForm.phpを利用しているfrontend側は修正されません。

Shouta Kashiwagi さんがほぼ13年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

Shouta Kashiwagi さんが12年以上前に更新

  • 対象バージョン252 から OpenPNE 3.8.x に変更

Shinichi Urabe さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.12 に変更
  • 3.6 で発生するかUnknown (未調査) にセット
  • 3.8 で発生するかUnknown (未調査) にセット

誠二 天重 さんが10年以上前に更新

  • コピー先 Backport(バックポート) #3641: opWidgetFormRichTextareaを利用するフォームにおいてHTML特殊文字を実体参照で入力すると変換されてしまう を追加

誠二 天重 さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.8.12 から OpenPNE 3.8.x に変更

isao sano さんが10年以上前に更新

  • コピー先 Backport(バックポート) #3665: opWidgetFormRichTextareaを利用するフォームにおいてHTML特殊文字を実体参照で入力すると変換されてしまう を追加

Shinichi Urabe さんが10年以上前に更新

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

PHPDoc の説明を日本語で書いている箇所がコア側に関して今回追加された箇所以外、一部ありますが、他は見当たらないです.
MemberConfigForm にも存在しますが、、、
統一させるためにも他とあわせておいた方がよいかと思います

$ ag "\* [\x20-\x7e]+?[^\x01-\x7E]+?" --ignore-dir=vendor --ignore-dir=web --ignore-dir=plugins -l |grep php$
lib/form/doctrine/MemberConfigForm.class.php
lib/widget/opWidgetFormRichTextarea.class.php
test/unit/helper/opUtilHelperTest.php

Shinichi Urabe さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更

Maki Takahashi さんが10年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

Maki Takahashi さんが10年以上前に更新

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

https://github.com/openpne/OpenPNE3/pull/178 にて
説明を英語に書き換えプルリクエストしました。

MemberConfigFormも高橋の仕業のようです…

Shinichi Urabe さんが10年以上前に更新

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

レビューOKです

Shinichi Urabe さんが10年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Fixing(修正待ち) に変更
  • 担当者 を削除 (Maki Takahashi)
  • 進捗率70 から 0 に変更

本件について、opWidgetFormRichTextarea 限定で発生する問題ではなく、他の widget においても起こりえる問題
opWidgetFormRichTextarea について限定しているのであれば、修正内容は問題無いが、
他の widget の挙動についてもあわせて対応方針を検討した方がよいかと考えられる

一旦、取り込みは保留とします

isao sano さんが7年以上前に更新

対象バージョン変更のため、修正内容の確認を行います。

isao sano さんが7年以上前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

isao sano さんが7年以上前に更新

再現します。

kaoru n さんがほぼ5年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE 3.10.x に変更

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