Role permissions proposal: researcher can see all items

Our idea of the user researcher is to be able to see all the elements even the unpublished ones.
In our opinion it could be useful to have a registered user to share with colleagues to see all the items in the database and how right it is not to make any changes (you could once logged in go directly to the default site without entering the backend).

To test this functionality we simply added “view-all” in the file:
/var/www/html/omeka-s/application/src/Service/AclFactory.php

....
....
$acl->allow(
  ‘researcher’,
  ‘Omeka\Entity\User’,
  ‘read’
);
$acl->allow(
  ‘researcher’,
  ‘Omeka\Entity\Resource’,
  ‘view-all’
 );
....
....

Therefore we submit our idea to slightly change the policy of the research user type to give the possibility for this kind of users to see unpublished items as well.