Infinite redirect loop when submitting video form without video (unsupported type)

Too many redirects occurred trying to open “http://example.com/just-title”. This might occur if you open a page that is redirected to open another page which then is redirected to open the original page.

Steps to reproduce:

1. Put in a value for the many required fields, but
2. Leave both the video upload form and the link to file/YouTube fields blank (they aren't technically required).
3. Press submit and watch your browser beat its head against your server.

I hardly feel like attaching an actual patch, as the current development version of the module is not public, but here it is.

Line 979 simply shouldn't be there: drupal_goto("node/$node->nid");
in:

    default:
      drupal_set_message('Video type not supported', 'error');
      drupal_goto("node/$node->nid");
      break;

There should also be a patch to make sure at least one form of video is being uploaded or linked to, but that's another issue.

Bigger picture: This is a great, important project that may have to be forked, temporarily, if comaintainers are not appointed. I'll get the namespace video_desperation if people want to apply patches to it!

(Our WSF2008 dev site has a vastly simplified video form, done through form_alter, but this bug was also tested on a clean install.)

^{benjamin, Agaric Design Collective}