Jeff Zaroyko : Fix a typo that caused some input validation to be skipped and added another statement to make ensure that part of the form is validated correctly

Chris Morgan cmorgan at winehq.org
Sat May 31 21:29:32 CDT 2008


Module: appdb
Branch: master
Commit: f2d44f7120386e0f39279616991e1f10aae98a9e
URL:    http://source.winehq.org/git/appdb.git/?a=commit;h=f2d44f7120386e0f39279616991e1f10aae98a9e

Author: Jeff Zaroyko <jeffzaroyko at gmail.com>
Date:   Sun Jun  1 12:06:47 2008 +1000

Fix a typo that caused some input validation to be skipped and added another statement to make ensure that part of the form is validated correctly

---

 include/testData.php |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/include/testData.php b/include/testData.php
index 45458d7..3e171cf 100644
--- a/include/testData.php
+++ b/include/testData.php
@@ -1015,8 +1015,11 @@ class testData{
 
         // Basic checking of logic.  Runs? can obviously only be 'Not Installable'
         // if the application does not install
-        if (($aValues['sInstalls'] != "No") && ($aValues['sRuns'] == "Not Installable"))
-            $errors .= "<li>You can only set Runs? to &#8216;Not Installable&#8217; if the applicatino&#8217;s installer does not work</li>\n";
+        if (($aValues['sInstalls'] != "No") && ($aValues['sRuns'] == "Not installable"))
+            $errors .= "<li>You can only set Runs? to &#8216;Not installable&#8217; if the application's installer does not work</li>\n";
+	    
+	if (($aValues['sInstalls'] == "No") && ($aValues['sRuns'] != "Not installable"))
+            $errors .= "<li>You can only set Runs? to &#8216;Not installable&#8217;' if the application's installer does not work</li>\n";
 
         return $errors;
 




More information about the wine-cvs mailing list