Ran into a problem with the GetUrl for commerce products.
We have the same setup as Paul in this post (scroll to bottom and read the last two entrys)
The issue is that currently you will always get the master catalog url.
So I propose a overload that takes a Catalog or Category to determent the current context for the product.
My blog post demonstrates how this can be achieved. We'll discuss if this could be a platform feature.