Module: appdb
Branch: master
Commit: d4907a77b42b72cbce5e49f0d4bde6007ec36d39
URL: http://source.winehq.org/git/appdb.git/?a=commit;h=d4907a77b42b72cbce5e49f0…
Author: Rosanne DiMesio <dimesio(a)earthlink.net>
Date: Wed Jul 12 14:47:47 2017 -0500
Change distribution to operating system in places visible to users
Distribution is a Linuxism that makes no sense for users of other
operating systems, and even many new Linux users are unfamiliar with
the term: some users try to add the name of a distributor or version to the distribution field. Operating system is more accurate and less likely to confuse users.
Signed-off-by: Rosanne DiMesio <dimesio(a)earthlink.net>
Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com>
---
include/distribution.php | 60 +++++++++++++++++++++++-----------------------
include/sidebar.php | 2 +-
include/testData.php | 8 +++----
include/testData_queue.php | 4 ++--
4 files changed, 37 insertions(+), 37 deletions(-)
Diff: http://source.winehq.org/git/appdb.git/?a=commitdiff;h=d4907a77b42b72cbce5e…
Module: appdb
Branch: master
Commit: c3635c89ab31de3a272aab97bda350d65f224837
URL: http://source.winehq.org/git/appdb.git/?a=commit;h=c3635c89ab31de3a272aab97…
Author: Rosanne DiMesio <dimesio(a)earthlink.net>
Date: Wed Jul 12 10:07:59 2017 -0500
Favor distributions selected from list over textboxes
When a distribution has been selected from the dropdown list, anything
typed in the add distribution textboxes is a mistake. Also corrects a typo that prevented the notice that the user had added a new distribution from displaying fully.
Signed-off-by: Rosanne DiMesio <dimesio(a)earthlink.net>
Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com>
---
include/testData.php | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/include/testData.php b/include/testData.php
index 051ef65..8dac819 100644
--- a/include/testData.php
+++ b/include/testData.php
@@ -814,7 +814,7 @@ class testData{
"please add it using the form below.";
} else
{
- $sDistributionHelp = "The user added a new distribution; ";
+ $sDistributionHelp = "The user added a new distribution; ".
"review it in the form below or replace ".
"it with one from the list.";
}
@@ -825,7 +825,7 @@ class testData{
echo '<td class=color0>',"\n";
echo $sDistributionHelp;
- distribution::make_distribution_list("iDistributionId", $this->iDistributionId);
+ distribution::make_distribution_list("iTestDistributionId", $this->iDistributionId);
echo '</td></tr>',"\n";
// Version List
@@ -927,7 +927,7 @@ class testData{
}
// No Distribution entered, and nothing in the list is selected
- if (empty($aValues['sDistribution']) && !$aValues['iDistributionId'])
+ if (empty($aValues['sDistribution']) && !$aValues['iTestDistributionId'])
$errors .= "<li>Please enter a distribution.</li>\n";
if (empty($aValues['sInstalls']))
@@ -973,7 +973,13 @@ class testData{
$this->shWhatDoesnt = $aValues['shWhatDoesnt'];
$this->shWhatNotTested = $aValues['shWhatNotTested'];
$this->sTestedDate = $aValues['sTestedDate'];
- $this->iDistributionId = $aValues['iDistributionId'];
+
+ //Favor distribution dropdown list selections over textboxes.
+ if($aValues['iTestDistributionId'])
+ $this->iDistributionId = $aValues['iTestDistributionId'];
+ else
+ $this->iDistributionId = $aValues['iDistributionId'];
+
$this->sTestedRelease = $aValues['sTestedRelease'];
$this->iStaging = intval($aValues['iStaging']);
$this->sInstalls = $aValues['sInstalls'];