Project

General

Profile

Bug(バグ) #1377

ActivityData::getBody() is too bad in template. ( ActivityData::getBody() はテンプレートで適切でない挙動をする )

Added by Shogo Kawahara about 9 years ago. Updated almost 2 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-07-20
Due date:
% Done:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

Overview (概要)

ActivityData::getBody() is too bad in template. ( ActivityData::getBody() はテンプレートで良くない挙動をする)

  • When body is re-wrote by "op_activity.filter_body", default of ActivityDataForm is re-wroted too.
  • Body is double-escaped by filter.
  • 本文が op_activity.filter_body によって書き換えられるとき、ActivityDataFormのデフォルト値も書き換えられます。
  • フィルタによって本文がダブルエスケープされることがあります。

Causes (原因)

  • Default of sfDoctrineForm depend on Doctrine_Record::get*.
  • In template, Doctrine_Record is comprehended sfOutputEscaperObjectDecorator.

Way to fix (修正方法)

  • To prevent double-escaping, filter move helper.

opSamplePlugin.zip (745 Bytes) Shogo Kawahara, 2010-07-27 19:28


Related issues

Related to OpenPNE 3 - Backport(バックポート) #1410: ActivityData::getBody() is too bad in template. ( ActivityData::getBody() はテンプレートで適切でない挙動をする ) Fixed(完了) 2010-07-20

Associated revisions

Revision 1a668644 (diff)
Added by Shogo Kawahara about 9 years ago

removed ActivityData::getBody() and added op_activity_body_filter() of helper (fixes #1377)

Revision 600546fd (diff)
Added by Shogo Kawahara about 9 years ago

fixed a test script for Activity (fixes #1377)

History

#1 Updated by Shogo Kawahara about 9 years ago

  • Assignee set to Shogo Kawahara
  • Target version set to OpenPNE 3.7.0

#2 Updated by Shogo Kawahara about 9 years ago

  • Subject changed from ActivityData::getBody() is too bad in template. ( ActivityData::getBody() はテンプレートで良くない挙動をする) to ActivityData::getBody() is too bad in template. ( ActivityData::getBody() はテンプレートで適切でない挙動をする )

#3 Updated by Shogo Kawahara about 9 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Shogo Kawahara about 9 years ago

検証用プラグイン(ユーザ側)

  • このプラグインを導入して、アクティビティの表示が太字になることを確認してください。

#5 Updated by Shogo Kawahara about 9 years ago

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

#6 Updated by Rimpei Ogawa almost 9 years ago

  • 3.6 で発生するか set to Yes

#7 Updated by Kousuke Ebihara almost 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#8 Updated by Fumie Toyooka almost 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

Also available in: Atom PDF