Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【ブログ】記事の一括削除を行った際にアイキャッチが削除されない #3004 #3007

Merged
merged 3 commits into from
Jan 6, 2024

Conversation

seto1
Copy link
Collaborator

@seto1 seto1 commented Jan 4, 2024

fix #3004

記事の一括削除時は plugins/bc-blog/src/Controller/Admin/BlogPostsController->beforeFilter 内の

$service->setupUpload($blogContentId);

が呼び出されず、ファイルのパスの生成がうまくいかずにファイルの削除に失敗しているようでしたので調整しました。

ご確認お願いします。

@ryuring
Copy link
Collaborator

ryuring commented Jan 4, 2024

@seto1 テストがこけているようですので一度確認いただいてもいいですか?

@ryuring ryuring added the Reviewed レビュー済 label Jan 4, 2024
@seto1
Copy link
Collaborator Author

seto1 commented Jan 5, 2024

@ryuring 修正しましたのでご確認お願いします。
setupUpload実行時はブログコンテンツのデータも必要でしたので対応しました。

@ryuring
Copy link
Collaborator

ryuring commented Jan 6, 2024

@seto1 ありがとうございます!

@ryuring ryuring merged commit 71e3897 into baserproject:dev-5 Jan 6, 2024
1 check passed
@momofff momofff added this to the 5.0.8 milestone Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed レビュー済
Projects
None yet
Development

Successfully merging this pull request may close these issues.

【ブログ】記事の一括削除を行った際にアイキャッチが削除されない
3 participants