aboutsummaryrefslogtreecommitdiffstats
path: root/apps/workflowengine/lib
Commit message (Collapse)AuthorAgeFilesLines
* refactor(workflowengine): Replace security annotations with respective ↵provokateurin2024-07-272-8/+8
| | | | | | attributes Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(workflows): Fix file systemtag cacheJoas Schilling2024-07-101-21/+9
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore: Add SPDX headerAndy Scherzinger2024-05-3132-684/+64
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* fix: Apply new coding standard to all filesCôme Chilliet2024-04-021-2/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Fix missing template parameter for IEventListenerCôme Chilliet2024-01-301-0/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-236-7/+7
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* fix(workflows): Finish password confirmationJoas Schilling2023-11-162-0/+8
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(workflowengine): use andWhere() not second where()Arthur Schiwon2023-11-131-1/+1
| | | | | | | - an unconditional where() condition is added right before - fixes also wrongly named methods in the Exception message Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix(workflow): Fix "Call to a member function getUID() on null" with appdataJoas Schilling2023-11-131-3/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* enh(IMountManager): Add method to get MountPoint from CachedMountInfoJonas2023-10-231-5/+11
| | | | Signed-off-by: Jonas <jonas@freesources.org>
* fix(isLegitimatedForUserId): Setup mountpoints to check file accessJonas2023-10-231-8/+22
| | | | | | | | | | | | | | | This fixes workflows on groupfolders, as it will consider access to files in groupfolders. It also fixes false positives where access to files was limited by other means not taken into account before, e.g. access control. For postDelete events, check for permissions of the parent folder instead, as the file itself no longer exists. Fixes: nextcloud/flow_notifications#71 Signed-off-by: Jonas <jonas@freesources.org>
* Migrate applications away from deprecated ILoggerCôme Chilliet2023-09-213-71/+19
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #38604 from ↵Côme Chilliet2023-08-072-6/+6
|\ | | | | | | | | fsamapoor/replace_strpos_calls_in_workflowengine_app Refactors "strpos" calls in /apps/workflowengine
| * Fixes "PossiblyFalseArgument" notices.Faraz Samapoor2023-07-032-6/+6
| | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com> Signed-off-by: Faraz Samapoor <fsa@adlas.at>
| * Refactors "strpos" calls in /apps/workflowengine to improve code readability.Faraz Samapoor2023-07-032-6/+6
| | | | | | | | | | Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com> Signed-off-by: Faraz Samapoor <fsa@adlas.at>
* | fix(workflowengine): Remove legacy event (deprecated since 17)Joas Schilling2023-07-191-10/+0
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix: Avoid failing with duplicate checksJulius Härtl2023-07-121-7/+1
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* workflowengine: Stop using a service alias for controllerjld31032023-06-192-3/+1
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* fix(workflow): Check tag attributeJoas Schilling2023-03-161-8/+29
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* techdebt(workflowengine): Remove transition event classesJoas Schilling2023-03-031-17/+11
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also check the scope when reading operations from the databaseJoas Schilling2023-02-231-0/+23
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Validate the scope when validating operationsJoas Schilling2023-02-231-3/+10
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* perf(workflowengine): Cache query that is performed on every requestJulius Härtl2023-02-101-2/+18
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* remove file exists check from `cacheAndReturnMimeType`Robin Appelman2022-12-201-2/+2
| | | | | | | | the method is only called if the file exists already a check against storing the fallback mimetype is added as a safety instead Signed-off-by: Robin Appelman <robin@icewind.nl>
* use mimetype from cache for workflow if availableRobin Appelman2022-12-201-4/+8
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Use TimedJob from OCP instead of OCCôme Chilliet2022-12-051-2/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Make sure to not pass null to DateTime::createFromFormatCôme Chilliet2022-11-141-2/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Move CappedMemoryCache to OCPCarl Schwan2022-07-141-1/+1
| | | | | | | | This is an helpful helper that should be used in more place than just server and this is already the case with groupfodlers, deck, user_oidc and more using it, so let's make it public Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Use SettingsSection in workflowengine settingsCarl Schwan2022-05-311-18/+16
| | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
* Add stricter psalm type for CappedMemoryCacheCarl Schwan2022-05-121-4/+5
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Change script loading order in worflowengineLouis Chemineau2022-03-311-6/+5
| | | | | | Fix: https://github.com/nextcloud/server/issues/31709 Signed-off-by: Louis Chemineau <louis@chmn.me>
* Fix similar potential problems with fetchOne loopsCôme Chilliet2022-03-082-2/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Fix psalm issuesCarl Schwan2022-01-131-1/+4
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Add helper method in WrapperCarl Schwan2022-01-131-12/+5
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* The storage is not static anymoreCarl Schwan2022-01-131-15/+11
| | | | | | Don't call twice $cache->getId Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Optimize FileSystemTags workflow for groupfolderCarl Schwan2022-01-131-9/+29
| | | | | | | | | | | | | | | | | | | | | In https://github.com/nextcloud/server/pull/28774 we disabled the caching for the groupfolder application since it worked due to the fact that in groupfolders, getFileIds could be called with the same $cacheId and path for actually different groupfolders. This revert this change and instead add the folderId from the groupFolder to the cacheId. This solve the issue of the uniqueness of the cacheId inside GroupFolder. Downside is that we introduce groupfolder specific implementation inside the server repo. The seconf optimization is to not consider paths starting with __groupfolders in executeCheck. This is due to the fact that files in the groupfolder application call two times executeCheck one time with the url __groupfolder/<folderId>/<path> and the other time with <path>. The first time will always return an empty systemTags array while the second call will return the correct system tags. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Fix RequestURL check for cli commandsCarl Schwan2022-01-111-6/+11
| | | | | | | | Fix https://github.com/nextcloud/files_automatedtagging/issues/526 Fix https://github.com/nextcloud/groupfolders/issues/1855 Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Move bundles to /distJohn Molakvoæ (skjnldsv)2022-01-081-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* fix testsRobin Appelman2021-12-021-2/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Detect mimetype by content only with contentJoas Schilling2021-10-271-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fixes an undefined index when getAccessList returns an empty arrayArthur Schiwon2021-10-221-1/+1
| | | | | | - [] is a valid return value that should be honored as having no access Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Do not cache file ids in FileSystemTags inside group foldersRichard Steinmetz2021-09-141-2/+10
| | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* Set alias for result of cast column functionDaniel Kesselberg2021-08-161-1/+2
| | | | | | On OCI an expression like to_char(events) end up as $row['to_char(events)'] in the query result. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Make "name" column nullable in workflow operationsVincent Petry2021-08-104-4/+69
| | | | | | | | | | | | | The "name" column is now unused and the code is always inserting an empty string. While this works with most databases, Oracle complains because an empty string is equivalent to null. To fix this, the column definition is changed to allow null values now. Also added some logging in case of database exceptions, because without this nothing would be logged to detect the above problem. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-0431-91/+198
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Rename External storages to External storageszaimen2021-05-201-1/+1
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* Prefer string $file over 1-element array in script()François Freitag2021-05-181-3/+1
| | | | | | | The string syntax is more obvious. There should be one (and preferably only one) way of doing things. Signed-off-by: François Freitag <mail@franek.fr>
* Merge pull request #25588 from nextcloud/techdept/custom-psr-loggerChristoph Wurst2021-03-051-2/+3
|\ | | | | Migrate custom loggers to PSR
| * Migrate WFE to the new PSR loggerChristoph Wurst2021-02-111-2/+3
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Move some settings over to the IInitialStateRoeland Jago Douma2021-03-022-12/+7
| | | | | | | | | | | | The old one has been deprecated. Part one of many. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>