Passed
Push — shared-storage-experiments ( 3f9681 )
by Matias
10:48
created

AddMissingImagesTask::getPicturesFromFolder()   A

Complexity

Conditions 6
Paths 5

Size

Total Lines 13
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 9
CRAP Score 6

Importance

Changes 3
Bugs 0 Features 0
Metric Value
cc 6
eloc 8
c 3
b 0
f 0
nc 5
nop 2
dl 0
loc 13
ccs 9
cts 9
cp 1
crap 6
rs 9.2222
1
<?php
2
/**
3
 * @copyright Copyright (c) 2017, Matias De lellis <[email protected]>
4
 * @copyright Copyright (c) 2018, Branko Kokanovic <[email protected]>
5
 *
6
 * @author Branko Kokanovic <[email protected]>
7
 *
8
 * @license GNU AGPL version 3 or any later version
9
 *
10
 * This program is free software: you can redistribute it and/or modify
11
 * it under the terms of the GNU Affero General Public License as
12
 * published by the Free Software Foundation, either version 3 of the
13
 * License, or (at your option) any later version.
14
 *
15
 * This program is distributed in the hope that it will be useful,
16
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
17
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
18
 * GNU Affero General Public License for more details.
19
 *
20
 * You should have received a copy of the GNU Affero General Public License
21
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
22
 *
23
 */
24
namespace OCA\FaceRecognition\BackgroundJob\Tasks;
25
26
use OCP\IConfig;
27
use OCP\IUser;
28
29
use OCP\Files\File;
30
use OCP\Files\Folder;
31
32
use OCA\FaceRecognition\BackgroundJob\FaceRecognitionBackgroundTask;
33
use OCA\FaceRecognition\BackgroundJob\FaceRecognitionContext;
34
use OCA\FaceRecognition\Db\Image;
35
use OCA\FaceRecognition\Db\ImageMapper;
36
use OCA\FaceRecognition\Helper\Requirements;
37
use OCA\FaceRecognition\Migration\AddDefaultFaceModel;
38
39
/**
40
 * Task that, for each user, crawls for all images in filesystem and insert them in database.
41
 * This is job that normally does file watcher, but this should be done at least once,
42
 * after app is installed (or re-enabled).
43
 */
