From aec09ed8d20a933809f0a00ca20d3985d4f9bafd Mon Sep 17 00:00:00 2001
From: Olivier Lambert <olivier.lambert@vates.fr>
Date: Tue, 17 Nov 2015 14:49:05 +0100
Subject: [PATCH] Rename vm.bootOrder() to vm.setBootOrder() and ensure VM is
 HVM

---
 src/api/vm.coffee | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/api/vm.coffee b/src/api/vm.coffee
index 99a2f5457..63090423a 100644
--- a/src/api/vm.coffee
+++ b/src/api/vm.coffee
@@ -891,26 +891,26 @@ stats.resolve = {
 exports.stats = stats;
 
 #---------------------------------------------------------------------
-
-# TODO: rename to setBootOrder
-# TODO: check current VM is HVM
-bootOrder = $coroutine ({vm, order}) ->
+setBootOrder = $coroutine ({vm, order}) ->
   xapi = @getXAPI vm
 
   order = {order: order}
+  if vm.virtualizationMode == 'hvm'
+    yield xapi.call 'VM.set_HVM_boot_params', vm._xapiRef, order
+    return true
 
-  yield xapi.call 'VM.set_HVM_boot_params', vm._xapiRef, order
+  @throw(
+    'INVALID_PARAMS'
+    'You can only set the boot order on a HVM guest'
+  )
 
-  return true
-
-
-bootOrder.params = {
+setBootOrder.params = {
   vm: { type: 'string' },
   order: { type: 'string' }
 }
 
-bootOrder.resolve = {
+setBootOrder.resolve = {
   vm: ['vm', 'VM', 'operate'],
 }
-exports.bootOrder = bootOrder
+exports.setBootOrder = setBootOrder
 #---------------------------------------------------------------------