OC\Preview\Watcher

Complexity

Total Complexity

12

Size/Duplication

Total Lines	61
Duplicated Lines	37.7 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	postWrite()	13	13	3
B	preDelete()	0	19	5
A	postDelete()	10	10	3

<?php
/**
 * @copyright Copyright (c) 2016, Roeland Jago Douma <roeland@famdouma.nl>
 *
 * @author Roeland Jago Douma <roeland@famdouma.nl>
 *
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */
namespace OC\Preview;

use OCP\Files\File;
use OCP\Files\Node;
use OCP\Files\Folder;
use OCP\Files\IAppData;
use OCP\Files\NotFoundException;

/**
 * Class Watcher
 *
 * @package OC\Preview
 *
 * Class that will watch filesystem activity and remove previews as needed.
 */
class Watcher {
	/** @var IAppData */
	private $appData;

	/** @var int[] */
	private $toDelete = [];

	/**
	 * Watcher constructor.
	 *
	 * @param IAppData $appData
	 */
	public function __construct(IAppData $appData) {
		$this->appData = $appData;
	}

	public function postWrite(Node $node) {

This method seems to be duplicated in your project.

		// We only handle files
		if ($node instanceof Folder) {
			return;
		}

		try {
			$folder = $this->appData->getFolder($node->getId());
			$folder->delete();
		} catch (NotFoundException $e) {
			//Nothing to do
		}
	}

	public function preDelete(Node $node) {
		// To avoid cycles
		if ($this->toDelete !== []) {
			return;
		}

		if ($node instanceof File) {
			$this->toDelete[] = $node->getId();
			return;
		}

		/** @var Folder $node */
		$nodes = $node->search('');
		foreach ($nodes as $node) {
			if ($node instanceof File) {
				$this->toDelete[] = $node->getId();
			}
		}
	}

	public function postDelete(Node $node) {

The parameter $node is not used and could be removed.

This method seems to be duplicated in your project.

		foreach ($this->toDelete as $fid) {
			try {
				$folder = $this->appData->getFolder($fid);
				$folder->delete();
			} catch (NotFoundException $e) {
				// continue
			}
		}
	}
}