Fix legacy module redirects

Legacy module redirects the old style URLs like "node/view/" to "node/" by using drupal_goto as menu callback. But the 4th parameter to drupal_goto was missing causing. Thanks to the evil menu magic, the provided was past as last argument to drupal_goto which results in an internal server error.

for example: http://drupal.org/node/view/148974 - You'll see that it does set the Location: header but instead of sending one of the redirect headers, it sends a 500 internal server error header.

Patch attached which sets the correct HTTP response code (permanent redirect).