44
class AddMissingImagesTask extends FaceRecognitionBackgroundTask {
45
	const FULL_IMAGE_SCAN_DONE_KEY = "full_image_scan_done";
46
47
	/** @var IConfig Config */
48
	private $config;
49
50
	/** @var ImageMapper Image mapper */
51
	private $imageMapper;
52
53
	/**
54
	 * @param IConfig $config Config
55
	 * @param ImageMapper $imageMapper Image mapper
56
	 */
57 10
	public function __construct(IConfig $config, ImageMapper $imageMapper) {
58 10
		parent::__construct();
59 10
		$this->config = $config;
60 10
		$this->imageMapper = $imageMapper;
61 10
	}
62
63
	/**
64
	 * @inheritdoc
65
	 */
66 4
	public function description() {
67 4
		return "Crawl for missing images for each user and insert them in DB";
68
	}
69
70
	/**
71
	 * @inheritdoc
72
	 */
73 10
	public function execute(FaceRecognitionContext $context) {
74 10
		$this->setContext($context);
75
76 10
		$model = intval($this->config->getAppValue('facerecognition', 'model', AddDefaultFaceModel::DEFAULT_FACE_MODEL_ID));
77
78
		// Check if we are called for one user only, or for all user in instance.
79 10
		$insertedImages = 0;
80 10
		$eligable_users = array();
81 10
		if (is_null($this->context->user)) {
82 2
			$this->context->userManager->callForSeenUsers(function (IUser $user) use (&$eligable_users) {
83 2
				$eligable_users[] = $user->getUID();
84 2
			});
85
		} else {
86 8
			$eligable_users[] = $this->context->user->getUID();
87
		}
88
89 10
		foreach($eligable_users as $user) {
90 10
			$userEnabled = $this->config->getUserValue($user, 'facerecognition', 'enabled', 'false');
91 10
			if ($userEnabled === 'false') {
92
				// Completely skip this task for this user, seems that disable analysis
93 2
				$this->logInfo('Skipping image scan for user ' . $user . ' that has disabled the analysis');
94 2
				continue;
95
			}
96
97 10
			$fullImageScanDone = $this->config->getUserValue($user, 'facerecognition', AddMissingImagesTask::FULL_IMAGE_SCAN_DONE_KEY, 'false');
98 10
			if ($fullImageScanDone === 'true') {
99
				// Completely skip this task for this user, seems that we already did full scan for him
100
				$this->logDebug('Skipping full image scan for user ' . $user);
101
				continue;
102
			}
103
104 10
			$insertedImages += $this->addMissingImagesForUser($user, $model);
105 10
			$this->config->setUserValue($user, 'facerecognition', AddMissingImagesTask::FULL_IMAGE_SCAN_DONE_KEY, 'true');
106 10
			yield;
107
		}
108
109 10
		$this->context->propertyBag['AddMissingImagesTask_insertedImages'] = $insertedImages;
110 10
		return true;
111
	}
112
113
	/**
114
	 * Crawl filesystem for a given user
115
	 *
116
	 * @param string $userId ID of the user for which to crawl images for
117
	 * @param int $model Used model
118
	 * @return int Number of missing images found
119
	 */
120 10
	private function addMissingImagesForUser(string $userId, int $model): int {
121 10
		$this->logInfo(sprintf('Finding missing images for user %s', $userId));
122 10
		\OC_Util::tearDownFS();
0 ignored issues
show
Bug introduced by
The type OC_Util was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
123 10
		\OC_Util::setupFS($userId);
124
125 10
		$userFolder = $this->context->rootFolder->getUserFolder($userId);
126 10
		return $this->parseUserFolder($userId, $model, $userFolder);
127
	}
128
129
	/**
130
	 * Recursively crawls given folder for a given user
131
	 *
132
	 * @param int $model Used model
133
	 * @param Folder $folder Folder to recursively search images in
134
	 * @return int Number of missing images found
135
	 */
136 10
	private function parseUserFolder(string $userId, int $model, Folder $folder): int {
137 10
		$insertedImages = 0;
138 10
		$nodes = $this->getPicturesFromFolder($folder);
139 10
		foreach ($nodes as $file) {
140 7
			$this->logDebug('Found ' . $file->getPath());
141
142 7
			$image = new Image();
143 7
			$image->setUser($userId);
144 7
			$image->setFile($file->getId());
145 7
			$image->setModel($model);
146
			// todo: this check/insert logic for each image is so inefficient it hurts my mind
147 7
			if ($this->imageMapper->imageExists($image) === null) {
148
				// todo: can we have larger transaction with bulk insert?
149 7
				$this->imageMapper->insert($image);
150 7
				$insertedImages++;
151
			}
152
		}
153
154 10
		return $insertedImages;
155
	}
156
157
	/**
158
	 * Return all images from a given folder.
159
	 *
160
	 * TODO: It is inefficient since it copies the array recursively.
161
	 *
162
	 * @param Folder $folder Folder to get images from
163
	 * @return array List of all images and folders to continue recursive crawling
164
	 */
165 10
	private function getPicturesFromFolder(Folder $folder, $results = array()) {
166 10
		$nodes = $folder->getDirectoryListing();
167 10
		foreach ($nodes as $node) {
168 8
			if ($node instanceof Folder and !$node->nodeExists('.nomedia')) {
169 3
				$results = $this->getPicturesFromFolder($node, $results);
170 8
			} else if ($node instanceof File) {
171 8
				if (Requirements::isImageTypeSupported($node->getMimeType())) {
172 7
					$results[] = $node;
173
				}
174
			}
175
		}
176
177 10
		return $results;
178
	}
179
}
180