Because access() is only called when handlers are initialised. If it failed here the handler would be removed from the view altogether. This needs to use the current node to determine the access, as well as user permissions.

I spoke to catch about this, we need to add back access. We have two levels here, whether we can view the node or not and whether we have a revision to access etc.. We need to check user_access still, as probably node_access() to determine if we even show a view link. So I think the current logic to revision linking is ok, we just need to add back the access stuff in some form. However, probably not in the access() method like before?

access() is only called on handler initialization, so the access check was moved to renderLink(). My understanding is that we need a better place for the access check, so that preRender doesn't get called?

Included interdiff to show what was changed in the view. Keep in mind that the test is failing (but not necessarily failing for the reasons that it should.) So, I still haven't tried to verify that the bug is still relevant. For example, the first fail is "Link containing href node/1 found." But, I think that we are expecting that part should work (even without the fix from this patch.)