fix: correctly scopes data variable within bulk update - #2901

[JarrodMFlesch]

Description

Fixes #2901

The issue was caused by data being stored outside of the promises. This lead to stale data being set and accessed when setting/passing data within the bulk update operation.

I also removed the code related to files. I found it counterintuitive to even think about files when bulk updating an upload enabled collection. The admin panel does not allow upload enabled collections to replace all files with one file when performing a bulk update. The code removal aligns the API with the admin panel UI/UX, as it would be strange to replace all files with one file.

• I have read and understand the CONTRIBUTING.md document in this repository

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• I have added tests that prove my fix is effective or that my feature works
• Existing test suite passes locally with my changes
• I have made corresponding changes to the documentation