BibTexFileExportPrinter::newItem() - Code Metrics - Inspection of "Move BibTex class, refs 510" - SemanticMediaWiki/SemanticResultFormats - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — bibtex-move ( 9d522c )

by mw

created 2019-07-27 13:43 UTC

BibTexFileExportPrinter::newItem() B

↳ Parent: BibTexFileExportPrinter

Complexity

Conditions	11
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	23
CRAP Score	11.0619

Importance

Changes

Metric	Value
dl	0
loc	42
ccs	23
cts	25
cp	0.92
rs	7.3166
c	0
b	0
f	0
cc	11
nc	8
nop	1
crap	11.0619

How to fix Complexity

<?php

namespace SRF\BibTex;

use SMWTimeValue as TimeValue;
use SMWQueryResult as QueryResult;
use SMW\Query\ResultPrinters\FileExportPrinter;

/**
 * Printer class for creating BibTeX exports
 *
 * For details on availble keys see the README
 *
 * Example of a book :
 *
 * @Book{abramowitz1964homf,
 *   author =     "Milton Abramowitz and Irene A. Stegun",
 *   title =     "Handbook of Mathematical Functions",
 *   publisher =     "Dover",
 *   year =     1964,
 *   address =     "New York",
 *   edition =     "ninth Dover printing, tenth GPO printing"
 * }
 *
 * @license GNU GPL v2+
 * @since 1.5
 *
 * @author Markus Krötzsch
 * @author Denny Vrandecic
 * @author Frank Dengler
 * @author Steren Giannini
 */
class BibTexFileExportPrinter extends FileExportPrinter {

	/**
	 * @see ResultPrinter::getName
	 *
	 * {@inheritDoc}
	 */
	public function getName() {
		return wfMessage( 'srf_printername_bibtex' )->text();
	}

	/**
	 * @see FileExportPrinter::getMimeType
	 *
	 * @since 1.8
	 *
	 * {@inheritDoc}
	 */
	public function getMimeType( QueryResult $queryResult ) {
		return 'text/bibtex';
	}

	/**
	 * @see FileExportPrinter::getFileName
	 *
	 * @since 1.8
	 *
	 * {@inheritDoc}
	 */
	public function getFileName( QueryResult $queryResult ) {

		if ( $this->params['filename'] !== '' ) {

			if ( strpos( $this->params['filename'], '.bib' ) === false ) {
				$this->params['filename'] .= '.bib';
			}

			return str_replace( ' ', '_', $this->params['filename'] );
		} elseif ( $this->getSearchLabel( SMW_OUTPUT_WIKI ) != '' ) {
			return str_replace( ' ', '_', $this->getSearchLabel( SMW_OUTPUT_WIKI ) ) . '.bib';
		}

		return 'BibTeX.bib';
	}

	/**
	 * @see ResultPrinter::getParamDefinitions
	 *
	 * @since 1.8
	 *
	 * {@inheritDoc}
	 */
	public function getParamDefinitions( array $definitions ) {
		$params = parent::getParamDefinitions( $definitions );

		$params['filename'] = [
			'message' => 'smw-paramdesc-filename',
			'default' => 'bibtex.bib',
		];

		return $params;
	}

	/**
	 * @since 3.1
	 *
	 * @param array $list
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
	 *
	 * @return string
	 */
	public function getFormattedList( $key, array $values ) {
		return $GLOBALS['wgLang']->listToText( $values );
	}

	/**
	 * @see ResultPrinter::getResultText
	 *
	 * {@inheritDoc}
	 */
	protected function getResultText( QueryResult $res, $outputMode ) {

		if ( $outputMode !== SMW_OUTPUT_FILE ) {
			return $this->getBibTexLink( $res, $outputMode );
		}

		$items = [];

		while ( $row = $res->getNext() ) {
			$items[] = $this->newItem( $row )->text();
		}

		return implode( "\r\n\r\n", $items );
	}

