Merge branch 'main' into main

Author: gadomski

CHANGELOG.md

@@ -11,6 +11,8 @@
 - fixed missing parameter "title" in pystac.extensions.classification.Classification ([#1539](https://github.com/stac-utils/pystac/pull/1539))
 
 ### Fixed
+
+- `proj:epsg` migration when `None` ([#1544](https://github.com/stac-utils/pystac/pull/1544))
 - fixed missing parameter "title" in pystac.extensions.classification.Classification ([#1539](https://github.com/stac-utils/pystac/pull/1539))
 
 ## [v1.12.2]

pystac/extensions/projection.py

@@ -477,10 +477,8 @@ def migrate(
             return
 
         # proj:epsg moved to proj:code
-        if "proj:epsg" in obj["properties"]:
-            epsg = obj["properties"]["proj:epsg"]
+        if epsg := obj["properties"].pop("proj:epsg", None):
             obj["properties"]["proj:code"] = f"EPSG:{epsg}"
-            del obj["properties"]["proj:epsg"]
 
         super().migrate(obj, version, info)
 

tests/extensions/test_projection.py

@@ -662,3 +662,15 @@ def test_v1_crs_string() -> None:
     item = pystac.Item.from_dict(data, migrate=False)
     assert item.ext.proj.epsg is not None
     assert item.ext.proj.crs_string == "EPSG:32617"
+
+
+def test_none_epsg(item: Item) -> None:
+    # https://github.com/stac-utils/pystac/issues/1543
+    d = item.to_dict(include_self_link=False, transform_hrefs=False)
+    d["stac_version"] = "1.0.0"
+    d["stac_extensions"] = [
+        "https://stac-extensions.github.io/projection/v1.1.0/schema.json"
+    ]
+    d["properties"]["proj:epsg"] = None
+    item = Item.from_dict(d, migrate=True)
+    assert item.ext.proj.code is None