appdb/include testResults.php

[WineHQ]

ChangeSet ID:	26057
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2006/06/27 00:44:24

Modified files:
	include        : testResults.php 

Log message:
	Tony Lambregts <tony.lambregts at gmail.com>
	Don't allow empty "what works", "what was tested" and "what was not" fields

Patch: http://cvs.winehq.org/patch.py?id=26057

Old revision  New revision  Changes     Path
 1.21          1.22          +3 -4       appdb/include/testResults.php

Index: appdb/include/testResults.php
diff -u -p appdb/include/testResults.php:1.21 appdb/include/testResults.php:1.22
--- appdb/include/testResults.php:1.21	27 Jun 2006  5:44:24 -0000
+++ appdb/include/testResults.php	27 Jun 2006  5:44:24 -0000
@@ -511,9 +511,9 @@ class testData{
     {
 
         $aClean = array(); //array of filtered user input
-        $aClean['sWhatWorks'] = makeSafe($_REQUEST['sWhatWorks']);
-        $aClean['sWhatDoesnt'] = makeSafe($_REQUEST['sWhatDoesnt']);
-        $aClean['sWhatNotTested'] = makeSafe($_REQUEST['sWhatNotTested']);
+        $aClean['sWhatWorks'] = trim(makeSafe($_REQUEST['sWhatWorks']));
+        $aClean['sWhatDoesnt'] = trim(makeSafe($_REQUEST['sWhatDoesnt']));
+        $aClean['sWhatNotTested'] = trim(makeSafe($_REQUEST['sWhatNotTested']));
         $aClean['sDistribution'] = makeSafe($_REQUEST['sDistribution']);
         $aClean['sTestedDate'] = makeSafe($_REQUEST['sTestedDate']);
         $aClean['sTestedRelease'] = makeSafe($_REQUEST['sTestedRelease']);
@@ -523,7 +523,6 @@ class testData{
         $aClean['sTestedRating'] = makeSafe($_REQUEST['sTestedRating']);
 
 	$errors = "";
-
         if (empty($aClean['sWhatWorks']))
             $errors .= "<li>Please enter what worked.</li>\n";