pkp/pkp-lib#10431 Hooks in policy to relax author file upload

[touhidurabir]

for #10431 .

[asmecher]

Just a few minor changes here.

[asmecher]

I prefer named parameters for new hooks:

Suggested change

	Hook::run('SubmissionFileStageAccessPolicy::effect', [$submission, $userRoles, $stageAssignments, $this->_fileStage, $this->_action, &$assignedFileStages]);
	Hook::run('SubmissionFileStageAccessPolicy::effect', ['submission' => $submission, 'userRoles' => $userRoles, 'stageAssignments' => $stageAssignments, 'fileStage' => $this->_fileStage, 'action' => $this->_action, 'assignedFileStages' => &$assignedFileStages]);

This makes for a lot less ambiguity in the hook callback functions, which would otherwise depend on function orders to know what parameters are being passed.

[asmecher]

I think this is incorrect, now that you're using Hook::run instead of Hook::call.

Suggested change

	* @hook SubmissionFileStageAccessPolicy::effect [[$submission, $userRoles, $stageAssignments, $this->_fileStage, $this->_action, &$assignedFileStages]]
	* @hook SubmissionFileStageAccessPolicy::effect [$submission, $userRoles, $stageAssignments, $this->_fileStage, $this->_action, &$assignedFileStages]

[asmecher]

As above

Suggested change

	* @hook SubmissionFileAccessPolicy::authorFileAccess [[$request, $mode, $submissionFileId, &$authorFileAccessOptionsPolicy]]
	* @hook SubmissionFileAccessPolicy::authorFileAccess [$request, $mode, $submissionFileId, &$authorFileAccessOptionsPolicy]

[asmecher]

As above

Suggested change

	Hook::run('SubmissionFileAccessPolicy::authorFileAccess', [$request, $mode, $submissionFileId, &$authorFileAccessOptionsPolicy]);
	Hook::run('SubmissionFileAccessPolicy::authorFileAccess', ['request' => $request, 'mode' => $mode, 'submissionFileId' => $submissionFileId, 'authorFileAccessOptionsPolicy' => &$authorFileAccessOptionsPolicy]);