DateParser::parseDate() - Code Metrics - Inspection of "[ical] Add `DateParser`" - SemanticMediaWiki/SemanticResultFormats - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — ical-dateparser ( 5468af )

by mw

created 2019-08-24 03:15 UTC

DateParser::parseDate() B

↳ Parent: DateParser

Complexity

Conditions	8
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	39
rs	8.0515
c	0
b	0
f	0
cc	8
nc	17
nop	2

<?php

namespace SRF\iCalendar;

use SMWDataValueFactory as DataValueFactory;
use SMWTimeValue as TimeValue;

/**
 * @license GNU GPL v2+
 * @since 3.1
 */
class DateParser {

	/**
	 * Extract a date string formatted for iCalendar from a SMWTimeValue object.
	 *
	 * @since 3.1
	 *
	 * @param TimeValue $dataValue
	 * @param boolean $isEnd
	 *
	 * @return string
	 */
	public function parseDate( TimeValue $dataValue, $isEnd = false ) {
		$year = $dataValue->getYear();

		// ISO range is limited to four digits
		if ( ( $year > 9999 ) || ( $year < -9998 ) ) {
			return '';
		}

		$year = number_format( $year, 0, '.', '' );
		$time = str_replace( ':', '', $dataValue->getTimeString( false ) );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

		// increment by one day, compute date to cover leap years etc.
		if ( ( $time == false ) && ( $isEnd ) ) {
			$dataValue = DataValueFactory::getInstance()->newDataValueByType(
				'_dat',
				$dataValue->getWikiValue() . 'T00:00:00-24:00'
			);
		}

		$month = $dataValue->getMonth();

		if ( strlen( $month ) == 1 ) {
			$month = '0' . $month;
		}

		$day = $dataValue->getDay();

		if ( strlen( $day ) == 1 ) {
			$day = '0' . $day;
		}

		$result = $year . $month . $day;

		if ( $time != false ) {
			$result .= "T$time";
		}

		return $result;
	}

}


1			<?php
2
3			namespace SRF\iCalendar;
4
5			use SMWDataValueFactory as DataValueFactory;
6			use SMWTimeValue as TimeValue;
7
8			/**
9			* @license GNU GPL v2+
10			* @since 3.1
11			*/
12			class DateParser {
13
14			/**
15			* Extract a date string formatted for iCalendar from a SMWTimeValue object.
16			*
17			* @since 3.1
18			*
19			* @param TimeValue $dataValue
20			* @param boolean $isEnd
21			*
22			* @return string
23			*/
24			public function parseDate( TimeValue $dataValue, $isEnd = false ) {
25			$year = $dataValue->getYear();
26
27			// ISO range is limited to four digits
28			if ( ( $year > 9999 ) \|\| ( $year < -9998 ) ) {
29			return '';
30			}
31
32			$year = number_format( $year, 0, '.', '' );
33			$time = str_replace( ':', '', $dataValue->getTimeString( false ) );
			0 ignored issues – show Documentation introduced 2015-11-28 10:35 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
34
35			// increment by one day, compute date to cover leap years etc.
36			if ( ( $time == false ) && ( $isEnd ) ) {
37			$dataValue = DataValueFactory::getInstance()->newDataValueByType(
38			'_dat',
39			$dataValue->getWikiValue() . 'T00:00:00-24:00'
40			);
41			}
42
43			$month = $dataValue->getMonth();
44
45			if ( strlen( $month ) == 1 ) {
46			$month = '0' . $month;
47			}
48
49			$day = $dataValue->getDay();
50
51			if ( strlen( $day ) == 1 ) {
52			$day = '0' . $day;
53			}
54
55			$result = $year . $month . $day;
56
57			if ( $time != false ) {
58			$result .= "T$time";
59			}
60
61			return $result;
62			}
63
64			}
65

SemanticMediaWiki / SemanticResultFormats

Push — ical-dateparser ( 5468af )

DateParser::parseDate() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like