GetAttachment::Handle()

Complexity

Conditions	5
Paths	13

Size

Total Lines	32
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	public function Handle($commandCode) {
		$attname = Request::GetGETAttachmentName();
		if (!$attname) {
			return false;
		}

		try {
			$attachment = self::$backend->GetAttachmentData($attname);
			$stream = $attachment->data;
			SLog::Write(LOGLEVEL_DEBUG, sprintf("HandleGetAttachment(): attachment stream from backend: %s", $stream));

			// need to check for a resource here, as eg. feof('Error') === false and causing infinite loop in while!
			if (!is_resource($stream)) {
				throw new StatusException(sprintf("HandleGetAttachment(): No stream resource returned by backend for attachment: %s", $attname), SYNC_ITEMOPERATIONSSTATUS_INVALIDATT);

It seems like $attname can also be of type true; however, parameter $values of sprintf() does only seem to accept double|integer|string, maybe add an additional type check?

			}

			header("Content-Type: application/octet-stream");
			self::$topCollector->AnnounceInformation("Starting attachment streaming", true);
			$l = fpassthru($stream);
			fclose($stream);
			if ($l === false) {
				throw new FatalException("HandleGetAttachment(): fpassthru === false !!!");
			}
			self::$topCollector->AnnounceInformation(sprintf("Streamed %d KB attachment", round($l / 1024)), true);
			SLog::Write(LOGLEVEL_DEBUG, sprintf("HandleGetAttachment(): attachment with %d KB sent to mobile", round($l / 1024)));
		}
		catch (StatusException $s) {
			// StatusException already logged so we just need to pass it upwards to send a HTTP error
			throw new HTTPReturnCodeException($s->getMessage(), HTTP_CODE_500, null, LOGLEVEL_DEBUG);
		}

		return true;
	}