[Bug 43702] New: No comments shown if no maintainers for app

https://bugs.winehq.org/show_bug.cgi?id=43702

Bug ID: 43702
           Summary: No comments shown if no maintainers for app
           Product: WineHQ Apps Database
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: appdb-unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

If an app has no maintainers, it shows "comments disabled", and one cannot
enter new comments. This is useful, because no one would moderate them.

However, it would still be useful to see the backlog of old comments, which may
contain hints and workarounds posted by users (when there still were
maintainers).

[Bug 43702] No comments shown if no maintainers for app

https://bugs.winehq.org/show_bug.cgi?id=43702

--- Comment #1 from Austin English  ---
Some links to sample applications would be useful.

[Bug 43702] No comments shown if no maintainers for app

https://bugs.winehq.org/show_bug.cgi?id=43702

--- Comment #2 from azrdev  ---
dunno if there had been any comments previously, but randomly picking any app
without maintainer yields
https://appdb.winehq.org/objectManager.php?sClass=application&iId=2

[Bug 43702] No comments shown if no maintainers for app

https://bugs.winehq.org/show_bug.cgi?id=43702

Joerg Schiermeier  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #3 from Joerg Schiermeier  ---
If there are workarounds wine AppDB is a nice place to show how the application
comes to life in wine. Maybe someone will add this as a hint in Bugzillas
howto.

I recommend to close this bug as WONTFIX.

[Bug 43702] No comments shown if no maintainers for app

https://bugs.winehq.org/show_bug.cgi?id=43702

--- Comment #4 from Joerg Schiermeier  ---
> Maybe someone will add this as a hint in Bugzillas howto.

And also in the Maintainer guide.

[Bug 43702] No comments shown if no maintainers for app

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=43702

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW

--- Comment #5 from Ken Sharp  ---
Confirming. And surprising.

[Bug 43702] No comments shown if no maintainers for app

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=43702

Stian Low  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #6 from Stian Low  ---
(In reply to Ken Sharp from comment #5)
> Confirming. And surprising.

The condition that controls this in it's current form was introduced by commit:

1868edfb61cda174f9c3a364ab1071d8ebf6a081
CommitDate: Tue May 30 14:30:12 2017 -0500

However, the conditional logic seems to have existed prior stretching back to
the beginning of commits.

See the second commit for this merge request for the solution described there:
https://gitlab.winehq.org/winehq/appdb/-/merge_requests/24

[Bug 43702] No comments shown if no maintainers for app

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=43702

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |58517