fix: don't set code for None epsg (stac-utils#1544)

* fix: don't set code for None epsg

* chore: update changelog

* Update pystac/extensions/projection.py

Co-authored-by: Julia Signell <[email protected]>

---------

Co-authored-by: Julia Signell <[email protected]>

Author: 2 people

CHANGELOG.md

@@ -7,6 +7,8 @@
 - `Collection.from_items` for creating a `pystac.Collection` from an `ItemCollection` ([#1522](https://github.com/stac-utils/pystac/pull/1522))
 
 ### 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