With that code, you're not redirecting to another page, you're only giving another portlet's id - there's no clue which page you'd like to be forwarding to.
I don't have an environment for a quick check available during the weekend (sorry), here's what you can try to do to find the solution yourself:
- Inspect the URL you're building - you'll see that it addresses the current page that you're on.
- Find a way to actually provide the name of the page you want to redirect to. Hint: The portlet you want to redirect to could be placed on many pages, or on none. You'll have to either dynamically find one of the pages that it's on (and hope it's the one you actually want) or configure it somehow.
- Also, architecturally I find it's a lot better to "just" link to a simple page, rather than to a portlet's URL on a target page. Otherwise you're coupling two distinct portlets quite closely together - changes to one might need to be reflected in the other. Rather use parameters (e.g. public render parameters) to communicate between different portlets - instead of addressing another portlet more or less directly.
I hope that this helps you find the solution yourself as I don't have an IDE available on this computer to provide sample code.