dns: worldwidedns: Leverage "slot" ID from Record.id

The update_record() and delete_record() functions require a Record
instance in their respective argument lists to identify the DNS record
to manipulate. Record objects instantiated by this driver contain the
assocated "slot" ID within the .id field, so:

1. There is no need for the user to specify the "slot" ID via
   .update_record(..., extra={"entry": ID}). However, since it was
   previously required, an "entry" key will be accepted as long as it is
   consistent with the Record instance.
2. There is no need for .delete_record() to search the Record's zone for
   the "slot" ID of the Record.

Author: Aaron Sierra

libcloud/dns/drivers/worldwidedns.py

@@ -262,15 +262,15 @@ def update_record(self, record, name, type, data, extra=None):
         :param data: Data for the record (depends on the record type).
         :type  data: ``str``
 
-        :param extra: Contains 'entry' Entry position (1 thru 40)
+        :param extra: Extra attributes (driver specific). (optional).
         :type extra: ``dict``
 
         :rtype: :class:`Record`
         """
 
-        if (extra is None) or ("entry" not in extra):
-            raise WorldWideDNSError(value="You must enter 'entry' parameter", driver=self)
-        record_id = extra.get("entry")
+        record_id = record.id
+        if "entry" in extra and str(extra["entry"]) != record_id:
+            raise WorldWideDNSError(value="Inconsistent 'entry' parameter", driver=self)
 
         if name == "":
             name = "@"
@@ -437,13 +437,7 @@ def delete_record(self, record):
         :rtype: ``bool``
         """
         zone = record.zone
-
-        for index in range(MAX_RECORD_ENTRIES):
-            if record.name == zone.extra["S%s" % (index + 1)]:
-                entry = index + 1
-
-                break
-        extra = {"S%s" % entry: "", "T%s" % entry: "NONE", "D%s" % entry: ""}
+        extra = {"S%s" % record.id: "", "T%s" % record.id: "NONE", "D%s" % record.id: ""}
         self.update_zone(zone, zone.domain, extra=extra)
 
         return True