[Ovirt-devel] [PATCH] Fixing a loading issue with the Start On form

[Arthur Clément]

Pushed

On mardi 21 septembre 2010 18:04:52 Arthur Clément wrote:
> ACK
> 
> On mardi 21 septembre 2010 17:20:26 Simon COURTOIS wrote:
> > Signed-off-by: Simon COURTOIS <scourtois at linagora.com>
> > ---
> > 
> >  src/app/views/vm/start.rhtml |   76
> > 
> > +++++++++++++++++++++-------------------- 1 files changed, 39
> > insertions(+), 37 deletions(-)
> > 
> > diff --git a/src/app/views/vm/start.rhtml b/src/app/views/vm/start.rhtml
> > index bed1d2d..f80cf70 100644
> > --- a/src/app/views/vm/start.rhtml
> > +++ b/src/app/views/vm/start.rhtml
> > @@ -4,6 +4,45 @@
> > 
> >  <%- content_for :description do -%>
> >  
> >    Please choose a Host to start the Virtual Machine. Leave the selection
> > 
> > blank to allow oVirt to choose the most appropriate starting host. <%-
> > end -%>
> > +
> > +<script type="text/javascript">
> > +  $(function() {
> > +
> > +    $('#start_vm_form').ajaxForm({
> > +      dataType: 'json',
> > +      success: afterVm
> > +    });
> > +
> > +  });
> > +
> > +  function start_vm_select(selected_rows) {
> > +    var selected_ids = new Array();
> > +    for (i=0; i<selected_rows.length; i++) {
> > +      load_widget_select(selected_rows[i]);
> > +      selected_ids[i] = selected_rows[i].id;
> > +    }
> > +    if (selected_ids.length == 1) {
> > +      $('#selected_starting_host')
> > +        .load('<%= url_for :controller => "host", :action =>
> > "quick_summary" %>', +          { id:
> > parseInt(selected_ids[0].substring(3)) });
> > +      $('#vm_action_data').val(selected_ids[0].substring(3));
> > +    }
> > +  }
> > +
> > +  function start_vm_deselect(selected_rows) {
> > +    var selected_ids = new Array()
> > +    for (i=0; i<selected_rows.length; i++) {
> > +      load_widget_deselect(selected_rows[i]);
> > +      selected_ids[i] = selected_rows[i].id;
> > +    }
> > +    refresh_summary_static('selected_starting_host',
> > +      '<div class="selection_left"> \
> > +        <div>No starting host selected.</div> \
> > +      </div>');
> > +    $('#vm_action_data').val('');
> > +  }
> > +</script>
> > +
> > 
> >  <div class="panel_header"></div>
> >  
> >  <form method="POST" id="start_vm_form" action="<%= url_for :action =>
> > 
> > 'vm_action' %>"> @@ -38,40 +77,3 @@
> > 
> >    </div>
> >    <%= popup_footer("$('#start_vm_form').submit()", "Start Virtual
> > 
> > Machine") %> </form>
> > -<script type="text/javascript">
> > -  $(function() {
> > -
> > -    $('#start_vm_form').ajaxForm({
> > -      dataType: 'json',
> > -      success: afterVm
> > -    });
> > -
> > -  });
> > -
> > -  function start_vm_select(selected_rows) {
> > -    var selected_ids = new Array();
> > -    for (i=0; i<selected_rows.length; i++) {
> > -      load_widget_select(selected_rows[i]);
> > -      selected_ids[i] = selected_rows[i].id;
> > -    }
> > -    if (selected_ids.length == 1) {
> > -      $('#selected_starting_host')
> > -        .load('<%= url_for :controller => "host", :action =>
> > "quick_summary" %>', -          { id:
> > parseInt(selected_ids[0].substring(3)) });
> > -      $('#vm_action_data').val(selected_ids[0].substring(3));
> > -    }
> > -  }
> > -
> > -  function start_vm_deselect(selected_rows) {
> > -    var selected_ids = new Array()
> > -    for (i=0; i<selected_rows.length; i++) {
> > -      load_widget_deselect(selected_rows[i]);
> > -      selected_ids[i] = selected_rows[i].id;
> > -    }
> > -    refresh_summary_static('selected_starting_host',
> > -      '<div class="selection_left"> \
> > -        <div>No starting host selected.</div> \
> > -      </div>');
> > -    $('#vm_action_data').val('');
> > -  }
> > -</script>
-- 
Arthur CLEMENT
Linagora Paris