fix(Thumbnails): Use local id rather than remote id

[joshtrichards]

This should the scenarios described in #13743
May also fix #13515 and #13989

It's a regression from #13164 where the wrong DAV property (id / remote id) was used instead of fileid / local id) for some thumbnail retrievals.

Looks like the bug made its way into >= 3.30.0 + (via backport) 3.29.3 aka "3.29.2"

• Tests written, or not not needed

[joshtrichards]

/backport to stable-3.29

[joshtrichards]

/backport to stable-3.30

[alperozturk96]

Didn't notice anything wrong. Thumbnails are generated without any issue.

[ZetaTom]

In my testing, I haven't been able to reproduce the problem with this fix in place.

Thank you for fixing this!

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	59	59
Errors	3	3

SpotBugs

Category	Base	New
Bad practice	65	65
Correctness	58	58
Dodgy code	298	298
Experimental	1	1
Internationalization	7	7
Malicious code vulnerability	1	1
Multithreaded correctness	7	7
Performance	53	53
Security	18	18
Total	508	508

[backportbot]

The backport to stable-3.29 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable-3.29
git pull origin stable-3.29

# Create the new backport branch
git checkout -b backport/14081/stable-3.29

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

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/14081/stable-3.29

Error: Failed to create pull request: Validation Failed: {"resource":"PullRequest","field":"base","code":"invalid"}

---

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