Passed
Push — user-enable-settings ( 42437f...3f14b4 )
by Matias
03:52
created

AddMissingImagesTask::execute()   A

Complexity

Conditions 4
Paths 6

Size

Total Lines 31
Code Lines 19

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 20
CRAP Score 4

Importance

Changes 0
Metric Value
cc 4
eloc 19
nc 6
nop 1
dl 0
loc 31
ccs 20
cts 20
cp 1
crap 4
rs 9.6333
c 0
b 0
f 0
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
use OCP\Files\IHomeStorage;
32
33
use OCA\FaceRecognition\BackgroundJob\FaceRecognitionBackgroundTask;
34
use OCA\FaceRecognition\BackgroundJob\FaceRecognitionContext;
35
use OCA\FaceRecognition\Db\Image;
36
use OCA\FaceRecognition\Db\ImageMapper;
37
use OCA\FaceRecognition\Helper\Requirements;
38
use OCA\FaceRecognition\Migration\AddDefaultFaceModel;
39
40
/**
41
 * Task that, for each user, crawls for all images in filesystem and insert them in database.
42
 * This is job that normally does file watcher, but this should be done at least once,
43
 * after app is installed (or re-enabled).
44
 */
45
class AddMissingImagesTask extends FaceRecognitionBackgroundTask {
46
	const FULL_IMAGE_SCAN_DONE_KEY = "full_image_scan_done";
47
48
	/** @var IConfig Config */
49
	private $config;
50
51
	/** @var ImageMapper Image mapper */
52
	private $imageMapper;
53
54
	/**
55
	 * @param IConfig $config Config
56
	 * @param ImageMapper $imageMapper Image mapper
57
	 */
58 5
	public function __construct(IConfig $config, ImageMapper $imageMapper) {
59 5
		parent::__construct();
60 5
		$this->config = $config;
61 5
		$this->imageMapper = $imageMapper;
62 5
	}
63
64
	/**
65
	 * @inheritdoc
66
	 */
67 4
	public function description() {
68 4
		return "Crawl for missing images for each user and insert them in DB";
69
	}
70
71
	/**
72
	 * @inheritdoc
73
	 */
74 5
	public function execute(FaceRecognitionContext $context) {
75 5
		$this->setContext($context);
76
77 5
		$model = intval($this->config->getAppValue('facerecognition', 'model', AddDefaultFaceModel::DEFAULT_FACE_MODEL_ID));
78
79
		// Check if we are called for one user only, or for all user in instance.
80 5
		$insertedImages = 0;
81 5
		$eligable_users = array();
82 5
		if (is_null($this->context->user)) {
83 2
			$this->context->userManager->callForSeenUsers(function (IUser $user) use (&$eligable_users) {
84 2
				$eligable_users[] = $user->getUID();
85 2
			});
86
		} else {
87 3
			$eligable_users[] = $this->context->user->getUID();
88
		}
89
90 5
		foreach($eligable_users as $user) {
91 5
			$fullImageScanDone = $this->config->getUserValue($user, 'facerecognition', AddMissingImagesTask::FULL_IMAGE_SCAN_DONE_KEY, 'false');
92 5
			if ($fullImageScanDone === 'true') {
93
				// Completely skip this task for this user, seems that we already did full scan for him
94 1
				$this->logDebug('Skipping full image scan for user ' . $user);
95 1
				continue;
96
			}
97
98 5
			$insertedImages += $this->addMissingImagesForUser($user, $model);
99 5
			$this->config->setUserValue($user, 'facerecognition', AddMissingImagesTask::FULL_IMAGE_SCAN_DONE_KEY, 'true');
100 5
			yield;
101
		}
102
103 5
		$this->context->propertyBag['AddMissingImagesTask_insertedImages'] = $insertedImages;
104 5
		return true;
105
	}
106
107
	/**
108
	 * Crawl filesystem for a given user
109
	 *
110
	 * @param string $userId ID of the user for which to crawl images for
111
	 * @param int $model Used model
112
	 * @return int Number of missing images found
113
	 */
114 5
	private function addMissingImagesForUser(string $userId, int $model): int {
115 5
		$this->logInfo(sprintf('Finding missing images for user %s', $userId));
116 5
		\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...
117 5
		\OC_Util::setupFS($userId);
118
119 5
		$userFolder = $this->context->rootFolder->getUserFolder($userId);
120 5
		return $this->parseUserFolder($model, $userFolder);
121
	}
122
123
	/**
124
	 * Recursively crawls given folder for a given user
125
	 *
126
	 * @param int $model Used model
127
	 * @param Folder $folder Folder to recursively search images in
128
	 * @return int Number of missing images found
129
	 */
130 5
	private function parseUserFolder(int $model, Folder $folder): int {
131 5
		$insertedImages = 0;
132 5
		$nodes = $this->getPicturesFromFolder($folder);
133 5
		foreach ($nodes as $file) {
134 2
			$this->logDebug('Found ' . $file->getPath());
135
136 2
			$image = new Image();
137 2
			$image->setUser($file->getOwner()->getUid());
138 2
			$image->setFile($file->getId());
139 2
			$image->setModel($model);
140
			// todo: this check/insert logic for each image is so inefficient it hurts my mind
141 2
			if ($this->imageMapper->imageExists($image) === null) {
142
				// todo: can we have larger transaction with bulk insert?
143 2
				$this->imageMapper->insert($image);
144 2
				$insertedImages++;
145
			}
146
		}
147
148 5
		return $insertedImages;
149
	}
150
151
	/**
152
	 * Return all images from a given folder.
153
	 *
154
	 * TODO: It is inefficient since it copies the array recursively.
155
	 *
156
	 * @param Folder $folder Folder to get images from
157
	 * @return array List of all images and folders to continue recursive crawling
158
	 */
159 5
	private function getPicturesFromFolder(Folder $folder, $results = array()) {
160
		// todo: should we also care about this too: instanceOfStorage(ISharedStorage::class);
161 5
		if ($folder->getStorage()->instanceOfStorage(IHomeStorage::class) === false) {
162
			return $results;
163
		}
164
165 5
		$nodes = $folder->getDirectoryListing();
166
167 5
		foreach ($nodes as $node) {
168 3
			if ($node instanceof Folder and !$node->nodeExists('.nomedia')) {
169 2
				$results = $this->getPicturesFromFolder($node, $results);
170 3
			} else if ($node instanceof File) {
171 3
				if (Requirements::isImageTypeSupported($node->getMimeType())) {
172 3
					$results[] = $node;
173
				}
174
			}
175
		}
176
177 5
		return $results;
178
	}
179
}
180