Refactor vlan edit functionality.

Author: Fernando Ojeda

SoftLayer/CLI/vlan/edit.py

@@ -22,17 +22,14 @@
 def cli(env, identifier, name, note, tags):
     """Edit a vlan's details."""
 
-    data = {
-        'name': name,
-        'note': note
-    }
+    new_tags = None
 
     if tags:
-        data['tags'] = ','.join(tags)
+        new_tags = ','.join(tags)
 
     mgr = SoftLayer.NetworkManager(env.client)
     vlan_id = helpers.resolve_id(mgr.resolve_vlan_ids, identifier, 'VLAN')
-    vlan = mgr.edit(vlan_id, **data)
+    vlan = mgr.edit(vlan_id, name=name, note=note, tags=new_tags)
 
     if vlan:
         click.secho("Vlan edited successfully", fg='green')

SoftLayer/managers/network.py

@@ -698,7 +698,7 @@ def edit(self, instance_id, name=None, note=None, tags=None):
 
         :param integer instance_id: the instance ID to edit.
         :param string name: valid name.
-        :param string note: note about this particular v;am.
+        :param string note: note about this particular vlan.
         :param string tags: tags to set on the vlan as a comma separated list.
                             Use the empty string to remove all tags.
         :returns: bool -- True or an Exception