Refs #32230 -- Made LayerMapping support pathlib.Path.

django/contrib/gis/utils/layermapping.py

@@ -8,6 +8,7 @@
 """
 import sys
 from decimal import Decimal, InvalidOperation as DecimalInvalidOperation
+from pathlib import Path
 
 from django.contrib.gis.db.models import GeometryField
 from django.contrib.gis.gdal import (
@@ -93,7 +94,7 @@ class LayerMapping:
         argument usage.
         """
         # Getting the DataSource and the associated Layer.
-        if isinstance(data, str):
+        if isinstance(data, (str, Path)):
             self.ds = DataSource(data, encoding=encoding)
         else:
             self.ds = data

docs/ref/contrib/gis/layermapping.txt

@@ -142,6 +142,10 @@ Keyword Arguments
                        Default is ``'default'``.
 =====================  =====================================================
 
+.. versionchanged:: 3.2
+
+    Support for :class:`pathlib.Path` ``data_source`` was added.
+
 ``save()`` Keyword Arguments
 ----------------------------
 

docs/releases/3.2.txt

@@ -117,6 +117,9 @@ Minor features
 * The :class:`~django.contrib.gis.gdal.DataSource` class now supports
   :class:`pathlib.Path`.
 
+* The :class:`~django.contrib.gis.utils.LayerMapping` class now supports
+  :class:`pathlib.Path`.
+
 :mod:`django.contrib.messages`
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 

tests/gis_tests/layermap/tests.py

@@ -1,8 +1,8 @@
 import datetime
-import os
 import unittest
 from copy import copy
 from decimal import Decimal
+from pathlib import Path
 
 from django.conf import settings
 from django.contrib.gis.gdal import DataSource
@@ -19,12 +19,12 @@ from .models import (
     has_nulls_mapping, inter_mapping,
 )
 
-shp_path = os.path.realpath(os.path.join(os.path.dirname(__file__), os.pardir, 'data'))
+shp_path = Path(__file__).resolve().parent.parent / 'data'
-city_shp = os.path.join(shp_path, 'cities', 'cities.shp')
+city_shp = shp_path / 'cities' / 'cities.shp'
-co_shp = os.path.join(shp_path, 'counties', 'counties.shp')
+co_shp = shp_path / 'counties' / 'counties.shp'
-inter_shp = os.path.join(shp_path, 'interstates', 'interstates.shp')
+inter_shp = shp_path / 'interstates' / 'interstates.shp'
-invalid_shp = os.path.join(shp_path, 'invalid', 'emptypoints.shp')
+invalid_shp = shp_path / 'invalid' / 'emptypoints.shp'
-has_nulls_geojson = os.path.join(shp_path, 'has_nulls', 'has_nulls.geojson')
+has_nulls_geojson = shp_path / 'has_nulls' / 'has_nulls.geojson'
 
 # Dictionaries to hold what's expected in the county shapefile.
 NAMES = ['Bexar', 'Galveston', 'Harris', 'Honolulu', 'Pueblo']
@@ -83,6 +83,11 @@ class LayerMapTest(TestCase):
             self.assertAlmostEqual(pnt1.x, pnt2.x, 5)
             self.assertAlmostEqual(pnt1.y, pnt2.y, 5)
 
+    def test_data_source_str(self):
+        lm = LayerMapping(City, str(city_shp), city_mapping)
+        lm.save()
+        self.assertEqual(City.objects.count(), 3)
+
     def test_layermap_strict(self):
         "Testing the `strict` keyword, and import of a LineString shapefile."
         # When the `strict` keyword is set an error encountered will force
@@ -307,7 +312,7 @@ class LayerMapTest(TestCase):
 
     def test_encoded_name(self):
         """ Test a layer containing utf-8-encoded name """
-        city_shp = os.path.join(shp_path, 'ch-city', 'ch-city.shp')
+        city_shp = shp_path / 'ch-city' / 'ch-city.shp'
         lm = LayerMapping(City, city_shp, city_mapping)
         lm.save(silent=True, strict=True)
         self.assertEqual(City.objects.count(), 1)