| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
| |
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
|
|
|
|
|
|
|
|
|
|
|
| |
A follow up on https://github.com/nextcloud/server/pull/40670
Based on discussions here: https://github.com/nextcloud/server/issues/38911#issuecomment-1741819188
This fixes the case were not all previews are generated, for example in the activity view: https://github.com/nextcloud/server/issues/38911#issuecomment-1738886106
Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
|
|
|
|
| |
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
|
|
|
|
| |
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
|
|
|
|
| |
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
|
|
|
|
| |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
|
|
|
| |
Signed-off-by: Joas Schilling <coding@schilljs.com>
|
|
|
|
| |
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
|
|
| |
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
user_usage_report keeps a counter for files read.
The app uses
- OC_Filesystem::read hook
- Event listener (the legacy one) for IPreview::EVENT
Request for previews do not trigger the hook, hence the additional event listener.
The thumbnails for a list or grid view should not count, so we need the width and height.
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|\
| |
| | |
Refactors "strpos" calls in lib/private to improve code readability.
|
| |
| |
| |
| | |
Signed-off-by: Faraz Samapoor <fsamapoor@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
We use a forked version of getID3 to read embedded images from mp3 files to use them as previews.
If the library is unable to extract a image or fails on something different we should handle it properly.
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
| |
| |
| |
| | |
Signed-off-by: JanisPlayer <54918417+JanisPlayer@users.noreply.github.com>
|
|\ \
| |/
|/|
| |
| | |
nextcloud/throw-exception-to-avoid-division-by-zero
fix(preview-generator): Throw exception before dividing by zero when generating previews
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
generating previews
If the maximum preview generated gives some kind of invalid IImage, it's dimentions and filename can be set to zero.
And then later we do a division by zero to keep the aspect ratio of the previews.
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Found via `codespell -q 3 -S l10n -L jus ./lib/public`
Signed-off-by: luz paz <luzpaz@github.com>
Update lib/public/Accounts/IAccount.php
Signed-off-by: luz paz <luzpaz@github.com>
Signed-off-by: Simon L <szaimen@e.mail.de>
Co-Authored-By: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
|
|\ \
| | |
| | | |
increase imaginary timeouts as for big files the processing could take very long
|
| | |
| | |
| | |
| | | |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
|\ \ \
| | | |
| | | | |
Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
|
| |\ \ \
| | | | |
| | | | | |
Prevent the OpenDocument preview generator from trying to open empty files
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
| | | | |
| | | | |
| | | | | |
Co-authored-by: Carl Schwan <carl@carlschwan.eu>
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Rationale: does not make sense, and triggers a deprecation error in
\ZipArchive.
Signed-off-by: Claus-Justus Heine <himself@claus-justus-heine.de>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
use directory listing in both functions to gain 25% speed on run where
every preview already exist.
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| | | |
| | | |
| | | | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| | | |
| | | |
| | | | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| | | |
| | | |
| | | | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| | | |
| | | |
| | | | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
getCachedPreview used to call `getFile`, and this calls `getDirectoryListing` (or underlying function that list directory) to find the file. This was done for every preview spec.
Now, this is done only once at the beginning of the loop, and the array is just iterated when needed to find the correct entry.
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
|/ /
| |
| |
| | |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
| |
| |
| | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| |
| |
| | |
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
| |
| |
| |
| |
| |
| | |
The use of `exec` will spawn a shell, using `/bin/sh` on POSIX platforms. But in restricted environment, such as AppArmor, this means giving execution to `/bin/sh`, which renders the execution restriction quite useless.
Using an array with `proc_open` reduces this, and paved the way for file streaming instead of temporary file.
Signed-off-by: Glandos <bugs-github@antipoul.fr>
|
|/
|
|
|
|
| |
turns out this can be quite slow
Signed-off-by: Robin Appelman <robin@icewind.nl>
|
|
|
|
| |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
|
|
|
| |
Signed-off-by: Simon L <szaimen@e.mail.de>
|
|
|
|
| |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
|
|
|
| |
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
|
|
| |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
|
|
|
| |
Signed-off-by: Varun Patil <varunpatil@ucla.edu>
|
|
|
|
| |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
|
|
|
| |
Signed-off-by: Varun Patil <varunpatil@ucla.edu>
|
|
|
|
| |
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
|
|
|
|
|
| |
Signed-off-by: Bowen Ding <dbw9580@live.com>
Signed-off-by: szaimen <szaimen@e.mail.de>
|
|
|
|
| |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
|
|
|
|
| |
Signed-off-by: Julius Härtl <jus@bitgrid.net>
|
|
|
|
| |
Signed-off-by: Christopher Ng <chrng8@gmail.com>
|