BotRiconferme\Task\Subtask\UserNotice::runInternal()

Complexity

Conditions	5
Paths	5

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php declare( strict_types=1 );
	public function runInternal() : int {
		$pages = $this->getDataProvider()->getCreatedPages();
		$users = $this->getDataProvider()->getUsersToProcess();

		if ( !$pages || !$users ) {

The expression $pages of type BotRiconferme\Page\PageRiconferma[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

			return self::STATUS_NOTHING;
		}

		$ricNums = [];
		foreach ( $pages as $page ) {
			$ricNums[ $page->getUser() ] = $page->getNum();
		}

		foreach ( $users as $user => $_ ) {
			$this->addMsg( $user, $ricNums[ $user ] );
		}

		return self::STATUS_GOOD;
	}