Rename Asset.properties to Asset.extra_fields

Author: duckontheweb

pystac/asset.py

@@ -1,7 +1,6 @@
 from copy import copy
 from typing import Any, Dict, List, Optional, TYPE_CHECKING, Union
 
-import pystac
 from pystac.utils import is_absolute_href, make_absolute_href
 
 if TYPE_CHECKING:
@@ -24,7 +23,7 @@ class Asset:
             are preferred. See :class:`~pystac.MediaType` for common media types.
         roles : Optional, Semantic roles (i.e. thumbnail, overview,
             data, metadata) of the asset.
-        properties : Optional, additional properties for this asset. This is used
+        extra_fields : Optional, additional fields for this asset. This is used
             by extensions as a way to serialize and deserialize properties on asset
             object JSON.
     """
@@ -48,12 +47,12 @@ class Asset:
     """Optional, Semantic roles (i.e. thumbnail, overview, data, metadata) of the
     asset."""
 
-    owner: Optional[Union[pystac.Item, pystac.Collection]]
+    owner: Optional[Union["Item_Type", "Collection_Type"]]
     """The :class:`~pystac.Item` or :class:`~pystac.Collection` that this asset belongs
     to, or ``None`` if it has no owner."""
 
-    properties: Optional[Dict[str, Any]]
-    """Optional, additional properties for this asset. This is used by extensions as a
+    extra_fields: Dict[str, Any]
+    """Optional, additional fields for this asset. This is used by extensions as a
     way to serialize and deserialize properties on asset object JSON."""
 
     def __init__(
@@ -63,18 +62,14 @@ def __init__(
         description: Optional[str] = None,
         media_type: Optional[str] = None,
         roles: Optional[List[str]] = None,
-        properties: Optional[Dict[str, Any]] = None,
+        extra_fields: Optional[Dict[str, Any]] = None,
     ) -> None:
         self.href = href
         self.title = title
         self.description = description
         self.media_type = media_type
         self.roles = roles
-
-        if properties is not None:
-            self.properties = properties
-        else:
-            self.properties = {}
+        self.extra_fields = extra_fields or {}
 
         # The Item which owns this Asset.
         self.owner = None
@@ -125,8 +120,8 @@ def to_dict(self) -> Dict[str, Any]:
         if self.description is not None:
             d["description"] = self.description
 
-        if self.properties is not None and len(self.properties) > 0:
-            for k, v in self.properties.items():
+        if self.extra_fields is not None and len(self.extra_fields) > 0:
+            for k, v in self.extra_fields.items():
                 d[k] = v
 
         if self.roles is not None:
@@ -146,7 +141,7 @@ def clone(self) -> "Asset":
             description=self.description,
             media_type=self.media_type,
             roles=self.roles,
-            properties=self.properties,
+            extra_fields=self.extra_fields,
         )
 
     def __repr__(self) -> str:
@@ -175,5 +170,5 @@ def from_dict(d: Dict[str, Any]) -> "Asset":
             title=title,
             description=description,
             roles=roles,
-            properties=properties,
+            extra_fields=properties,
         )

pystac/extensions/datacube.py

@@ -391,7 +391,7 @@ class AssetDatacubeExtension(DatacubeExtension[pystac.Asset]):
 
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
         else:

pystac/extensions/eo.py

@@ -408,9 +408,9 @@ def _get_bands(self) -> Optional[List[Band]]:
         if bands is None:
             asset_bands: List[Dict[str, Any]] = []
             for _, value in self.item.get_assets().items():
-                if BANDS_PROP in value.properties:
+                if BANDS_PROP in value.extra_fields:
                     asset_bands.extend(
-                        cast(List[Dict[str, Any]], value.properties.get(BANDS_PROP))
+                        cast(List[Dict[str, Any]], value.extra_fields.get(BANDS_PROP))
                     )
             if any(asset_bands):
                 bands = asset_bands
@@ -454,7 +454,7 @@ def _get_bands(self) -> Optional[List[Band]]:
 
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
 

pystac/extensions/file.py

@@ -103,7 +103,7 @@ class FileExtension(PropertiesExtension, ExtensionManagementMixin[pystac.Item]):
 
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
 

pystac/extensions/item_assets.py

@@ -76,7 +76,7 @@ def create_asset(self, href: str) -> pystac.Asset:
             description=self.description,
             media_type=self.media_type,
             roles=self.roles,
-            properties={
+            extra_fields={
                 k: v
                 for k, v in self.properties.items()
                 if k

pystac/extensions/label.py

@@ -665,7 +665,7 @@ def add_labels(
         self.obj.add_asset(
             "labels",
             pystac.Asset(
-                href=href, title=title, media_type=media_type, properties=properties
+                href=href, title=title, media_type=media_type, extra_fields=properties
             ),
         )
 

pystac/extensions/pointcloud.py

@@ -552,7 +552,7 @@ def __repr__(self) -> str:
 class AssetPointcloudExtension(PointcloudExtension[pystac.Asset]):
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
             self.repr_id = f"href={asset.href} item.id={asset.owner.id}"

pystac/extensions/projection.py

@@ -314,7 +314,7 @@ class AssetProjectionExtension(ProjectionExtension[pystac.Asset]):
 
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
 

pystac/extensions/raster.py

@@ -647,7 +647,7 @@ class RasterExtension(PropertiesExtension, ExtensionManagementMixin[pystac.Item]
 
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]
 

pystac/extensions/sar.py

@@ -334,7 +334,7 @@ def __repr__(self) -> str:
 class AssetSarExtension(SarExtension[pystac.Asset]):
     def __init__(self, asset: pystac.Asset):
         self.asset_href = asset.href
-        self.properties = asset.properties
+        self.properties = asset.extra_fields
         if asset.owner and isinstance(asset.owner, pystac.Item):
             self.additional_read_properties = [asset.owner.properties]