ファイルダウンロードの認証をトピック表示権限の確認に変更する

[tsukamoto]

■現象

ファイルダウンロードの認証がトピックの表示権限があるかどうかでおこなわれていない。

公開コミュニティでは、非コミュニティメンバでもトピックを閲覧でき、添付ファイルへのリンクも表示される。

ダウンロード権限は、ダウンロードリンクを閲覧できる権限と合っているべきだと思う。また添付ファイルを含めてトピックコンテンツであることを考えれば、「リンクをコミュニティメンバ以外に表示しない」という解決より、「トピックを閲覧できる人は添付ファイルもダウンロードできる」という解決の方が自然だと思われる。

■原因

ファイルダウンロードの認証がコミュニティメンバーかどうかでおこなわれている。

■修正内容

webapp/pc/do/c_file_downloadで行っている認証を、コミュニティメンバか否かの判定から、トピックの表示権限の判定に変更する。

■関連情報

2.10.1ベースでのdiff

Index: webapp/modules/pc/do/c_file_download.php
===================================================================
--- webapp/modules/pc/do/c_file_download.php    (revision 224)
+++ webapp/modules/pc/do/c_file_download.php    (working copy)
@@ -29,7 +29,7 @@
         $c_commu_id = $c_topic['c_commu_id'];

         $status = db_common_commu_status($u, $c_commu_id);
-        if (!$status['is_commu_member']) {
+        if (!$status['is_bbs_view']) {
             handle_kengen_error();
         }

[tejimaa]

私も、ご指摘の方が自然だと思います。 マイルストン変更しちゃいました。

実装してくれるかな。

[ogawa]

安定版に修正入れたいと思います。

[ebihara]

やります

[ebihara]

以下でパッチを取り込みました。ご確認ください。

• 2.10.x - r4791
• trunk - r4792

[kiwa]

安定版のみ確認しました。OKです。