Project

General

Profile

Bug(バグ) #4032

リッチテキストエリア設定でボタンの並び替えが機能しなくなっている

Added by Youichi Kimura almost 4 years ago. Updated 3 months ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2020-01-31
Due date:
% Done:

100%

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

Description

Overview (現象)

管理画面のリッチテキストエリア設定 (/pc_backend.php/advanced/richTextarea) で、「ボタンの設定」に表示されている各ボタンの順序をマウスドラッグで変更する機能が動作しなくなっている。

Causes (原因)

#3010 にてリッチテキストエリア設定を「上級者向け設定」に移動した際に、view.yml の設定や url_for の URL の修正が漏れていたことが原因。

source:apps/pc_backend/modules/sns/config/view.yml@6e121793:

richTextareaSuccess:
  javascripts:
    - jquery.min.js
    - jquery-ui.min.js

source:apps/pc_backend/modules/advanced/templates/richTextareaSuccess.php@6e121793#L45:

    $.ajax({
      url: "'.url_for('sns/changeRichTextareaButtonOrder').'",
      type: "POST",
      data: postData
    });

Way to fix (修正内容)

view.ymlapps/pc_backend/modules/advanced/config/view.yml に移動し、url_for を正しい URL に書き換える。


Subtasks

Backport(バックポート) #4362: リッチテキストエリア設定でボタンの並び替えが機能しなくなっているFixed(完了)kaoru n

Backport(バックポート) #4380: リッチテキストエリア設定でボタンの並び替えが機能しなくなっているFixed(完了)isao sano

History

#1 Updated by Youichi Kimura almost 4 years ago

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

下記 Pull Request にて修正しました
https://github.com/openpne/OpenPNE3/pull/381

#2 Updated by kaoru n over 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#3 Updated by kaoru n 8 months ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x

#4 Updated by kaoru n 8 months ago

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

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

https://github.com/openpne/OpenPNE3/pull/381
については、old-master 向けであるためクローズしています

#5 Updated by kaoru n 8 months ago

  • Assignee changed from Youichi Kimura to kaoru n

#6 Updated by Rimpei Ogawa 4 months ago

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

#7 Updated by isao sano 3 months ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

試験完了しました。
問題ありません。

#8 Updated by kaoru n 3 months ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF