Allow using "limit" in get_objs_recursively() if md5 files are included

bihealth / sodar-server

SODAR: System for Omics Data Access and Retrieval

https://github.com/bihealth/sodar-server

MIT License

14 stars 3 forks source link

Allow using "limit" in get_objs_recursively() if md5 files are included #1887

Open mikkonie opened 5 months ago

mikkonie commented 5 months ago

While doing #1883 I noticed that in get_objs_recursively(), the limit arg is ignored if the md5 option is enabled. This makes no sense.

After the recent changes in #1883, it would make sense to use limit but double it if also returning .md5 files. However, I'll have to go through current usages of this method and see what makes sense.

mikkonie commented 5 months ago

Postponing this to v1.0 as it's not really urgently needed at this moment. This will most likely change when doing #1881.