vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

公開画面側にプラグイン画面へのリンクを追加しないように変更 #1084

Closed kurudrive closed 5 months ago

kurudrive commented 5 months ago

チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)

https://vws.vektor-inc.co.jp/forums/topic/90184

WordPress 6.5 で公開画面の管理バーにプラグインページへのリンクが表示されるようになった。 その都合上 ExUnit が追加するプラグインへのリンクと合わせて二重で表示されるようになってしまった。

スクリーンショット 2024-04-18 1 48 06

どういう変更をしたか?

ソースコードについて

デザイン・UI

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。 書いていない場合は書かない理由を記載してください。

その他

変更内容について何を確認したか、どういう方法で確認をしたかなど


確認URL

ローカルでよろしくお願いいたします。

レビュワーの確認方法・確認する内容など

実装者に同じ

レビュワーに回す前の確認事項


レビュワー向け

確認して変更が反映されていない場合の確認事項

drill-lancer commented 5 months ago

@kurudrive 動作自体は問題ありませんでした。 ・・・が、そもそも 6.5 以上の場合は

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/blob/1bbd68993c87cf3777d1994a9e2fedf83e6e3ffc/veu-packages.php#L197-L206

の部分を処理しないようにするほうがスマートな気がします。

kurudrive commented 5 months ago

@drill-lancer 解説しましょう(・w・ 6.5 標準機能において、管理画面側には プラグイン のリンクは出ないんですよ。 なので、カスタム投稿タイプとか沢山作ってて、通常の プラグイン のメニューの位置がかなり下の場合に限って、 管理画面の管理バーにプラグインのリンクがあるのが便利に思うケースも存在しうるので、まだ残しているという状態デス。