aboutsummaryrefslogtreecommitdiffstats
path: root/apps/workflowengine/lib
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* | Bump nextcloud/coding-standard from 0.3.0 to 0.5.0dependabot-preview[bot]2021-02-182-3/+3
|/ | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 0.3.0 to 0.5.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v0.3.0...v0.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Migrate internal classes to the OCP db col typesChristoph Wurst2021-01-121-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove unneeded casts that were found by PsalmMorris Jobke2021-01-111-1/+1
| | | | | | In preparation of the update of Psalm from 4.2.1 to 4.3.1+ (see https://github.com/nextcloud/server/pull/24521) Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Bump doctrine/dbal from 2.12.0 to 3.0.0Christoph Wurst2021-01-083-17/+19
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* consider local external storages, tooArthur Schiwon2020-12-151-1/+2
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* enables the file name check also to match name of mountpointsArthur Schiwon2020-12-151-2/+20
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Only execute plain mimetype check for directories and do the fallback only ↵Morris Jobke2020-11-301-2/+6
| | | | | | | | for non-directories Ref #23096 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Files client and info bundles are always loaded so we don't need to load ↵Julius Härtl2020-11-251-2/+0
| | | | | | them again Signed-off-by: Julius Härtl <jus@bitgrid.net>
* avoid empty null default with value that will be inserted anywaysArthur Schiwon2020-11-232-13/+2
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* inform the user when flow config data exceeds thresholdsArthur Schiwon2020-10-281-0/+9
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Fix type errors detected by PsalmChristoph Wurst2020-10-141-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #23096 from nextcloud/bugfix/noid/mimetypes2Morris Jobke2020-10-051-0/+13
|\ | | | | Also check the path based mimetype for flow rule checks
| * Make sure to check the mimetype on the path if a secure one could not be ↵Julius Härtl2020-09-291-0/+13
| | | | | | | | | | | | detemined though the content Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Format code to a single space around binary operatorsChristoph Wurst2020-10-052-2/+2
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update urls to stackoverflow.com to "HTTPS"J0WI2020-09-171-3/+3
| | | | Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* Don't use SELECT DISTINCT when to_char() is used in a WHERE statementJoas Schilling2020-09-071-3/+3
| | | | | | Nice bug in Oracle Signed-off-by: Joas Schilling <coding@schilljs.com>