[Bug 58510] New: Submitting new test results lists all versions of Wine

[WineHQ Bugzilla]

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

Bug ID: 58510
           Summary: Submitting new test results lists all versions of Wine
           Product: WineHQ Apps Database
           Version: unspecified
          Hardware: x86-64
               URL: https://appdb.winehq.org/objectManager.php
                OS: Linux
            Status: NEW
          Keywords: download, regression, source
          Severity: normal
          Priority: P2
         Component: appdb-unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
   Regression SHA1: a0e96bdcdbb5f3b950574685d9d9a66c9e9985a9
      Distribution: ---

Created attachment 78978
  --> http://bugs.winehq.org/attachment.cgi?id=78978
HTML

A recent change has made submitting a new test result list all versions of
Wine, whereas before it was only the last few and the stable release.

I'm guessing a0e96bdcdbb5f3b950574685d9d9a66c9e9985a9 may be the commit in
question.

[Bug 58510] Submitting new test results lists all versions of Wine

[WineHQ Bugzilla]

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

Stian Low  changed:

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

--- Comment #1 from Stian Low  ---
This bug was indeed spawned by:

https://bugs.winehq.org/show_bug.cgi?id=58473 Test results are not displayed
for applications

https://bugs.winehq.org/show_bug.cgi?id=56706 AppDB does not properly separate
the Bugzilla DB

AppDB previously had a separate versions table from Bugzilla and part of the
effort to separate them was to have AppDB use the Bugzilla versions list
available via HTML at https://bugs.winehq.org/query.cgi?format=advanced.

If the versions should not be synced then that may be justification for adding
the versions DB table back to AppDB.

If the list from Bugzilla is in proper order then it can be limited to only the
latest range to narrow the list.  Otherwise it may be further justification for
just adding the DB table back and storing the versions separately.

I'll take a look at the bugzilla repo to see if there's a way to grab a
filtered list of versions as another possible solution to narrow the list.

[Bug 58510] Submitting new test results lists all versions of Wine

[WineHQ Bugzilla]

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

--- Comment #2 from Stian Low  ---
The reasoning behind this change was that the same wine versions when creating
a new bug report should be the same list when creating new app version test
results.

https://bugs.winehq.org/enter_bug.cgi?product=Wine

Is there a flaw in that reasoning?  Does it matter if all versions are listed? 
If the list is ordered from latest to oldest is that a sufficient fix so that
you don't have to scroll all the way to the bottom?  That was missing from the
submitted solution.

[Bug 58510] Submitting new test results lists all versions of Wine

[WineHQ Bugzilla]

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

--- Comment #3 from Ken Sharp  ---
> Does it matter if all versions are listed?

I don't really know. If you look at the message next to the box:

“Version not listed? Your Wine is too old, upgrade!”

Someone wants users to upgrade Wine. Perhaps test results for old versions of
Wine are not very useful. Bug reports for old versions of Wine certainly
aren't, but you can't hide old versions on Bugzilla because of old bug reports.

If users never upgrade then we could have 1,000 results for the same version of
Wine, and I doubt that is very useful.

I guess it's a policy decision.

How were the versions populated originally?

[Bug 58510] Submitting new test results lists all versions of Wine

[WineHQ Bugzilla]

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

--- Comment #4 from Stian Low  ---
(In reply to Ken Sharp from comment #3)
> How were the versions populated originally?

via winehq_appdb_stianlow/tables/bugzilla_tables.sql which was removed during
the DB separation merge:

-- WineHQ Bugzilla for AppDB Development

/*
 * versions information
 */

CREATE TABLE `versions` (
  `value` varchar(64) NOT NULL default '',
  `product_id` smallint(6) NOT NULL default '0',
  `id` mediumint(9) NOT NULL auto_increment,
  PRIMARY KEY  (`id`),
  UNIQUE KEY `versions_product_id_idx` (`product_id`,`value`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 PACK_KEYS=1;

--
-- Dumping data for table `versions`
--

INSERT INTO `versions` VALUES ('unspecified',1,1);
INSERT INTO `versions` VALUES ('CVS',1,2);
INSERT INTO `versions` VALUES ('unspecified',2,3);
INSERT INTO `versions` VALUES ('unspecified',3,4);
INSERT INTO `versions` VALUES ('unspecified',4,5);
INSERT INTO `versions` VALUES ('20010305',1,6);
INSERT INTO `versions` VALUES ('20010216',1,7);
INSERT INTO `versions` VALUES ('20010112',1,8);
INSERT INTO `versions` VALUES ('20010326',1,9);