fix #1903 コンテンツ一覧（表）コピーの際にコピー先のタイトルに「のコピー」とつかない #2779
Conversation
|
コンテンツ管理での処理には矛盾があり、だから今回の問題になっていると思います。
ただ、コピーの仕様について、コンテンツ管理で管理されているエンティティ以外のエンティティについてどういう処理になっているか調べたところ、基本的にはテーブルで処理されていました。 コピーの仕様としてはこちらの方が正しいと思います。 また、今回、formタグの中を変更する修正となっていますが、このformタグが将来的にコピー以外にも利用される可能性があるため、こちらの変更は望ましくありません。 その上で考慮した結果、新しい名前の指定がある場合には、そのタイトルを利用し、ない場合には「_copy」を付与する仕様にして欲しいです。 このようにする場合、以下のテーブルの変更が必要となるため、修正が大変ですが、よろしくお願いします。
|
|
@HungDV2022 ちなみに、「新しい名前の指定がある場合には、そのタイトルを利用し、ない場合には「_copy」を付与する仕様」にするのは、コンテンツ管理で管理するエンティティだけで大丈夫です。BlogPosts や、MailFields は今回対象外にしてください。 |
|
CustomContentsTable以外は修正しました。 CustomContentsTableはコピー機能がないので対応しないです。 |
| @@ -220,7 +220,7 @@ class="bca-btn-icon" | |||
| <input type="hidden" name="type" value="<?= $content->type ?>"> | |||
| <input type="hidden" name="entity_id" value="<?= $content->entity_id ?>"> | |||
| <input type="hidden" name="parent_id" value="<?= $content->parent_id ?>"> | |||
| <input type="hidden" name="title" value="<?= h($content->title) ?>"> | |||
| <input type="hidden" name="title" value="<?= h($content->title.' のコピー') ?>"> | |||
There was a problem hiding this comment.
@HungDV2022 こちらは元に戻してください。
There was a problem hiding this comment.
@ryuring もっとに戻ったらタイトルに「のコピー」を付けなくてもいいですか。
There was a problem hiding this comment.
また、今回、formタグの中を変更する修正となっていますが、このformタグが将来的にコピー以外にも利用される可能性があるため、こちらの変更は望ましくありません。
その上で考慮した結果、新しい名前の指定がある場合には、そのタイトルを利用し、ない場合には「_copy」を付与する仕様にして欲しいです。
こちらに記載しているとおり、テーブル側で付ける仕様を想定しています。
There was a problem hiding this comment.
こちらを修正しました。確認お願いします。 @ryuring
| @@ -45,7 +45,7 @@ | |||
| 'bcTreeAlertMessage6' => __d('baser_core', '追加に失敗しました。'), | |||
| 'bcTreeInfoMessage1' => __d('baser_core', 'ゴミ箱は空です'), | |||
| 'bcTreeInfoMessage2' => __d('baser_core', 'ゴミ箱より戻しました。一覧に遷移しますのでしばらくお待ち下さい。'), | |||
| 'bcTreeCopyTitle' => __d('baser_core', '%s のコピー'), | |||
| 'bcTreeCopyTitle' => __d('baser_core', '%s'), | |||
There was a problem hiding this comment.
@HungDV2022 この行はどこでも使われていないのであれば削除してください。というか意味がないですよね。
There was a problem hiding this comment.
@HungDV2022 javascript側も調整お願いします。
There was a problem hiding this comment.
@ryuring この行を削除しました。
JS側はこちらに調整しました。
plugins/bc-admin-third/src/js/admin/_lib/jquery.bcTree.js
確認お願いします。
|
@HungDV2022 ありがとうございました。マージしますね。 |
No description provided.