Updated docstrings on ExploitTarget

Bryan Worrell · Bryan Worrell · commit d167928e6d03 · 2015-05-14T08:19:56.000-04:00
diff --git a/stix/exploit_target/__init__.py b/stix/exploit_target/__init__.py
@@ -59,18 +59,17 @@ def __init__(self, id_=None, idref=None, timestamp=None, title=None,
  
     @property
     def vulnerabilities(self):
-        """A list of ``Vulnerability`` objects 
+        """A collection of :class:`.Vulnerability` objects. This behaves like
+        a ``MutableSequence`` type.
         
         Default Value: ``None``
 
         Returns:
-            A list of
-            :class:`stix.exploit_target.vulnerability`
+            A list of :class:`.Vulnerability`
 
         Raises:
             ValueError: If set to a value that is not ``None`` and not an
-                instance of
-                :class:`stix.exploit_target.vulnerability`
+                instance of :class:`.Vulnerability`
 
         """
         
@@ -80,74 +79,69 @@ def vulnerabilities(self):
     def vulnerabilities(self, value):
         self._vulnerabilities = _Vulnerabilities(value)
 
-    def add_vulnerability(self, v):
-        """Adds a vulnerability to the ``vulnerabilies`` list property.
+    def add_vulnerability(self, value):
+        """Adds a vulnerability to the :attr:`vulnerabilities` list property.
 
         Note:
             If ``None`` is passed in no value is added 
 
         Args:
-            v: A Vulnerability value.
+            value: A :class:`.Vulnerability` object..
             
-        Raises: ValueError if the ``v`` param is of type
-            :class:`stix.exploit_target.vulnerability`
-    
+        Raises:
+            ValueError: if the `value` param is of type :class:`.Vulnerability`
 
         """
-        self.vulnerabilities.append(v)
+        self.vulnerabilities.append(value)
  
     @property
     def weaknesses(self):
-        """A list of ``Weakness`` objects 
+        """A collection of :class:`.Weakness` objects. This behaves like
+        a ``MutableSequence`` type.
         
         Default Value: ``None``
 
         Returns:
-            A list of
-            :class:`stix.exploit_target.weakness`
+            A list of :class:`.Weakness` objects.
 
         Raises:
             ValueError: If set to a value that is not ``None`` and not an
-                instance of
-                :class:`stix.exploit_target.weakness`
+                instance of :class:`.Weakness`
 
         """
-        
         return self._weaknesses
     
     @weaknesses.setter
     def weaknesses(self, value):
         self._weaknesses = _Weaknesses(value)
             
-    def add_weakness(self, v):
-        """Adds a weakness to the ``weaknesses`` list property.
+    def add_weakness(self, value):
+        """Adds a weakness to the :attr:`weaknesses` list property.
 
         Note:
             If ``None`` is passed in no value is added 
 
         Args:
-            v: A weakness value.
+            value: A :class:`.Weakness` object.
             
-        Raises: ValueError if the ``v`` param is of type :class:`stix.exploit_target.weakness`
-    
+        Raises: ValueError if the `value` param is of type :class:`.Weakness`
 
         """
-        self.weaknesses.append(v)
+        self.weaknesses.append(value)
  
     @property
     def configuration(self):
-        """A list of ``Configuration`` objects 
+        """A list of :class:`.Configuration` objects. This behaves like
+        a ``MutableSequence`` type.
         
         Default Value: ``None``
 
         Returns:
-            A list of
-            :class:`stix.exploit_target.configuration`
+            A list of :class:`.Configuration` objects.
 
         Raises:
             ValueError: If set to a value that is not ``None`` and not an
-                instance of
-                :class:`stix.exploit_target.configuration`
+                instance of :class:`.Configuration`.
 
         """
         
@@ -157,19 +151,20 @@ def configuration(self):
     def configuration(self, value):
         self._configuration = _Configurations(value)
 
-    def add_configuration(self, v):
-        """Adds a configuration to the ``configurations`` list property.
+    def add_configuration(self, value):
+        """Adds a configuration to the :attr:`configurations` list property.
 
         Note:
             If ``None`` is passed in no value is added 
 
         Args:
-            v: A configuration value.
+            value: A configuration value.
             
-        Raises: ValueError if the ``v`` param is of type :class:`stix.exploit_target.configuration`
+        Raises:
+            ValueError: If the `value` param is of type :class:`.Configuration`
 
         """
-        self.configuration.append(v)
+        self.configuration.append(value)
  
     def to_obj(self, return_obj=None, ns_info=None):
         if not return_obj:
