[Bug 58521] New: "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

            Bug ID: 58521
           Summary: "View Distributions" shows 0 test results
           Product: WineHQ Apps Database
           Version: unspecified
          Hardware: x86-64
               URL: https://appdb.winehq.org/objectManager.php?sClass=dist
                    ribution&sTitle=View%20Distributions
                OS: Linux
            Status: NEW
          Keywords: download, source
          Severity: normal
          Priority: P2
         Component: appdb-unknown
          Assignee: wine-bugs(a)winehq.org
          Reporter: imwellcushtymelike(a)gmail.com
      Distribution: ---

Each distro/OS shows 0 test results:
https://appdb.winehq.org/objectManager.php?sClass=distribution&sTitle=View%2...

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

--- Comment #1 from Rosanne DiMesio <[e-mail]> ---
Still an issue.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

Stian Low <[e-mail]> changed:

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

--- Comment #2 from Stian Low <[e-mail]> ---
Regression caused by my commit from 1 year ago to support AppDB and Bugzilla
running on separate servers to improve traffic:
- https://bugs.winehq.org/show_bug.cgi?id=56706

5ca77e82075127387ef7c20be886b42db3843250
Author:     Stian Low <[e-mail]>
AuthorDate: Thu Jul 3 20:02:07 2025 +0300
Commit:     Stian Low <[e-mail]>
CommitDate: Tue Jul 8 10:19:14 2025 +0300

Will submit merge to fix:

modified   include/distribution.php
@@ -70,7 +70,7 @@ class distribution {
                             ORDER BY testedRating, testedRelease";
         }

-        if($hResult = query_parameters($sQuery, BUGZILLA_PRODUCT_ID,
$this->iDistributionId, APPS_DB))
+        if($hResult = query_parameters($sQuery, $this->iDistributionId,
APPS_DB))
         {
             while($oRow = query_fetch_object($hResult))
             {

Apologies for missing this one.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

--- Comment #3 from Stian Low <[e-mail]> ---
Created attachment 81593
  --> http://bugs.winehq.org/attachment.cgi?id=81593
Screenshot distro test results count corrected for my localhost instance

Pending merge should fix:
https://gitlab.winehq.org/winehq/appdb/-/merge_requests/33

I may have accidentally reverted removal of BUGZILLA_PRODUCT_ID from my merge a
year ago because I remember a deliberate effort to remove all instances of it
and verified via grep.  In any case this merge should remove any and all
remaining references to BUGZILLA_PRODUCT_ID.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

--- Comment #4 from Stian Low <[e-mail]> ---
(In reply to Stian Low from comment #3)
> Pending merge should fix:
> https://gitlab.winehq.org/winehq/appdb/-/merge_requests/33

MR-33 merged but only fixes for admins.

MR-35 pending to fix new error for regular users due to SQL query different
from admins which has invalid syntax:
- https://gitlab.winehq.org/winehq/appdb/-/merge_requests/35

Somehow bug fixed by MR-33 prevented SQL error from being thrown due to
misplaced comma.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

--- Comment #5 from Stian Low <[e-mail]> ---
(In reply to Ken Sharp from comment #0)
> Each distro/OS shows 0 test results:
> https://appdb.winehq.org/objectManager.
> php?sClass=distribution&sTitle=View%20Distributions

(In reply to Stian Low from comment #4)
> MR-35 pending to fix new error for regular users due to SQL query

MR-35 merge so distros link is working for regular users again instead of only
admins.

This bug may be fixed.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

Ken Sharp <[e-mail]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |f3306729e461032800416aab5c5
                   |                            |e95a6c16766f9
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Ken Sharp <[e-mail]> ---
Lovely!

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 58521] "View Distributions" shows 0 test results

[WineHQ Bugzilla]

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

Ken Sharp <[e-mail]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #7 from Ken Sharp <[e-mail]> ---
Closing bugs marked as fixed.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.