Added fileID and record variables to onBulkUpload hook (initial PR)

[mooror]

This PR adds the $fileID and $record variables to the "onBulkUpload" extension point that is in the createDataObject() method (in BulkUploadHandler.php). This allows developers to easily set additional fields on the newly created records and use data from the uploaded file to do so.

See the related issue for a usage example

Description

Manual testing steps

Issues

• Enhance the "onBulkUpload" extension point so new DataObjects can be altered #296

Pull request checklist

• The target branch is correct
  • See picking the right version
• All commits are relevant to the purpose of the PR (e.g. no debug statements, unrelated refactoring, or arbitrary linting)
  • Small amounts of additional linting are usually okay, but if it makes it hard to concentrate on the relevant changes, ask for the unrelated changes to be reverted, and submitted as a separate PR.
• The commit messages follow our commit message guidelines
• The PR follows our contribution guidelines
• Code changes follow our coding conventions
• This change is covered with tests (or tests aren't necessary for this change)
• Any relevant User Help/Developer documentation is updated; for impactful changes, information is added to the changelog for the intended release
• CI is green

[GuySartorelli]

There are a lot of commits on here that aren't related to your change, which are pull in from a merge.
As discussed on slack, please do one of the following:

1. Close this PR, and start again using the 4 branch as a base
2. reset your commits

[GuySartorelli]

Please also check that you have read the checkbox information and applied it correctly - for example you've ticked "CI is green" but CI hasn't even run. This makes it hard to trust that you've even looked at any of the information in the checkboxes.

[mooror]

Oh I thought the "This branch has no conflicts with the base branch" was apart of the CI.

For future reference, how can I tell when the CI has run and if its green?

[GuySartorelli]

Right now you'll see something like this (not exactly the same but it should tell you something's awaiting approval)

When CI has passed, it'll literally be green and say "All checks have passed"

The favicon should also have a green tick when CI has passed and is green.

[mooror]

Oh Okay thank you

[GuySartorelli]

@mooror Just a reminder to please fix this PR as per #297 (comment)

[GuySartorelli]

@mooror FYI we're tagging the beta for the next minor release next week - if this doesn't get merged before then it'll be another 6 months before the next release.

[mooror]

Oh thanks for the reminders. I will try to get this done today

[mooror]

Replaced by #302

[mooror]

#302 had a commit that was named incorrectly, so I changed it and created a new PR (#303)