rudiedirkx / imap-reader

IMAP.php

Doc Comments +12 added lines

@@ -116,6 +116,9 @@ 
 		$this->unseen = $unseen;
 	}
 
+	/**
+	 * @param boolean $clear
+	 */
 	protected function flags( $flags, $clear ) {
 		$cb = $clear ? 'imap_clearflag_full' : 'imap_setflag_full';
 
@@ -214,6 +217,9 @@ 
 		return $this->parts;
 	}
 
+	/**
+	 * @param string $IMAPMessagePartClass
+	 */
 	protected function messageParts( $parts, $sectionPrefix = array(), $IMAPMessagePartClass ) {
 		$sectionPrefix = (array)$sectionPrefix;
 
@@ -263,6 +269,9 @@ 
 		return $this->decode($body);
 	}
 
+	/**
+	 * @param string $content
+	 */
 	public function decode( $content ) {
 		return quoted_printable_decode($content);
 	}
@@ -313,6 +322,9 @@ 
 		return file_put_contents($filepath, $this->decode($this->content()));
 	}
 
+	/**
+	 * @param string $content
+	 */
 	public function decode( $content ) {
 		return base64_decode($content);
 	}

Spacing +48 added lines, -48 removed lines

@@ -11,7 +11,7 @@ 
 
 	public $mbox; // IMAP resource
 