	private function getBibTexLink( QueryResult $res, $outputMode ) {

		// Can be viewed as HTML if requested, no more parsing needed
		$this->isHTML = $outputMode == SMW_OUTPUT_HTML;

		$link = $this->getLink(
			$res,
			$outputMode
		);

		return $link->getText( $outputMode, $this->mLinker );
	}

	/**
	 * @since 3.1
	 *
	 * @param $row array of SMWResultArray
	 *
	 * @return bibTexItem
	 */
	private function newItem( array /* of SMWResultArray */ $row ) {

		$item = new Item();
		$item->setFormatterCallback( [ $this, 'getFormattedList' ] );

		foreach ( $row as /* SMWResultArray */ $field ) {
			$printRequest = $field->getPrintRequest();
			$values = [];

			$label = strtolower( $printRequest->getLabel() );
			$dataValue = $field->getNextDataValue();

			if ( $dataValue === false ) {
				continue;
			}

			if ( $label === 'date' && $dataValue instanceof TimeValue ) {
				$item->set( 'year', $dataValue->getYear() );
				$item->set( 'month', $dataValue->getMonth() );
			} elseif ( $label === 'author' || $label === 'authors' ) {
				$values[] = $dataValue->getShortWikiText();

				while ( ( /* SMWDataValue */ $dataValue = $field->getNextDataValue() ) !== false ) {
					$values[] = $dataValue->getShortWikiText();
				}

				$item->set( 'author', $values );
			} elseif ( $label === 'editor' || $label === 'editors' ) {
				$values[] = $dataValue->getShortWikiText();

				while ( ( /* SMWDataValue */ $dataValue = $field->getNextDataValue() ) !== false ) {
					$values[] = $dataValue->getShortWikiText();
				}

				$item->set( 'editor', $values );
			} else {
				$item->set( $label, $dataValue->getShortWikiText() );
			}
		}

		return $item;
	}

}


