fix(files_reminders): Join the reminders with the file system metadata

[solracsf]

• Resolves: [Bug]: More than 1000 expressions in a list are not allowed on Oracle. (postgresql, in files_reminders / cacheFolder) #53527

Eliminates the need to load fileIds into PHP by using a database INNER JOIN with the oc_filecache table, filtering files by their parent (the folder's ID) directly in the SQL query. This avoids the IN clause entirely

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[provokateurin]

Thanks, this looks much better and should fix #53527

[provokateurin]

@solracsf any reason to keep it as a draft?

[solracsf]

I'll be on holiday for the next 3 weeks, but feel free to open/review.

[provokateurin]

/backport to stable31

[provokateurin]

/backport to stable30

[backportbot]

The backport to stable30 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable30
git pull origin stable30

# Create the new backport branch
git checkout -b backport/53548/stable30

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 4eb357e9

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/53548/stable30

Error: Failed to check for changes with origin/stable30: No changes found in backport branch

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[provokateurin]

The stable30 backport must also include #53644!

[solracsf]

The stable30 backport must also include #53644!

findAllInFolder() is v31+