commons_wikis_menu_alter() alters illegitimate menu items

All this code looks for is routes that have the word revisions in them, but there could be any number of legitimate uses of that word that don't involve nodes. For example, the Revisions tabs on Commerce entities are defined via Views and have the word "revisions" in them. For obvious reasons, commons_wiki_user_revision_access() is going to fail - these are not nodes, and the entity isn't even passed in to the access callback.

I suppose this alter function should be adjusted specifically to look for node routes that are actually passing node entities to their access callbacks (presumably validated by exploding the route on /, looking for %node, and ensuring their position matches the parameter designated in the route's access arguments array). However, I think a more robust solution would involve selectively applying this alteration to known routes.

Just for reference / searchability, here are the error messages this bug generates:

• Notice: Trying to get property of non-object in commons_wikis_user_revision_access() (line 71 of /mnt/www/html/staffoutdooredcomdev/docroot/profiles/commons/modules/commons/commons_wikis/commons_wikis.module).
• Warning: Illegal offset type in node_access() (line 3006 of /mnt/www/html/staffoutdooredcomdev/docroot/modules/node/node.module).