Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to restrict scan to certain folder (recursively) #270

Open
icetear opened this issue Jan 23, 2020 · 8 comments
Open

Add option to restrict scan to certain folder (recursively) #270

icetear opened this issue Jan 23, 2020 · 8 comments

Comments

@icetear
Copy link

icetear commented Jan 23, 2020

The maps:scan-photos command actually runs through all the user files which is definitely not wanted when you have not only photos in your userspace. An option to restrict the scan to a certain folder ist needed. This is true for maps:scan-tracks also (because .gpx is not unique to .gpx Tracks, but also to GuitarPro files for example).

@tacruc
Copy link
Collaborator

tacruc commented Jan 23, 2020

Would something like work #228?
Limiting the media scam is just the initql scam after that changes are monitored.

@icetear
Copy link
Author

icetear commented Jan 23, 2020

It would be much better design to restrict the scanning itself and not needing to control it via an additional file. I can think of a bunch of constellations where folders containing images are created automatically without user interaction. Think of project directories with media assets or think of external storage where you have only read-access.

@tacruc
Copy link
Collaborator

tacruc commented Jan 23, 2020

Well let's stay consistent with other apps like photos.
@nextcloud/photos how do you handle this?

Let's also merge #271 to this discussion as it addresses the same issue or?

@tacruc
Copy link
Collaborator

tacruc commented Jan 23, 2020

From #271
@icetear:

In order to skip scanning hidden files (e.g. thumbnails and autogenerated system files) and to control what kind of images need to be scanned, a regexp restriction as a scanning pattern would be very handy.

@szaimen
Copy link
Contributor

szaimen commented Jan 23, 2020

what about occ maps:scan-photos --path=/files/userid/folder ?
or occ maps:scan-photos userid ?

@jancborchardt
Copy link
Member

We have a very related issue in the Photos app at:
Limit Photos to a single folder + subtree nextcloud/photos#141

We should probably coordinate and make sure the Maps app can take the same location that is specified in the Photos app. (At least for photos, tracks etc are separate.)

cc @skjnldsv @Mikescops for Photos

@vascowhite
Copy link

vascowhite commented Feb 20, 2020

I came here looking for this option.

I think it would be useful to be able to have several folders scanned as I have photographs and images in several album folders. Some I would want to show on the map, but not others.

I hope that's clear. I'll try and clarify further if not.

I should add that this is a fantastic addition to Nextcloud, I love it!! Thanks for developing it.

@yunyuyuan
Copy link

Maps scanned my external storage, where a large number of material photos of games are stored.🤣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants