Merge pull request #75 from cobbler/fix/cli-issues

Fix client issues

Author: SchoolGuy

cobblerclient.go

@@ -415,6 +415,14 @@ func (c *Client) updateCobblerFields(what string, item reflect.Value, id string)
 			}
 			break
 		}
+
+		if fieldType == "Resource" {
+			err := c.updateCobblerFields(what, reflect.ValueOf(v.Interface()), id)
+			if err != nil {
+				return err
+			}
+			break
+		}
 	}
 
 	// Fields that can inherit from other items can only be set after the parent is set.
@@ -474,7 +482,7 @@ func (c *Client) updateCobblerFields(what string, item reflect.Value, id string)
 		field := tag.Get("mapstructure")
 		cobblerTag := tag.Get("cobbler")
 
-		if cobblerTag == "noupdate" || fieldType == "Item" || fieldType == "Meta" {
+		if cobblerTag == "noupdate" || fieldType == "Item" || fieldType == "Resource" || fieldType == "Meta" {
 			continue
 		}
 

fixtures/create-interface-create-interface-req.xml

@@ -34,7 +34,7 @@
                         </value>
                     </member>
                     <member>
-                        <name>ConnectedMode-eth0</name>
+                        <name>connected_mode-eth0</name>
                         <value>
                             <boolean>0</boolean>
                         </value>

fixtures/modify-interface-req.xml

@@ -38,7 +38,7 @@
                         </value>
                     </member>
                     <member>
-                        <name>ConnectedMode-eth0</name>
+                        <name>connected_mode-eth0</name>
                         <value>
                             <boolean>0</boolean>
                         </value>

image.go

@@ -167,7 +167,7 @@ type Image struct {
 	VirtPath             string         `mapstructure:"virt_path"`
 	VirtRam              Value[int]     `mapstructure:"virt_ram"`
 	VirtType             string         `mapstructure:"virt_type"`
-	SupportedBootLoaders []string       `mapstructure:"supported_boot_loaders"`
+	SupportedBootLoaders []string       `mapstructure:"supported_boot_loaders" cobbler:"noupdate"`
 
 	Client
 }

system.go

@@ -78,7 +78,7 @@ type Interface struct {
 	BondingOpts        string   `mapstructure:"bonding_opts" structs:"bonding_opts"`
 	BridgeOpts         string   `mapstructure:"bridge_opts" structs:"bridge_opts"`
 	CNAMEs             []string `mapstructure:"cnames" structs:"cnames"`
-	ConnectedMode      bool     `mapstructure:"connected_mode"`
+	ConnectedMode      bool     `mapstructure:"connected_mode" structs:"connected_mode"`
 	DHCPTag            string   `mapstructure:"dhcp_tag" structs:"dhcp_tag"`
 	DNSName            string   `mapstructure:"dns_name" structs:"dns_name"`
 	Gateway            string   `mapstructure:"if_gateway" structs:"if_gateway"`