Commit graph

798 commits

Author SHA1 Message Date
Ferdinand Thiessen
35d4e562eb
fix(files): Ensure actions never overflow filename
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-24 11:41:37 +01:00
Ferdinand Thiessen
0a08768a37
Merge pull request #51400 from nextcloud/chore/tests-hot-key
test: speed up hotkey tests by 2.4s
2025-03-17 21:55:50 +01:00
Ferdinand Thiessen
884effa7c5
fix(files_sharing): ensure share status action works also in grid view
Remove some hacks from files app about the *files_sharing* status
action, in general not sure why this hack was there instead of being in
the correct app - but it broke the grid view.

So now the sharing information is also available in grid view.

Moreover the icon is fixed in size to not overflow the actions menu.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-13 18:28:32 +01:00
Caitlin Jordan
ee013b4789 fix(files): correctly handle plural translation forms
Co-authored-by: Caitlin Jordan <jordan.cait17@gmail.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-12 09:02:55 +01:00
Ferdinand Thiessen
8fbfa84a54
test: speed up hotkey tests by 2.4s
That is about 2500% as we needed to always wait for the timeout of
500ms. As the event was already handled - compared to now where we only
wait for one tick which is about 20ms.

Also the tests did not work as expected - for shift and meta key they
did not report the failing tests.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-11 18:11:12 +01:00
Ferdinand Thiessen
7b5cd60614
fix(files): also enable delete action if trashbin is disabled
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-11 15:05:44 +01:00
Ferdinand Thiessen
9a71e1b8c2
fix(files): also show file list headers on empty views
It is needed, e.g. for the note-to-recipient, that the headers
are also shown when there is no content (yet).

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-05 11:15:16 +01:00
Ferdinand Thiessen
4eb2c45c33
fix(files_sharing): ensure downloaded file has the correct filename
Single file shares use the share token as source name, so we need to use
the displayname. To do so we need to set the download attribute to the
displayname of the file to download.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-03-05 08:31:25 +01:00
Kate
2582a55300
Merge pull request #51146 from nextcloud/fix/files-trash-download
fix(files_trashbin): disable bulk download for trashbin
2025-03-03 16:50:34 +01:00
Ferdinand Thiessen
8b36b915fe
fix(files_trashbin): disable bulk download for trashbin
The backend does not allow bulk download within the trashbin,
so we need to disable this also on the frontend.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-28 18:13:24 +01:00
Valdnet
288287935a fix(l10n): use do not instead of don't
Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
2025-02-28 16:43:14 +01:00
Ferdinand Thiessen
f7f41b6cb4
refactor: use eventbus for updated filters
prevent filters not applied if `filtersChanged` was set before file list
mounted.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-28 10:43:26 +01:00
Ferdinand Thiessen
a673547397
fix(files): Ensure the filter instance is mounted
`reset` could be called before the filters are mounted,
in this case it is valid to update the presets, but we must not try to
access the vue instance (as it does not exist yet).

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-28 10:43:26 +01:00
Varun Patil
dbef6977db fix(files): exception when share-attributes is undefined
Signed-off-by: Varun Patil <varunpatil@ucla.edu>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
2025-02-27 11:10:40 +00:00
Joas Schilling
095ab4419e
fix(l10n): Improve english source strings
- No leading/trailing whitespace
- Use asci single quote

Signed-off-by: Joas Schilling <coding@schilljs.com>
2025-02-26 09:54:32 +01:00
Valdnet
5c9134124b
fix(l10n): Spelling standardization
Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
2025-02-25 18:40:06 +01:00
Ferdinand Thiessen
e712104e58
feat(files): allow to ignore warning to change file type
* Missing pieces of https://github.com/nextcloud/server/issues/46528
  * Add checkbox to not show this dialog again
  * Add user config as suggested by designers in files settings to
    reenable or diable this behavior.
  * Fix behavior of dialog: It says "keep .ext" but it does not keep the
    extension but cancels the operation. From the button label the user
    expects that the operation is continued but with the old extension.
  * Added more test coverage by adding component tests.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-25 02:16:56 +01:00
Jérôme Herbinet
e3d8edb7fc
fix: rename 'edit locally' to 'open locally'
Replace "Edit" by "Open"

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
2025-02-21 16:42:00 +01:00
skjnldsv
4a96441437 fix(files): handle failed node properly
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-21 10:49:43 +01:00
Ferdinand Thiessen
b06f5ba4c4
refactor: Migrate nextcloud-vue usage to new import schema
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-20 02:28:13 +01:00
skjnldsv
99579f0bbc fix(files): do not show extension warning for folders renaming
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-19 15:35:50 +01:00
skjnldsv
c87a6ba811 fix(files): attach sidebar to main content for vue debug
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-18 16:01:24 +01:00
skjnldsv
4f795e144c fix(files): properly show file not found error
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-18 10:53:23 +01:00
skjnldsv
2e50a39265 fix(files): properly forward open params from short urls
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-18 10:53:23 +01:00
Julien Veyssier
d8c24b1520
fix(transferownership): allow searching by email, display user.shareWithDisplayNameUnique in item subline, adjust style
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
2025-02-17 11:31:25 +01:00
Elizabeth Danzberger
7db90efe3b
fix(template-filler): Hide template fields with unknown labels
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
2025-02-13 09:24:00 +01:00
Ferdinand Thiessen
fc9108b11a
fix(files): only send config update requests if user is logged in
Since we use the files app also for public shares it is not guaranteed
that there is a user logged in, in that case the update for user / view
config will fail.
So ensure there is a user or do not send a request.

