Should be using entity_uri to get uri's instead of building paths.

With the use of modules that set a custom uri callback the URLs are not correctly built. Instead of manually building the path with the switch statement for entity type the module should check the entity type and then call entity_uri to get the path.

I don't have the time to verify my edits don't have other adverse effects yet once I do I will see if I can submit a patch but it might be a while if someone else wants to tackle these changes.