aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_versions
Commit message (Collapse)AuthorAgeFilesLines
* fix(l10n): Update translations from TransifexNextcloud bot30 hours2-2/+2
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* feat(file-versions): #53151 add VersionCreatedEvent eventadd-VersionCreatedEventBorja Domínguez Vázquez42 hours6-5/+63
| | | | | Signed-off-by: Borja Domínguez Vázquez <bdominguez@virtualwareco.com> Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(l10n): Update translations from TransifexNextcloud bot3 days2-0/+2
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* fix(l10n): Update translations from TransifexNextcloud bot5 days2-0/+32
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* feat: replace material icons with outline versionsskjnldsv7 days1-1/+1
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(l10n): Update translations from TransifexNextcloud bot2025-07-082-0/+56
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* chore: get revision for a nodeBorja Domínguez Vázquez2025-07-075-3/+25
| | | | | | | | | | | | | | | | | | | | | Signed-off-by: Borja Domínguez Vázquez <borja.dominguez@hotmail.com> Signed-off-by: Borja Domínguez Vázquez <iam.n3uro@gmail.com> wip Signed-off-by: Borja Domínguez Vázquez <borja.dominguez@hotmail.com> Signed-off-by: Borja Domínguez Vázquez <iam.n3uro@gmail.com> wip Signed-off-by: Borja Domínguez Vázquez <borja.dominguez@hotmail.com> Signed-off-by: Borja Domínguez Vázquez <iam.n3uro@gmail.com> Update IVersionBackend.php Signed-off-by: Borja Domínguez Vázquez <iam.n3uro@gmail.com> wip
* chore: use constants for metadataBorja Domínguez Vázquez2025-07-052-4/+9
| | | | Signed-off-by: Borja Domínguez Vázquez <borja.dominguez@hotmail.com>
* Merge pull request #52963 from ↵Arthur Schiwon2025-07-031-1/+43
|\ | | | | | | | | nextcloud/fix/avoid-crashing-versions-listener-on-non-existing-file fix(files_versions): Log error instead of crashing when event listeners get called on non-existing files
| * fix(files_versions): Log error instead of crashing when event listeners get ↵fix/avoid-crashing-versions-listener-on-non-existing-fileCôme Chilliet2025-07-011-1/+43
| | | | | | | | | | | | called on non-existing files Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #53276 from nextcloud/fix/log-failure-from-file-eventsArthur Schiwon2025-07-031-0/+19
|\ \ | | | | | | fix(files_versions): Log failure to compute node path
| * | chore: Use an if/else instead of the try/catch to make meaning more obviousfix/log-failure-from-file-eventsCôme Chilliet2025-06-171-2/+3
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | fix(files_versions): Log failure to compute node pathCôme Chilliet2025-06-031-0/+18
| | | | | | | | | | | | | | | | | | | | | To avoid hidden bugs, add some debug logging when a node path cannot be computed from a received event, to have more information for debugging Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | chore: run rectorrector-phpunit10Robin Appelman2025-07-014-7/+6
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | chore: apply new CSFixer rulesFerdinand Thiessen2025-07-0115-2/+16
| |/ |/| | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de> # Conflicts: # apps/settings/lib/SetupChecks/PhpOpcacheSetup.php
* | fix(l10n): Update translations from TransifexNextcloud bot2025-06-3086-738/+2
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | fix: create new file with content when creating test files for versioning testsversion-test-new-fileRobin Appelman2025-06-231-2/+2
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-06-232-2/+6
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | fix(files_versions): give the version label a min width so it does not get ↵Julien Veyssier2025-06-181-1/+11
| | | | | | | | | | | | invisible if the author name is long Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-06-092-2/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-06-052-2/+2
|/ | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* fix(l10n): Update translations from TransifexNextcloud bot2025-05-312-0/+2
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* fix(files_versions): "undefined restored" on restoreAlejo2025-05-291-1/+1
| | | | Signed-off-by: Alejo <xlejo@protonmail.com>
* fix(l10n): Update translations from TransifexNextcloud bot2025-05-212-2/+2
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* test: Migrate files_versions to PHPUnit 10tests/noid/migrate-files_versionsJoas Schilling2025-05-208-103/+63
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(files_versions): only close stream if still open, fixing S3fix/s3-versionsFerdinand Thiessen2025-05-181-1/+7
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(l10n): Update translations from TransifexNextcloud bot2025-05-182-2/+4
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* fix(files_version): use `Storage::writeStream` instead of deprecated ↵chore/move-stream-copy-implementationFerdinand Thiessen2025-05-161-4/+16
| | | | | | `streamCopy` if possible Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore: move streamCopy implementation from `OC_Helper` to `OCP\Files`Ferdinand Thiessen2025-05-161-2/+5
| | | | | | | | | The function was already there but called the legacy version. So moved the implementation and migrated all usages of it. Sadly the interface was slightly different so adjusted it to be compatible with both legacy and the OCP one. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #52820 from nextcloud/refactor/rector-coreKate2025-05-151-1/+1
|\
| * refactor: Extend rector to core/refactor/rector-coreprovokateurin2025-05-151-1/+1
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(files_versions): Remove routes leading to deleted ajax filesCôme Chilliet2025-05-141-10/+0
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #48560 from nextcloud/fix/migrate-encryption-away-from-hooksCôme Chilliet2025-05-142-5/+9
|\ | | | | feat(encryption): Migrate from hooks to events
| * chore: Assert rename success in versionning testsCôme Chilliet2025-05-131-1/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore(files_versions): Only mock getSystemValue method to avoid problems in ↵Côme Chilliet2025-05-131-1/+4
| | | | | | | | | | | | files_versions tests Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * feat(encryption): Migrate from hooks to eventsCôme Chilliet2025-05-131-3/+3
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | refactor: Apply rector refactoringschore/deps/rector-2.0provokateurin2025-05-142-2/+3
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(files_versions): Folder should not be hardcoedfixHardcodedVersionsFolderGit'Fellow2025-05-091-4/+8
|/ | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix(l10n): Update translations from TransifexNextcloud bot2025-04-302-2/+2
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #52369 from nextcloud/fix/files-versions-authorFerdinand Thiessen2025-04-291-1/+2
|\ | | | | fix(files_versions): wait for version to be created before setting metadata
| * fix(files_versions): wait for version to be created before setting metadatafix/files-versions-authorFerdinand Thiessen2025-04-231-1/+2
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-04-282-2/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | fix(files_versions): only handle path updates when there is pathfix/files-versions-listenersFerdinand Thiessen2025-04-271-4/+12
| | | | | | | | | | | | | | | | | | `getPathForNode` can fail with null for various reasons (e.g. no owner), in this cases we need to just skip the event handling. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | Merge pull request #51355 from AIlkiv/refactor/rollback-hook-usageStephan Orbaugh2025-04-251-8/+10
|\ \ | | | | | | refactor: cleanup rollback hook usage
| * | refactor: cleanup rollback hook usageailkiv2025-04-251-8/+10
| | | | | | | | | | | | Signed-off-by: ailkiv <a.ilkiv.ye@gmail.com>
* | | fix(files_versions): create version if previous does not existfix/files-version-creationFerdinand Thiessen2025-04-231-2/+11
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This issue happens reproducible if: - Versions is disabled - Upload a file - Enable versions - Upload same file unchanged - Now the error happens. Problem is that the mtime is unchanged so no version will be created on the upload, but it tries to update the last version which does not exists. Instead of "upload same file unchanged" you can also - like in the example stack trace above - use Android with an SD card with invalid mtime -> the mtime will be stripped so its always the same. Instead of disable versions the same also happens if e.g. the versions creation failed due to other issues. The solution now is to catch the exception and create if not exists. A cleaner solution would be to have a method on the versions backend like `hasVersionEntity(File $file, int $revision): bool` but this would be a breaking change or at least a feature that apps need to implement. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-04-182-2/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | fix(l10n): Update translations from TransifexNextcloud bot2025-04-172-2/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | Fix(l10n): Update translations from TransifexNextcloud bot2025-04-112-2/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | Fix(l10n): Update translations from TransifexNextcloud bot2025-04-092-4/+4
| | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>