lint: Lint test fixes

include/class.email.php

@@ -200,10 +200,10 @@ class Email {
    function getIdByEmail($email) {
 
         $sql='SELECT email_id FROM '.EMAIL_TABLE.' WHERE email='.db_input($email);
-        if(($res=db_query($sql)) && db_num_rows($res))
-            list($id)=db_fetch_row($res);
+        if(!($res=db_query($sql)) || !db_num_rows($res))
+            return false;
 
-        return $id;
+        return db_result($res);
     }
 
     function lookup($var) {

include/class.mailparse.php

@@ -94,19 +94,21 @@ class Mail_Parse {
         }
 
         // Look for application/tnef attachment and process it
-        foreach ($this->struct->parts as $i=>$part) {
-            if (!$part->parts && $part->ctype_primary == 'application'
-                    && $part->ctype_secondary == 'ms-tnef') {
-                try {
-                    $tnef = new TnefStreamParser($part->body);
-                    $this->tnef = $tnef->getMessage();
-                    // No longer considered an attachment
-                    unset($this->struct->parts[$i]);
-                }
-                catch (TnefException $ex) {
-                    // TNEF will remain an attachment
-                    $this->notes[] = 'TNEF parsing exception: '
-                        .$ex->getMessage();
+        if ($this->struct && $this->struct->parts) {
+            foreach ($this->struct->parts as $i=>$part) {
+                if (!@$part->parts && $part->ctype_primary == 'application'
+                        && $part->ctype_secondary == 'ms-tnef') {
+                    try {
+                        $tnef = new TnefStreamParser($part->body);
+                        $this->tnef = $tnef->getMessage();
+                        // No longer considered an attachment
+                        unset($this->struct->parts[$i]);
+                    }
+                    catch (TnefException $ex) {
+                        // TNEF will remain an attachment
+                        $this->notes[] = 'TNEF parsing exception: '
+                            .$ex->getMessage();
+                    }
                 }
             }
         }
@@ -489,6 +491,12 @@ class EmailDataParser {
             return $this->err('Email parse failed ['.$parser->getError().']');
 
         $data =array();
+        $data['emailId'] = 0;
+        $data['subject'] = $parser->getSubject();
+        $data['header'] = $parser->getHeader();
+        $data['mid'] = $parser->getMessageId();
+        $data['priorityId'] = $parser->getPriority();
+
         //FROM address: who sent the email.
         if(($fromlist = $parser->getFromAddressList())) {
             $from=$fromlist[0]; //Default.
@@ -542,10 +550,6 @@ class EmailDataParser {
             $data['flags']['bounce'] = TicketFilter::isBounce($data['header']);
         }
 
-        $data['subject'] = $parser->getSubject();
-        $data['header'] = $parser->getHeader();
-        $data['mid'] = $parser->getMessageId();
-        $data['priorityId'] = $parser->getPriority();
         $data['emailId'] = $emailId;
 
         if (($replyto = $parser->getReplyTo())) {

setup/test/tests/test.jslint.php

@@ -22,4 +22,3 @@ class JsSyntaxTest extends Test {
 
 return 'JsSyntaxTest';
 ?>
-