-	public function __construct( $server, $username, $password, $mailbox = 'INBOX', $flags = array() ) {
+	public function __construct($server, $username, $password, $mailbox = 'INBOX', $flags = array()) {
 		$this->server = $server;
 		$this->username = $username;
 		$this->password = $password;
@@ -23,16 +23,16 @@ 
 
 	public function connect() {
 		$server = $this->server;
-		if ( $this->flags ) {
-			$server .= '/' . implode('/', $this->flags);
+		if ($this->flags) {
+			$server .= '/'.implode('/', $this->flags);
 		}
 
 		$mailbox = '{'.$server.'}'.$this->mailbox;
 		$this->mbox = imap_open($mailbox, $this->username, $this->password);
 	}
 
-	public function messages( $options = true ) {
-		if ( is_bool($options) ) {
+	public function messages($options = true) {
+		if (is_bool($options)) {
 			$options = array('seen' => $options);
 		}
 
@@ -46,31 +46,31 @@ 
 		$IMAPMessageClass = $this->IMAPMessageClass;
 
 		$headers = imap_headers($this->mbox);
-		if ( $options['newestFirst'] ) {
+		if ($options['newestFirst']) {
 			$headers = array_reverse($headers);
 		}
 
 		$messages = array();
 		$eligables = 0;
-		foreach ( $headers AS $n => $header ) {
-			if ( preg_match('/(U?)\s+(\d+)\)/', $header, $match) ) {
+		foreach ($headers AS $n => $header) {
+			if (preg_match('/(U?)\s+(\d+)\)/', $header, $match)) {
 				$unseen = (bool)trim($match[1]);
 				$msgNum = (int)$match[2];
 
 				$eligable = $options['seen'] || $unseen;
-				if ( $eligable ) {
+				if ($eligable) {
 					$eligables++;
 				}
 
-				if ( $eligable ) {
-					if ( $eligables > $options['offset'] ) {
-						if ( !$options['limit'] || !isset($messages[$options['limit']-1]) ) {
+				if ($eligable) {
+					if ($eligables > $options['offset']) {
+						if (!$options['limit'] || !isset($messages[$options['limit']-1])) {
 							$messages[] = new $IMAPMessageClass($this, $msgNum, $header, $unseen);
 						}
 					}
 				}
 
-				if ( $options['limit'] && isset($messages[$options['limit']-1]) ) {
+				if ($options['limit'] && isset($messages[$options['limit']-1])) {
 					break;
 				}
 			}
@@ -79,8 +79,8 @@ 
 		return $messages;
 	}
 
-	static public function options( $options, $base ) {
-		foreach ( $options AS $name => $value ) {
+	static public function options($options, $base) {
+		foreach ($options AS $name => $value) {
 			$base[$name] = $value; // overwrite base
 		}
 
@@ -109,35 +109,35 @@ 
 	public $HTMLBody;
 	public $attachments = array(); // typeof Array<IMAPMessageAttachment>
 
-	public function __construct( IMAPMailbox $mailbox, $msgNumber, $header, $unseen ) {
+	public function __construct(IMAPMailbox $mailbox, $msgNumber, $header, $unseen) {
 		$this->mailbox = $mailbox;
 		$this->msgNumber = $msgNumber;
 		$this->header = $header;
 		$this->unseen = $unseen;
 	}
 
-	protected function flags( $flags, $clear ) {
+	protected function flags($flags, $clear) {
 		$cb = $clear ? 'imap_clearflag_full' : 'imap_setflag_full';
 
 		$feedback = array();
-		foreach ( (array)$flags AS $flag ) {
-			$flag = '\\' . ucfirst($flag);
+		foreach ((array)$flags AS $flag) {
+			$flag = '\\'.ucfirst($flag);
 			$feedback[] = $cb($this->mailbox->mbox, (string)$this->msgNumber, $flag);
 		}
 
 		return is_array($flags) ? $feedback : $feedback[0];
 	}
 
-	public function flag( $flags ) {
+	public function flag($flags) {
 		return $this->flags($flags, false);
 	}
 
-	public function unflag( $flags ) {
+	public function unflag($flags) {
 		return $this->flags($flags, true);
 	}
 
 	public function subject() {
-		if ( !$this->subject ) {
+		if (!$this->subject) {
 			$headers = $this->headers();
 
 			$subject = imap_utf8($headers->Subject);
@@ -149,7 +149,7 @@ 
 	}
 
 	public function headers() {
-		if ( !$this->headers ) {
+		if (!$this->headers) {
 			$this->headers = imap_headerinfo($this->mailbox->mbox, $this->msgNumber);
 		}
 
@@ -157,7 +157,7 @@ 
 	}
 
 	public function parts() {
-		if ( !$this->parts ) {
+		if (!$this->parts) {
 			$structure = $this->structure();
 
 			// Possibilities:
@@ -170,12 +170,12 @@ 
 			$parts = $attachments = array();
 
 			// - PLAIN
-			if ( 'PLAIN' == $structure->subtype ) {
+			if ('PLAIN' == $structure->subtype) {
 				$parts[] = new $IMAPMessagePartClass($this, $structure, '1');
 			}
 
 			// - ALTERNATIVE
-			else if ( 'ALTERNATIVE' == $structure->subtype ) {
+			else if ('ALTERNATIVE' == $structure->subtype) {
 				// get message parts
 				$parts = $this->messageParts($structure->parts, null, $IMAPMessagePartClass);
 			}
@@ -184,14 +184,14 @@ 
 			else {
 				$IMAPMessageAttachmentClass = self::$IMAPMessageAttachmentClass;
 
-				foreach ( $structure->parts AS $i => $part ) {
-					if ( 'ALTERNATIVE' == $part->subtype ) {
+				foreach ($structure->parts AS $i => $part) {
+					if ('ALTERNATIVE' == $part->subtype) {
 						$parts = array_merge($parts, $this->messageParts($part->parts, $i+1, $IMAPMessagePartClass));
 					}
 					else {
 						$parts[] = new $IMAPMessagePartClass($this, $part, $i+1);
 
-						if ( $part->ifdisposition && 'ATTACHMENT' == $part->disposition ) {
+						if ($part->ifdisposition && 'ATTACHMENT' == $part->disposition) {
 							$attachments[] = new $IMAPMessageAttachmentClass($this, $part, $i+1);
 						}
 					}
@@ -201,11 +201,11 @@ 
 			$this->parts = $parts;
 			$this->attachments = $attachments;
 
-			foreach ( $parts AS $part ) {
-				if ( 'PLAIN' == $part->subtype && !$this->plainBody ) {
+			foreach ($parts AS $part) {
+				if ('PLAIN' == $part->subtype && !$this->plainBody) {
 					$this->plainBody = $part;
 				}
-				else if ( 'HTML' == $part->subtype && !$this->HTMLBody ) {
+				else if ('HTML' == $part->subtype && !$this->HTMLBody) {
 					$this->HTMLBody = $part;
 				}
 			}
@@ -214,11 +214,11 @@ 
 		return $this->parts;
 	}
 
-	protected function messageParts( $parts, $sectionPrefix = array(), $IMAPMessagePartClass ) {
+	protected function messageParts($parts, $sectionPrefix = array(), $IMAPMessagePartClass) {
 		$sectionPrefix = (array)$sectionPrefix;
 
 		$partObjects = array();
-		foreach ( $parts AS $i => $part ) {
+		foreach ($parts AS $i => $part) {
 			$s = $sectionPrefix;
 			$s[] = (string)($i+1);
 			$section = implode('.', $s);
@@ -230,7 +230,7 @@ 
 	}
 
 	public function structure() {
-		if ( !$this->structure ) {
+		if (!$this->structure) {
 			$this->structure = imap_fetchstructure($this->mailbox->mbox, $this->msgNumber);
 		}
 
@@ -251,7 +251,7 @@ 
 	public $message; // typeof IMAPMessage
 	public $structure; // typeof stdClass
 
-	public function __construct( $message, $structure, $section ) {
+	public function __construct($message, $structure, $section) {
 		$this->message = $message;
 		$this->structure = $structure;
 		$this->section = (string)$section;
@@ -263,7 +263,7 @@ 
 		return $this->decode($body);
 	}
 
-	public function decode( $content ) {
+	public function decode($content) {
 		return quoted_printable_decode($content);
 	}
 
@@ -273,18 +273,18 @@ 
 
 	public $filename = '';
 
-	public function __construct( $message, $structure, $section ) {
+	public function __construct($message, $structure, $section) {
 		parent::__construct($message, $structure, $section);
 
 		$this->filename();
 	}
 
 	public function filename() {
-		if ( !$this->filename ) {
+		if (!$this->filename) {
 			// from dparameters
-			if ( $this->structure->ifdparameters ) {
-				foreach ( $this->structure->dparameters AS $param ) {
-					if ( 'FILENAME' == $param->attribute ) {
+			if ($this->structure->ifdparameters) {
+				foreach ($this->structure->dparameters AS $param) {
+					if ('FILENAME' == $param->attribute) {
 						$this->filename = $param->value;
 						break;
 					}
@@ -292,9 +292,9 @@ 
 			}
 
 			// from parameters
-			if ( !$this->filename && $this->structure->ifparameters ) {
-				foreach ( $this->structure->parameters AS $param ) {
-					if ( 'NAME' == $param->attribute ) {
+			if (!$this->filename && $this->structure->ifparameters) {
+				foreach ($this->structure->parameters AS $param) {
+					if ('NAME' == $param->attribute) {
 						$this->filename = $param->value;
 						break;
 					}
@@ -305,15 +305,15 @@ 
 		return $this->filename;
 	}
 
-	public function save( $filepath ) {
-		if ( '/' == substr($filepath, -1) ) {
+	public function save($filepath) {
+		if ('/' == substr($filepath, -1)) {
 			$filepath .= $this->filename;
 		}
 
 		return file_put_contents($filepath, $this->decode($this->content()));
 	}
 
-	public function decode( $content ) {
+	public function decode($content) {
 		return base64_decode($content);
 	}
 

Braces +2 added lines, -4 removed lines

@@ -187,8 +187,7 @@ 
 				foreach ( $structure->parts AS $i => $part ) {
 					if ( 'ALTERNATIVE' == $part->subtype ) {
 						$parts = array_merge($parts, $this->messageParts($part->parts, $i+1, $IMAPMessagePartClass));
-					}
-					else {
+					} else {
 						$parts[] = new $IMAPMessagePartClass($this, $part, $i+1);
 
 						if ( $part->ifdisposition && 'ATTACHMENT' == $part->disposition ) {
@@ -204,8 +203,7 @@ 
 			foreach ( $parts AS $part ) {
 				if ( 'PLAIN' == $part->subtype && !$this->plainBody ) {
 					$this->plainBody = $part;
-				}
-				else if ( 'HTML' == $part->subtype && !$this->HTMLBody ) {
+				} else if ( 'HTML' == $part->subtype && !$this->HTMLBody ) {
 					$this->HTMLBody = $part;
 				}
 			}

test.br-autohelpdesk.php

Spacing +27 added lines, -27 removed lines

@@ -3,34 +3,34 @@ 
 require 'env.php';
 require 'IMAP.php';
 
-$mbox = imap_open('{' . IMAP_HELPDESK_HOST . '/novalidate-cert}INBOX', IMAP_HELPDESK_USER, IMAP_HELPDESK_PASS);
+$mbox = imap_open('{'.IMAP_HELPDESK_HOST.'/novalidate-cert}INBOX', IMAP_HELPDESK_USER, IMAP_HELPDESK_PASS);
 
 $headers = imap_headers($mbox);
 print_r($headers);
 
-foreach ( $headers AS $hd ) {
-	if ( preg_match('/(\d+)\)/', $hd, $match) ) {
+foreach ($headers AS $hd) {
+	if (preg_match('/(\d+)\)/', $hd, $match)) {
 		$msgNum = $match[1];
 
 		$hd = imap_headerinfo($mbox, $msgNum);
 		$new = !!trim($hd->Unseen);
 
-		if ( $new ) {
+		if ($new) {
 
 			// subject -- should contain #code#
 			$title = get_plain_text_subject($mbox, $msgNum, $hd);
 			$code = preg_match('/#(\d+)#$/', $title, $match) ? (int)$match[1] : 0;
-echo $title . "\n";
+echo $title."\n";
 
 			// body -- get only last part (no conversation history)
 			$attachments = array();
 			$full_body = get_plain_text_body($mbox, $msgNum, $attachments);
 			$body = get_last_body_part($full_body);
-			if ( $attachments ) {
-				$body .= "\n\n== Attachments:\n* " . implode("\n* ", $attachments);
+			if ($attachments) {
+				$body .= "\n\n== Attachments:\n* ".implode("\n* ", $attachments);
 			}
 #echo $full_body . "\n===================================================\n";
-echo $body . "\n\n===================================================\n===================================================\n===================================================\n";
+echo $body."\n\n===================================================\n===================================================\n===================================================\n";
 
 			flush();
 		}
@@ -43,49 +43,49 @@ 
 imap_close($mbox, CL_EXPUNGE);
 
 
-function get_last_body_part( $body ) {
+function get_last_body_part($body) {
 	$lines = preg_split('/(\r\n|\r|\n)/', $body);
 
 	$body = '';
-	foreach ( $lines AS $line ) {
-		if ( '>' == substr($line, 0, 1) || '-----' == substr($line, 0, 5) || '_____' == substr($line, 0, 5) ) {
+	foreach ($lines AS $line) {
+		if ('>' == substr($line, 0, 1) || '-----' == substr($line, 0, 5) || '_____' == substr($line, 0, 5)) {
 			break;
 		}
 
-		$body .= $line . "\r\n";
+		$body .= $line."\r\n";
 	}
 
 	return trim($body);
 }
 
-function get_plain_text_subject( $mbox, $msgNum, $headers = null ) {
+function get_plain_text_subject($mbox, $msgNum, $headers = null) {
 	$headers or $headers = imap_headerinfo($mbox, $msgNum);
 
 	$subjectParts = imap_mime_header_decode($headers->Subject);
 	$subject = '';
-	foreach ( $subjectParts AS $p ) {
+	foreach ($subjectParts AS $p) {
 		$subject .= $p->text;
 	}
 
 	return trim($subject);
 }
 
-function get_plain_text_body( $mbox, $msgNum, &$attachments = array() ) {
+function get_plain_text_body($mbox, $msgNum, &$attachments = array()) {
 	$structure = imap_fetchstructure($mbox, $msgNum);
 
 	// only plain text
-	if ( 'PLAIN' == $structure->subtype ) {
+	if ('PLAIN' == $structure->subtype) {
 		return trim(imap_qprint(imap_body($mbox, $msgNum)));
 	}
 
-	if ( isset($structure->parts) ) {
+	if (isset($structure->parts)) {
 		// get attachments
-		foreach ( $structure->parts AS $partNum => $part ) {
-			if ( in_array($part->subtype, array('JPEG', 'PNG', 'GIF')) ) {
+		foreach ($structure->parts AS $partNum => $part) {
+			if (in_array($part->subtype, array('JPEG', 'PNG', 'GIF'))) {
 				// oeh an image
-				$name = 'image-from-email-'.$msgNum.'.' . strtolower($part->subtype);
-				foreach ( $part->parameters AS $param ) {
-					if ( 'NAME' == $param->attribute ) {
+				$name = 'image-from-email-'.$msgNum.'.'.strtolower($part->subtype);
+				foreach ($part->parameters AS $param) {
+					if ('NAME' == $param->attribute) {
 						$name = $param->value;
 					}
 				}
@@ -95,15 +95,15 @@ 
 		}
 
 		// multipart (probably) -- look for plain text part
-		foreach ( $structure->parts AS $partNum => $part ) {
-			if ( 'PLAIN' == $part->subtype ) {
+		foreach ($structure->parts AS $partNum => $part) {
+			if ('PLAIN' == $part->subtype) {
 				$body = imap_fetchbody($mbox, $msgNum, (string)($partNum+1));
 
 				return trim($body);
 			}
-			else if ( 'ALTERNATIVE' == $part->subtype && isset($part->parts) ) {
-				foreach ( $part->parts AS $subPartNum => $subPart ) {
-					if ( 'PLAIN' == $subPart->subtype ) {
+			else if ('ALTERNATIVE' == $part->subtype && isset($part->parts)) {
+				foreach ($part->parts AS $subPartNum => $subPart) {
+					if ('PLAIN' == $subPart->subtype) {
 						$body = imap_fetchbody($mbox, $msgNum, ($partNum+1).'.'.($subPartNum+1));
 
 						return trim($body);

Braces +1 added lines, -2 removed lines

@@ -100,8 +100,7 @@
 				$body = imap_fetchbody($mbox, $msgNum, (string)($partNum+1));
 
 				return trim($body);
-			}
-			else if ( 'ALTERNATIVE' == $part->subtype && isset($part->parts) ) {
+			} else if ( 'ALTERNATIVE' == $part->subtype && isset($part->parts) ) {
 				foreach ( $part->parts AS $subPartNum => $subPart ) {
 					if ( 'PLAIN' == $subPart->subtype ) {
 						$body = imap_fetchbody($mbox, $msgNum, ($partNum+1).'.'.($subPartNum+1));

test.11nk5.php

Spacing +7 added lines, -7 removed lines

@@ -6,40 +6,40 @@
 $mbox = new IMAPMailbox(IMAP_11NK5_HOST, IMAP_11NK5_USER, IMAP_11NK5_PASS, 'INBOX', ['ssl', 'novalidate-cert']);
 
 $messages = $mbox->messages(false);
-echo count($messages) . " new messages...\n\n";
+echo count($messages)." new messages...\n\n";
 
 
 exit;
 
 
-foreach ( $messages AS $message ) {
+foreach ($messages AS $message) {
 
 	$title = $message->subject();
 var_dump($title);
 
 	$message->parts();
 	$body = $message->plainBody;
-	if ( $body ) {
+	if ($body) {
 		$text = $body->content();
 
-		if ( preg_match('#^https?://[a-z0-9]#i', $text) ) {
+		if (preg_match('#^https?://[a-z0-9]#i', $text)) {
 			$tags = preg_split('/\s+/', $text);
 
 			$url = array_shift($tags);
 
 			$tags = implode(' ', $tags);
 
-			if ( $tags ) {
+			if ($tags) {
 var_dump($url);
 var_dump($tags);
 
-				if ( $message->unseen ) {
+				if ($message->unseen) {
 					$q = compact('title', 'url', 'tags');
 					$qs = http_build_query($q);
 
 					$rsp = @file_get_contents('http://hotblocks.nl/tags/index.php?'.$qs);
 					echo "SUBMIT URL: ";
-					if ( $rsp ) {
+					if ($rsp) {
 						var_dump(strlen($rsp));
 					}
 					else {

Braces +1 added lines, -2 removed lines

@@ -41,8 +41,7 @@
 					echo "SUBMIT URL: ";
 					if ( $rsp ) {
 						var_dump(strlen($rsp));
-					}
-					else {
+					} else {
 						echo "FAIL\n";
 						$message->unflag('seen');
 					}

test.br-bounces.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 require 'env.php';
 require 'IMAP.php';
 
-$mbox = imap_open('{' . IMAP_BOUNCES_HOST . '/novalidate-cert}INBOX', IMAP_BOUNCES_USER, IMAP_BOUNCES_PASS);
+$mbox = imap_open('{'.IMAP_BOUNCES_HOST.'/novalidate-cert}INBOX', IMAP_BOUNCES_USER, IMAP_BOUNCES_PASS);
 
 $headers = imap_headers($mbox);
 print_r($headers);

test.class.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 $messages = $mbox->messages(array('seen' => false, 'offset' => 0, 'limit' => 3, 'newestFirst' => true));
 #print_r($messages);
 
-foreach ( $messages AS $message ) {
+foreach ($messages AS $message) {
 
 	echo "------------------------------------------------------------------------\n";
 
@@ -20,7 +20,7 @@ 
 
 	print_r($message);
 
-	foreach ( $message->attachments AS $attachment ) {
+	foreach ($message->attachments AS $attachment) {
 		var_dump($attachment->save(__DIR__.'/attachments/'));
 	}