Support » Plugin: VK All in One Expansion Unit » ガイドライン違反について

  • Resolved technote

    (@technote0space)


    WordPressのプラグインは WordPress.org 以外でのホスティングを明確に禁止しています。
    https://developer.wordpress.org/plugins/wordpress-org/detailed-plugin-guidelines/#3-a-stable-version-of-a-plugin-must-be-available-from-its-wordpress-plugin-directory-page
    WordPress.org からダウンロードしたプラグインに 『WordPress.org 以外から配布したものに更新する機能』が実装されていることはガイドラインに違反していると考えています。

    実際全く同様の機能を持ったプラグインがレビューされたときはこれを理由にリジェクトされています。
    レビューを通った後はコードレビューはないに等しいので簡単にこのような機能を埋め込むことはできますが、好ましいことだとは思いません。
    v8.3.2 から導入されているこの WordPress.org 以外でのホスティング機能は削除されるべきだと考えます。

Viewing 8 replies - 1 through 8 (of 8 total)
  • technote

    (@technote0space)

    https://developer.wordpress.org/plugins/wordpress-org/detailed-plugin-guidelines/#8-plugins-may-not-send-executable-code-via-third-party-systems

    このガイドラインにも違反しています。
    早急に対応がされない場合、削除依頼を出さざるを得ません。

    Plugin Author kurudrive

    (@kurudrive)

    確認しました。ご指摘ありがとうございます。
    修正すると共にガイドラインを再度確認させていただきます。

    technote

    (@technote0space)

    @kurudrive
    確認ありがとうございます。
    よろしくお願いいたします。

    Plugin Author kurudrive

    (@kurudrive)

    すみませんあと、

    https://developer.wordpress.org/plugins/wordpress-org/detailed-plugin-guidelines/#8-plugins-may-not-send-executable-code-via-third-party-systems
    
    このガイドラインにも騙しています。早急に対応
    がされない場合、削除依頼を出しますを得ません。

    について、具体的にどのあたりの事でしょうか?
    問題点は確実に修正したいのでご教授いただけると助かります。

    technote

    (@technote0space)

    お世話になっております。

    https://developer.wordpress.org/plugins/wordpress-org/detailed-plugin-guidelines/#3-a-stable-version-of-a-plugin-must-be-available-from-its-wordpress-plugin-directory-page

    > Distributing code via alternate methods, while not keeping the code hosted here up to date, may result in a plugin being removed.

    WordPress.org のコードを最新にせずに他の方法(公式ディレクトリ以外)でコードが配布されている点

    https://developer.wordpress.org/plugins/wordpress-org/detailed-plugin-guidelines/#8-plugins-may-not-send-executable-code-via-third-party-systems

    > Executing outside code within a plugin when not acting as a service is not allowed, for example:
    > Serving updates or otherwise installing plugins, themes, or add-ons from servers other than WordPress.org’s

    プラグイン内で外部のコードが実行(WordPress.org 以外からのプラグインやテーマ、アドオンの更新やインストール)されている点

    レビュアーには上記二点の指摘を報告しましたが、どちらが(もしくはこれ以外かもしれませんが)違反しているかに関してはレビュアーからの返答にはなかったためわかりません。
    またレビュアーから開発者に対しどのような連絡が行われたかなどもわかりません。
    いったんクローズされたのでガイドラインに抵触していたという判断を私はしていますが、もしかしたらSlackなどで直接抗議等を行えば覆る状態なのかもしれません。
    今どのような状態なのかはレビュアーに直接聞いていただく必要があると考えています。

    実際にレビュアーに指摘したコードは以下の個所ですが、前述の通りこの箇所が原因かどうかははっきりとはわかりかねます。

    https://plugins.trac.wordpress.org/browser/vk-all-in-one-expansion-unit/tags/8.3.2/vkExUnit.php#L37

    37	        require 'plugins_admin/plugin-update-checker/plugin-update-checker.php';
    38	        $myUpdateChecker = Puc_v4_Factory::buildUpdateChecker(
    39	                'https://lightning.nagoya/wp-content/vk-data-files/vk-all-in-one-expansion-unit/plugin-update-config.json',
    40	                __FILE__,
    41	                'vk-all-in-one-expansion-unit'
    42	        );

    よろしくお願いいたします。

    • This reply was modified 3 months, 2 weeks ago by  technote.
    • This reply was modified 3 months, 2 weeks ago by  technote.
    • This reply was modified 3 months, 2 weeks ago by  technote.
    Plugin Author kurudrive

    (@kurudrive)

    > @technote0space

    詳細ありがとうございます!

    はい、レビュワーの方からは plugin-update-checker の内容で連絡は受けまして、
    関連箇所については先程削除いたしました。

    https://github.com/vektor-inc/VK-All-in-One-Expansion-Unit/commit/06fcf82e15235a645fa59a17157d1cac6ac75496

    上記ご指摘は2点ですが、VK All in One Expansion Unit ではアップデート機能以外で外部のプログラム実行はしていないので、内容としては UpdateChecker の削除で両方とも解消されるという認識で良いでしょうか?

    まだあらためてガイドラインを読み直してる途中なので、
    他のガイドライン違反が潜んでいるかもしれませんが、
    まずは今回の指摘内容での修正漏れや私の認識間違いがあってはいけないので、
    お手数ですがご回答いただけますと幸いです。

    よろしくお願いいたします。

    • This reply was modified 3 months, 2 weeks ago by  kurudrive.
    technote

    (@technote0space)

    お世話になっております。

    早速の対応ありがとうございます。
    最終的にはレビュワーの判断になるとは思いますが、提示いただいたUpdateCheckerの削除で指摘の事項は解消されると考えています。

    よろしくお願いいたします。

    Plugin Author kurudrive

    (@kurudrive)

    > @technote0space

    ありがとうございます。再公開されました。
    この度はご指摘ありがとうございました!

    現在改めてガイドラインを確認しておりますが、
    また問題点など発見しましたらご指摘いただければ幸いです。

    諸々対応いただきありがとうございました。

Viewing 8 replies - 1 through 8 (of 8 total)
  • You must be logged in to reply to this topic.