Update: Remove checkContentAccess in favour of generic _access

[taylortom]

Context

Part of the generic _access mechanism — see adapt-security/adapt-authoring-api#98.

Summary

Remove the hardcoded checkContentAccess method and the content.accessCheckHook.tap() call from init(). The access checks it currently performs are replaced by:

• _access.public — replaces _isShared (handled by api module via enableAccessControl)
• _access.users — replaces _shareWithUsers (handled by users module)
• createdBy ownership — handled by authored module

Data migration

A migration script is needed to convert existing course documents:

• _isShared: true → _access.public: true
• _shareWithUsers: [...] → _access.users: [...]
• Remove old _isShared and _shareWithUsers fields

Dependencies

• Requires New: Add generic _access mechanism to AbstractApiModule adapt-authoring-api#98
• Requires adapt-security/adapt-authoring-authored createdBy access check
• Requires adapt-security/adapt-authoring-users _access.users/_access.groups support