Posted by mig5 on October 2, 2009 at 1:08am
| Project: | Hosting |
| Version: | 6.x-0.4-alpha1 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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
| Attachment | Size |
|---|---|
| migrate.png | 61.39 KB |
Comments
#1
Fixed in HEAD.
#2
there is '}' missing
<?phpdiff --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',
?>
#3
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 :(
#4
Automatically closed -- issue fixed for 2 weeks with no activity.