Project

General

Profile

Bug(バグ) #3413

管理画面のhtml挿入で,テキストモードからプレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える場合がある

Added by 誠二 天重 about 6 years ago. Updated over 2 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2013-09-30
Due date:
% Done:

50%

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

Description

概要

管理画面のhtml挿入で,テキストモードで例のようなソースを入力し,プレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える。


<span><script src="http://hoge.com"></script></span>
<span><a href="/"></a></span>

現象が発生する条件としては,spanタグやdivタグ,pタグなどの中で,プレビュー時に出力内容を持たないようなソースを入力した場合に発生する。

spanではなくdivでくくった場合は,入力内容が

<div>&nbsp;</div>
になる。

再現手順

  1. 管理画面のhtml挿入画面に遷移
  2. テキストモードのhtml挿入テキストエリアに<span><script src="http://hoge.com"></script></span>と入力する
  3. プレビューモード切替用ラジオボタンにチェックを入れる
  4. テキストモード切替用ラジオボタンにチェックを入れる
    ->手順2の入力内容が消えている
    ※srcで指定するurlが実在しているかどうかは本現象にかかわらない。

Related issues

Related to OpenPNE 3 - Bug(バグ) #2495: HTML挿入で テキストモード→プレビューモード→テキストモード の切り替えを行うとリンクの遷移先の指定が変更されてしまう場合がある Won't fix(対応せず) 2011-10-14
Related to OpenPNE 3 - Bug(バグ) #2496: HTML挿入で テキストモード→プレビューモード→テキストモード の切り替えを行うと <hr> の color 属性が消える Rejected(差し戻し)
Related to OpenPNE 3 - Backport(バックポート) #3861: 管理画面のhtml挿入で,テキストモードからプレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える場合がある Rejected(差し戻し) 2015-11-09
Related to OpenPNE 3 - Backport(バックポート) #3862: 管理画面のhtml挿入で,テキストモードからプレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える場合がある Rejected(差し戻し) 2015-11-09
Related to OpenPNE 3 - Task(タスク) #4071: OpenPNEに同梱されている TinyMCE 3.4.4 が IE11 に対応していないため、バージョンアップについて調査する New(新規) 2016-12-20

History

#1 Updated by 誠二 天重 about 6 years ago

  • Description updated (diff)

#2 Updated by Akihiro KOBAYASHI about 5 years ago

  • Related to Bug(バグ) #2495: HTML挿入で テキストモード→プレビューモード→テキストモード の切り替えを行うとリンクの遷移先の指定が変更されてしまう場合がある added

#3 Updated by Akihiro KOBAYASHI about 5 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Akihiro KOBAYASHI

ここら辺を使って調査してみます
http://www.560designs.com/memo/839.html

#4 Updated by Akihiro KOBAYASHI about 5 years ago

原因:TinyMCEの自動整形機能によって空タグ,brタグ等が自動削除されるようです.
よって、lib/widget/opWidgetFormRichTextarea.class.phpを
http://www.tinymce.com/wiki.php/Configuration3x
等参考に編集してみます.

#5 Updated by Akihiro KOBAYASHI about 5 years ago

lib/widget/opWidgetFormRichTextarea.class.phpにおいて
protected $tinyMCEConfigs = array( ...内に

'valid_elements' => 'script[src]',

を追加したところ、
<span><script src="http://hoge.com"></script></span>

<script src="http://hoge.com" type="text/javascript"></script>
と変換され、値が残るようになりました

これは属性値srcがある場合のみ値が保持されるためです

これに基づいて
'valid_elements' => '*[*]',
と変更する事で対処してみて、どのようになるか検証します

引き続き調査します

#6 Updated by Akihiro KOBAYASHI about 5 years ago

  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

#7 Updated by Akihiro KOBAYASHI about 5 years ago

  • Related to Bug(バグ) #2496: HTML挿入で テキストモード→プレビューモード→テキストモード の切り替えを行うと <hr> の color 属性が消える added

#8 Updated by Akihiro KOBAYASHI about 5 years ago

  • Target version set to OpenPNE 3.9.0-old

lib/widget/opWidgetFormRichTextarea.class.php内に
tinymce.init{
...
'valid_elements' => '*[*]',
'extended_valid_elements' => '*[*]',
'convert_fonts_to_spans' => false,
'force_p_newlines' => false,
'forced_root_block' => '',
'remove_linebreaks' => false,
'preformatted' => true,
'apply_source_formatting' => false

を追加することで

<span><script src="http://hoge.com"></script></span>
<span><a href="/"></a></span>

の入力内容の保持に成功しました
ただ、

<div><script src="http://hoge.com"></script></div>
<div><a href="/"></a></div>

とした際に、

<div></div><div></div>

となり、div内の内容がすべて消えてしまうので、まだ調査が必要です

#9 Updated by Akihiro KOBAYASHI about 5 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

プルリクエストしました
https://github.com/openpne/OpenPNE3/pull/234

原因
tinymceの自動整形機能により、勝手にタグを削除してしまったりしている

対策
tinymceの設定を以下のように変更
valid_children' => 'div[*], ',
'valid_elements' => '
[*]',
'extended_valid_elements' => '*[*]',
'convert_fonts_to_spans' => false,
'force_p_newlines' => false,
'forced_root_block' => '',
'remove_linebreaks' => false,
'preformatted' => true,
'apply_source_formatting' => false,

valid_childrenについては*[*]では対応できなかったものがあったため(<div><script src="http://hoge.com"></script></div><div><a href="/"></a></div> → <div></div<div></div)になってしまう)、特別にdiv[*]と明記して対処した

#10 Updated by kaoru n over 4 years ago

  • Assignee deleted (Akihiro KOBAYASHI)

#11 Updated by isao sano about 4 years ago

  • Related to Backport(バックポート) #3861: 管理画面のhtml挿入で,テキストモードからプレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える場合がある added

#12 Updated by isao sano about 4 years ago

  • Related to Backport(バックポート) #3862: 管理画面のhtml挿入で,テキストモードからプレビューモードに切り替えた後,テキストモードに戻った時に入力内容が消える場合がある added

#13 Updated by kaoru n over 3 years ago

  • Assignee set to isao sano

#14 Updated by Youichi Kimura over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

'valid_elements' => '*[*]' について、同じ変更が含まれている #2496 が影響範囲の大きさなどの理由で差し戻しとなっているため解決するまで当チケットも差し戻しの状態にします。

#16 Updated by kaoru n almost 3 years ago

  • Related to Task(タスク) #4071: OpenPNEに同梱されている TinyMCE 3.4.4 が IE11 に対応していないため、バージョンアップについて調査する added

#17 Updated by isao sano over 2 years ago

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

#18 Updated by isao sano over 2 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

Also available in: Atom PDF