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

プラグインを無効にして再度インストールするとアクセスルール設定が重複して登録される #3990

Closed
1 task
ryuring opened this issue Nov 7, 2024 · 5 comments
Labels
Bug バグ

Comments

@ryuring
Copy link
Collaborator

ryuring commented Nov 7, 2024

概要

例えば、ブログプラグインを無効化して、再度インストールした場合、アクセスルールグループより、ブログ記事の設定編集画面に移動すると、設定が重複して登録されている。

baserCMS version : 5.1.3

TODO

  • インストールの際、pluginsテーブルに登録されていない場合のみアクセスルールグループを登録する処理に変更する(詳細仕様未確認)
@ryuring ryuring added the Bug バグ label Nov 7, 2024
@cu-akasakay
Copy link
Contributor

@ryuring
こちら最新verでも発生するので未対応のままでしょうか。
であれば、私のほうで対応します。

@ryuring
Copy link
Collaborator Author

ryuring commented Feb 27, 2025

@cu-akasakay ありがとうございます、アクセスルールを登録するのは、最初のインストールの際だけにして、再有効化の際は登録しないようにする感じですかね。
よろしくお願いします🤲

@cu-akasakay
Copy link
Contributor

@ryuring
有効化の場合はアクセスルールを作成しない、という修正にしようと思いましたが、
有効化する際に全てのユーザーか管理ユーザーのみか選択できるので考慮が必要そうです。
例えば、管理ユーザーのみでインストール後に無効化して、全てのユーザーで再有効化したときに
アクセスルールが作成されません。

修正案としては作成直前でpermission_group_id、user_group_id、urlをキーに存在確認をして、
あれば作成しない、とかでしょうか。

現状は全てのユーザー -> 管理ユーザーのみに変更してもアクセスルールが削除されていないので
そこも対応(または仕様とするか)しないとどちらにせよ片手落ちな気がしますが・・・。

@ryuring
Copy link
Collaborator Author

ryuring commented Feb 27, 2025

@cu-akasakay という事は、アンインストール時(無効化時)に、対象データを削除するというのが一番キレイな形になるかもですね。
それはいけそうですか?

@cu-akasakay
Copy link
Contributor

@ryuring
確かに無効化と同時にアクセスルールも削除するのであれば動きとしては問題ないですね。
無効化(detach)側を修正してみます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug バグ
Projects
None yet
Development

No branches or pull requests

2 participants