[AppDB] version: Only display comments section in case version has maintainers

[Christopher Cope]

I understand the premise, but I disagree. A lot of apps that I use don't have maintainers. However, the comments are typically helpful. I believe this should be approached differently.

"André Hentschel" <nerv at dawncrow.de> wrote:

>Am 28.04.2013 18:57, schrieb Rosanne DiMesio:
>> Forced moderation of comments could work for apps with maintainers
>who are doing their job, but most of the spam I recently deleted was in
>unmaintained apps. My suggestion would be to block comments altogether
>on unmaintained apps, not just because of spam, but because of other
>inappropriate things that are not being monitored, such as posting
>links to illegal downloads. As for the maintainers who clearly aren't
>doing their jobs (about 20-25% of the spam I found was in entries with
>maintainers), admins already have the capability of removing them.
>
>
>---
> include/version.php |    9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
>diff --git a/include/version.php b/include/version.php
>index 8f02b24..bf1d7ab 100644
>--- a/include/version.php
>+++ b/include/version.php
>@@ -1180,9 +1180,12 @@ class version {
> 
>         echo note::displayNotesForEntry($this->iVersionId);
> 
>-        // Comments Section
>-        if($this->iVersionId)
>-            Comment::view_app_comments($this->iVersionId);
>+        if(sizeof($aMaintainers)>0)
>+        {
>+            // Display comments section in case version has
>maintainers
>+            if($this->iVersionId)
>+                Comment::view_app_comments($this->iVersionId);
>+        }
>     }
> 
>     public static function lookup_name($versionId)
>-- 
>1.7.10.4
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20130428/92ed4428/attachment.html>