+global\iCalendarReader::parse()

Complexity

Conditions	34
Paths	71

Size

Total Lines	101
Code Lines	74

Duplication

Lines	0
Ratio	0 %

<?php
	public function parse( $url = '' ) {
		$cache_group = 'icalendar_reader_parse';
		$disable_get_key = 'disable:' . md5( $url );

		// Check to see if previous attempts have failed
		if ( false !== wp_cache_get( $disable_get_key, $cache_group ) )
			return false;

		// rewrite webcal: URI schem to HTTP
		$url = preg_replace('/^webcal/', 'http', $url );
		// try to fetch
		$r = wp_remote_get( $url, array( 'timeout' => 3, 'sslverify' => false ) );
		if ( 200 !== wp_remote_retrieve_response_code( $r ) ) {
			// We were unable to fetch any content, so don't try again for another 60 seconds
			wp_cache_set( $disable_get_key, 1, $cache_group, 60 );
			return false;
		}

		$body = wp_remote_retrieve_body( $r );
		if ( empty( $body ) )
			return false;

		$body = str_replace( "\r\n", "\n", $body );
		$lines = preg_split( "/\n(?=[A-Z])/", $body );

		if ( empty( $lines ) )
			return false;

		if ( false === stristr( $lines[0], 'BEGIN:VCALENDAR' ) )
			return false;

		foreach ( $lines as $line ) {
			$add  = $this->key_value_from_string( $line );

Equals sign not aligned correctly; expected 1 space but found 2 spaces

			if ( ! $add ) {
				$this->add_component( $type, false, $line );

The variable $type does not seem to be defined for all execution paths leading up to this point.

false is of type boolean, but the function expects a string.

				continue;
			}
			list( $keyword, $value ) = $add;

			switch ( $keyword ) {
				case 'BEGIN':
				case 'END':
					switch ( $line ) {
						case 'BEGIN:VTODO':
							$this->todo_count++;
							$type = 'VTODO';
							break;
						case 'BEGIN:VEVENT':
							$this->event_count++;
							$type = 'VEVENT';
							break;
						case 'BEGIN:VCALENDAR':
						case 'BEGIN:DAYLIGHT':
						case 'BEGIN:VTIMEZONE':
						case 'BEGIN:STANDARD':
							$type = $value;
							break;
						case 'END:VTODO':
						case 'END:VEVENT':
						case 'END:VCALENDAR':
						case 'END:DAYLIGHT':
						case 'END:VTIMEZONE':
						case 'END:STANDARD':
							$type = 'VCALENDAR';
							break;
					}
					break;
				case 'TZID':
					if ( 'VTIMEZONE' == $type && ! $this->timezone )
						$this->timezone = $this->timezone_from_string( $value );
					break;
				case 'X-WR-TIMEZONE':
					if ( ! $this->timezone )
						$this->timezone = $this->timezone_from_string( $value );
					break;
				default:
					$this->add_component( $type, $keyword, $value );
					break;
			}
		}

		// Filter for RECURRENCE-IDs
		$recurrences = array();
		if ( array_key_exists( 'VEVENT', $this->cal ) ) {
			foreach ( $this->cal['VEVENT'] as $event ) {
				if ( isset( $event['RECURRENCE-ID'] ) ) {
					$recurrences[] = $event;
				}
			}
			foreach ( $recurrences as $recurrence ) {
				for ( $i = 0; $i < count( $this->cal['VEVENT'] ); $i++ ) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

					if ( $this->cal['VEVENT'][ $i ]['UID'] == $recurrence['UID'] && ! isset( $this->cal['VEVENT'][ $i ]['RECURRENCE-ID'] ) ) {
						$this->cal['VEVENT'][ $i ]['EXDATE'][] = $recurrence['RECURRENCE-ID'];
						break;
					}
				}
			}
		}

		return $this->cal;
	}