CSV expander queue worker fails to generate ADO ingest update queue items when run from cron or hydroponics

[patdunlavey]

The reason is because, when running under cron and AmiUtilityService::preprocessAmiSet() is called from...

ami/src/Plugin/QueueWorker/CsvADOQueueWorker.php

Line 90 in f9d012e

$info = $this->AmiUtilityService->preprocessAmiSet($data->info['csv_file'], $data, $invalid, FALSE);

... the current user is anonymous (0), and as a result the access check...

ami/src/AmiUtilityService.php

Line 1842 in f9d012e

if (!$existing_object || !$existing_object->access('update')) {

... fails.

After discussion on slack, we agreed that it made sense to replicate the logic from...

ami/src/Plugin/QueueWorker/IngestADOQueueWorker.php

Lines 1188 to 1192 in f9d012e

	$account = $data->info['uid'] == \Drupal::currentUser()->id() ? \Drupal::currentUser() : $this->entityTypeManager->getStorage('user')->load($data->info['uid']);
	if ($data->pluginconfig->op !== 'create' && $account && $existing && count($existing) == 1) {
	$existing_object = reset($existing);
	if (!$existing_object->access('update', $account)) {

... where the access check is performed using the user account provided in the AMI set.

PR forthcoming!

[DiegoPino]

@patdunlavey thanks for opening this. Fixed via
bfdb7c5
and
567fea5

Thanks for your contribution!