Elgg\Pages\Upgrades\MigratePageTop

Complexity

Total Complexity

9

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	migrate()	0	15	2
A	getVersion()	0	2	1
A	shouldBeSkipped()	0	2	1
A	needsIncrementOffset()	0	2	1
A	countItems()	0	5	1
A	run()	0	23	3

<?php

namespace Elgg\Pages\Upgrades;

use ElggObject;
use \Elgg\Upgrade\Batch;
use \Elgg\Upgrade\Result;

/**
 * Migrate 'object', 'page_top' to 'object', 'page'
 * also set metadata 'parent_guid' to 0
 *
 * @since 3.0
 */
class MigratePageTop implements Batch {
	
	/**
	 * {@inheritDoc}
	 */
	public function getVersion() {
		return 2017110700;
	}
	
	/**
	 * {@inheritDoc}
	 */
	public function shouldBeSkipped() {
		return empty($this->countItems());
	}
	
	/**
	 * {@inheritDoc}
	 */
	public function countItems() {
		return elgg_get_entities([

The expression return elgg_get_entities(array('type' => 'object', 'subtype' => 'page_top', 'count' => true)) also could return the type ElggEntity[]|false which is incompatible with the return type mandated by Elgg\Upgrade\Batch::countItems() of integer.

			'type' => 'object',
			'subtype' => 'page_top',
			'count' => true,
		]);
	}
	
	/**
	 * {@inheritDoc}
	 */
	public function needsIncrementOffset() {
		return false;
	}
	
	/**
	 * {@inheritDoc}
	 */
	public function run(Result $result, $offset) {
		
		/* @var \ElggBatch $page_tops */
		$page_tops = elgg_get_entities([
			'type' => 'object',
			'subtype' => 'page_top',
			'offset' => $offset,
			'limit' => 25,
			'batch' => true,
			'batch_inc_offset' => $this->needsIncrementOffset(),
		]);
		
		/* @var $page_top \ElggObject */
		foreach ($page_tops as $page_top) {
			if ($this->migrate($page_top)) {
				$result->addSuccesses();
			} else {
				$result->addError("Error migrating page_top: {$page_top->guid}");
				$result->addFailures();
			}
		}
		
		return $result;
	}
	
	/**
	 * Migrate one page_top to a page
	 *
	 * @param ElggObject $page_top the top page to migrate
	 *
	 * @return bool
	 */
	protected  function migrate(ElggObject $page_top) {
		
		$dbprefix = elgg_get_config('dbprefix');
		$query = "UPDATE {$dbprefix}entities
			SET subtype = 'page'
			WHERE guid = {$page_top->guid}
		";
		
		if (!update_data($query)) {
			return false;
		}
		
		$page_top->parent_guid = 0;
		
		return true;
	}
}