Also refactor both stores to setup styles to fix (remove) initialization hack,
which causes Typescript issues.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-08 14:28:42 +01:00
skjnldsv
73e4a13a1c
chore(files): add actions cypress tests
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-07 10:07:00 +01:00
skjnldsv
e10947a548
fix(files): support submenu in batch actions header too
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-06 15:25:37 +01:00
Ferdinand Thiessen
dbc5c10bc4
fix(files): Do not download files with openfile query flag
Instead of downloading files, if there is no other default action,
we should just open the details tab.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-06 02:17:25 +01:00
Ferdinand Thiessen
d9996b92dc
fix(files): Correctly scroll selected file into view
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-02-05 18:40:00 +01:00
skjnldsv
85ba80b8ce fix(files): properly update store on files conversions success
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-02-04 16:38:34 +01:00
Christopher Ng
a447b21805 fix(files): Fix casing
Signed-off-by: Christopher Ng <chrng8@gmail.com>
2025-02-04 12:22:16 +01:00
Christopher Ng
ad9b07d85e chore(files): Change "filename" to "file name"
Signed-off-by: Christopher Ng <chrng8@gmail.com>
2025-02-04 12:22:16 +01:00
nfebe
8863b85420 fix: Use searchFrom property for client side plugins
The client-side plugin `in-folder` uses the `files` provider, this makes it
overlap with the main files provider itself.

This change follows eecda06f1a after it was discovered
that some apps/providers like `dav` use providers from another app like `contacts`

Signed-off-by: nfebe <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
2025-01-30 17:35:00 +00:00
provokateurin
4aede1b43d
fix(files): Allow opening the same file repeatedly
Signed-off-by: provokateurin <kate@provokateurin.de>
2025-01-30 13:53:53 +01:00
Christopher Ng
7a8d069907 perf(files): Check node type property instead of instance
Signed-off-by: Christopher Ng <chrng8@gmail.com>
2025-01-29 15:32:10 -08:00
nfebe
a5307ac512 fix(unified-search): Prevent multiple file picker triggers in in-folder search
Signed-off-by: nfebe <fenn25.fn@gmail.com>
2025-01-29 12:50:12 +01:00
nfebe
eecda06f1a fix(unified-search): Use appId for searching
Each provider may search from a particular app so we should use that for searching.

Before this commit, we used `provider.id` instead of `provider.appId` the problem with the previous
approach is that it forces the provider id to be a valid search provider (an app that supports search)
limiting the developers ability to use unique IDs to identify the different providers (especially plugin providers)
inside the places filter.

For example the Files search plugin "In folder" (search in folder plugin) was required to have id as `files` while the
files provider itself already has id as `files`.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
2025-01-29 12:50:12 +01:00
Pawel Boguslawski
c7bcfbf864 fix: Hide "Create templates folder" option if templates are disabled in configuration
When both `skeletondirectory` and `templatedirectory` are set to empty
strings in configuration, templates folder creation should be disabled
and no Create templates folder option should be present
in new folder menu.

Related: https://github.com/nextcloud/server/issues/39266
Related: https://github.com/nextcloud/server/issues/46455
Author-Change-Id: IB#1156403

Signed-off-by: Pawel Boguslawski <pawel.boguslawski@ib.pl>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2025-01-27 16:08:53 +01:00
skjnldsv
5dc091aa58 fix(files): jsunit file_conversions
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-01-22 18:12:54 +01:00
skjnldsv
6673c127ec fix(files): better conversion error messages
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-01-22 16:29:36 +00:00
skjnldsv
7d8bb60bfe feat(files): add conversion action
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
2025-01-22 16:29:36 +00:00
Ferdinand Thiessen
344c8a409c
fix(files): Ensure favorites set in sidebar work
When marking a file as favorite from within the sidebar make sure it
really works, this fixes two issues:
1. The source needs to be the plain source not URL encoded, as otherwise
   the source of the node would be encoded twice (and show with encoding
   in the navigation)
2. The store should also listen for the update events as the sidebar has
   no access to the real node to update it, instead the store should -
   as long as we only have the legacy sidebar - update the node when
   added or removed as favorite.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-01-21 17:55:53 +01:00
Ferdinand Thiessen
b5b28fda16
fix(files): Properly reset all file list filters on view change
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-01-21 16:38:06 +01:00
Ferdinand Thiessen
4a9954f5ef
fix(files): File type filter UI sync with filter state
When changing the folder the filter will be re-mounted by the file list,
so we need to pass the current state of the filter to the filter UI.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-01-17 08:59:17 +01:00
Ferdinand Thiessen
24851d930d
chore: Resolve ESLint warnings
- Add default value to non-required Vue props
- Reformat function to async function if needed
- Add some documentation
- Allow `any` in places where it makes sense (tests)
- Order vue component sections as required

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2025-01-16 22:18:29 +01:00
Richard Steinmetz
835994a811
fix(files): sort not working after changing views
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
2025-01-16 08:57:19 +01:00
Maksim Sukharev
b06a8767f0 fix(files): hide move dialog action if destination has no CREATE permission
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
2025-01-15 13:07:12 +01:00
Louis Chemineau
b1bc793940
fix: Remove duplicate propfind call
This also was an XHR call which is not going through the end to end encryption proxy

Signed-off-by: Louis Chemineau <louis@chmn.me>
2025-01-15 11:20:01 +01:00