If no other platforms exist other than the main aegir platform, the batch migrate form still renders the Submit button but of course the rest of the form is empty.

Submitting the form gives the error 'Platform field is required'. We need some more checking to see if any other platforms actually even exist before allowing the form to be submit.

Screenshot attached

CommentFileSizeAuthor
#2 hosting_migrate_batch.patch418 bytesunivate
migrate.png61.39 KBmig5

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

Fixed in HEAD.

univate’s picture

univate commented
Status: Fixed » Needs review
StatusFileSize
new hosting_migrate_batch.patch418 bytes

there is '}' missing 

diff --git migrate/hosting_migrate.batch.inc migrate/hosting_migrate.batch.inc
index 92c688a..8d650b8 100644
--- migrate/hosting_migrate.batch.inc
+++ migrate/hosting_migrate.batch.inc
@@ -37,6 +37,7 @@ function hosting_migrate_platform($form_state, $node) {
         '#type' => 'submit',
         '#value' => t('Submit'),
       );
+    }
     else {
       $form['no_platforms'] = array(
         '#type' => 'item',
Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Fixed

Gah! Sorry, I don't know how that happened.. it was in my working copy but somehow missing from my copy in my cvs checkout.. weird.

Sorry about that. Fixed. And thanks :(

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.