1		<?php
2
3		namespace SRF\BibTex;
4
5		use SMWTimeValue as TimeValue;
6		use SMWQueryResult as QueryResult;
7		use SMW\Query\ResultPrinters\FileExportPrinter;
8
9		/**
10		* Printer class for creating BibTeX exports
11		*
12		* For details on availble keys see the README
13		*
14		* Example of a book :
15		*
16		* @Book{abramowitz1964homf,
17		* author = "Milton Abramowitz and Irene A. Stegun",
18		* title = "Handbook of Mathematical Functions",
19		* publisher = "Dover",
20		* year = 1964,
21		* address = "New York",
22		* edition = "ninth Dover printing, tenth GPO printing"
23		* }
24		*
25		* @license GNU GPL v2+
26		* @since 1.5
27		*
28		* @author Markus Krötzsch
29		* @author Denny Vrandecic
30		* @author Frank Dengler
31		* @author Steren Giannini
32		*/
33		class BibTexFileExportPrinter extends FileExportPrinter {
34
35		/**
36		* @see ResultPrinter::getName
37		*
38		* {@inheritDoc}
39		*/
40		public function getName() {
41		return wfMessage( 'srf_printername_bibtex' )->text();
42		}
43
44		/**
45		* @see FileExportPrinter::getMimeType
46		*
47		* @since 1.8
48		*
49		* {@inheritDoc}
50		*/
51	1	public function getMimeType( QueryResult $queryResult ) {
52	1	return 'text/bibtex';
53		}
54
55		/**
56		* @see FileExportPrinter::getFileName
57		*
58		* @since 1.8
59		*
60		* {@inheritDoc}
61		*/
62	2	public function getFileName( QueryResult $queryResult ) {
63
64	2	if ( $this->params['filename'] !== '' ) {
65
66	2	if ( strpos( $this->params['filename'], '.bib' ) === false ) {
67	1	$this->params['filename'] .= '.bib';
68		}
69
70	2	return str_replace( ' ', '_', $this->params['filename'] );
71		} elseif ( $this->getSearchLabel( SMW_OUTPUT_WIKI ) != '' ) {
72		return str_replace( ' ', '_', $this->getSearchLabel( SMW_OUTPUT_WIKI ) ) . '.bib';
73		}
74
75		return 'BibTeX.bib';
76		}
77
78		/**
79		* @see ResultPrinter::getParamDefinitions
80		*
81		* @since 1.8
82		*
83		* {@inheritDoc}
84		*/
85	1	public function getParamDefinitions( array $definitions ) {
86	1	$params = parent::getParamDefinitions( $definitions );
87
88	1	$params['filename'] = [
89		'message' => 'smw-paramdesc-filename',
90		'default' => 'bibtex.bib',
91		];
92
93	1	return $params;
94		}
95
96		/**
97		* @since 3.1
98		*
99		* @param array $list
		0 ignored issues – show Bug introduced 2019-07-27 12:57 UTC by Report Bug Copy Issue Report There is no parameter named `$list`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
100		*
101		* @return string
102		*/
103	1	public function getFormattedList( $key, array $values ) {
104	1	return $GLOBALS['wgLang']->listToText( $values );
105		}
106
107		/**
108		* @see ResultPrinter::getResultText
109		*
110		* {@inheritDoc}
111		*/
112	1	protected function getResultText( QueryResult $res, $outputMode ) {
113
114	1	if ( $outputMode !== SMW_OUTPUT_FILE ) {
115		return $this->getBibTexLink( $res, $outputMode );
116		}
117
118	1	$items = [];
119
120	1	while ( $row = $res->getNext() ) {
121	1	$items[] = $this->newItem( $row )->text();
122		}
123
124	1	return implode( "\r\n\r\n", $items );
125		}
126
127		private function getBibTexLink( QueryResult $res, $outputMode ) {
128
129		// Can be viewed as HTML if requested, no more parsing needed
130		$this->isHTML = $outputMode == SMW_OUTPUT_HTML;
131
132		$link = $this->getLink(
133		$res,
134		$outputMode
135		);
136
137		return $link->getText( $outputMode, $this->mLinker );
138		}
139
140		/**
141		* @since 3.1
142		*
143		* @param $row array of SMWResultArray
144		*
145		* @return bibTexItem
146		*/
147	1	private function newItem( array /* of SMWResultArray */ $row ) {
148
149	1	$item = new Item();
150	1	$item->setFormatterCallback( [ $this, 'getFormattedList' ] );
151
152	1	foreach ( $row as /* SMWResultArray */ $field ) {
153	1	$printRequest = $field->getPrintRequest();
154	1	$values = [];
155
156	1	$label = strtolower( $printRequest->getLabel() );
157	1	$dataValue = $field->getNextDataValue();
158
159	1	if ( $dataValue === false ) {
160	1	continue;
161		}
162
163	1	if ( $label === 'date' && $dataValue instanceof TimeValue ) {
164		$item->set( 'year', $dataValue->getYear() );
165		$item->set( 'month', $dataValue->getMonth() );
166	1	} elseif ( $label === 'author' \|\| $label === 'authors' ) {
167	1	$values[] = $dataValue->getShortWikiText();
168
169	1	while ( ( /* SMWDataValue */ $dataValue = $field->getNextDataValue() ) !== false ) {
170	1	$values[] = $dataValue->getShortWikiText();
171		}
172
173	1	$item->set( 'author', $values );
174	1	} elseif ( $label === 'editor' \|\| $label === 'editors' ) {
175	1	$values[] = $dataValue->getShortWikiText();
176
177	1	while ( ( /* SMWDataValue */ $dataValue = $field->getNextDataValue() ) !== false ) {
178	1	$values[] = $dataValue->getShortWikiText();
179		}
180
181	1	$item->set( 'editor', $values );
182		} else {
183	1	$item->set( $label, $dataValue->getShortWikiText() );
184		}
185		}
186
187	1	return $item;
188		}
189
190		}
191

SemanticMediaWiki / SemanticResultFormats

Push — bibtex-move ( 9d522c )

BibTexFileExportPrinter::newItem() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like