maps icon indicating copy to clipboard operation
maps copied to clipboard

Scan photos of specific directory

Open tetebueno opened this issue 10 months ago • 3 comments

I've been getting memory exhaustion when scanning all photos collection on an initial load; trying to address #525 I made these changes and was able to scan each year/month directory of photos succesfully like this (the example is for the folder with photos of july '84):

$ docker exec -t -u www-data container-name php occ maps:scan-photos -vvv username Photos/1984/06

I didn't like the ifs all over the implementation, but it's something like a make it work approach. If I can get comments about it, we can make it better.

Cheers.

tetebueno avatar Mar 27 '24 02:03 tetebueno

Thanks for your contribution. Can you pass the path as optional argument? Do you need help to fix the DCO?

tacruc avatar Mar 27 '24 17:03 tacruc

You are very welcome.

Done with DCO, thanks.

Is there anything else besides this so the path parameter is optional? I believe it already is since help command displays it like this: ... [<user_id> [<path>]].

tetebueno avatar Mar 28 '24 01:03 tetebueno

@tetebueno Do you think you will make the changes befor the NC29 release. Then I would wait with https://github.com/nextcloud/maps/pull/1244 to include this.

I'm sorry that it is on such short notice after not reviewing for a month. Just have to decide whether to wait or not.

tacruc avatar Apr 21 '24 19:04 tacruc