BUG: df.replace over pd.Period columns (#34871)

doc/source/whatsnew/v1.2.0.rst

@@ -348,7 +348,7 @@ Strings
 Interval
 ^^^^^^^^
 
--
+- Bug in :meth:`DataFrame.replace` and :meth:`Series.replace` where :class:`Interval` dtypes would be converted to object dytpes (:issue:34871)
 -
 
 Indexing
@@ -387,6 +387,11 @@ I/O
 - Removed ``private_key`` and ``verbose`` from :func:`read_gbq` as they are no longer supported in ``pandas-gbq`` (:issue:`34654`, :issue:`30200`)
 - Bumped minimum pytables version to 3.5.1 to avoid a ``ValueError`` in :meth:`read_hdf` (:issue:`24839`)
 
+Period
+^^^^^^
+
+- Bug in :meth:`DataFrame.replace` and :meth:`Series.replace` where :class:`Period` dtypes would be converted to object dytpes (:issue:34871)
+
 Plotting
 ^^^^^^^^
 

pandas/core/internals/blocks.py

@@ -1974,6 +1974,12 @@ class ObjectValuesExtensionBlock(ExtensionBlock):
     def external_values(self):
         return self.values.astype(object)
 
+    def _can_hold_element(self, element: Any) -> bool:
+        tipo = maybe_infer_dtype_type(element)
+        if tipo is not None:
+            return issubclass(tipo.type, self.dtype.type)
+        return isinstance(element, self.dtype.type)
+
 
 class NumericBlock(Block):
     __slots__ = ()

pandas/tests/frame/methods/test_replace.py

@@ -1517,17 +1517,15 @@ def test_replace_with_duplicate_columns(self, replacement):
 
         tm.assert_frame_equal(result, expected)
 
-    @pytest.mark.xfail(
-        reason="replace() changes dtype from period to object, see GH34871", strict=True
-    )
-    def test_replace_period_ignore_float(self):
+    @pytest.mark.parametrize("value", [pd.Period("2020-01"), pd.Interval(0, 5)])
+    def test_replace_period_ignore_float(self, value):
         """
         Regression test for GH#34871: if df.replace(1.0, 0.0) is called on a df
-        with a Period column the old, faulty behavior is to raise TypeError.
+        with a Period/Interval column the old, faulty behavior is to raise TypeError.
         """
-        df = pd.DataFrame({"Per": [pd.Period("2020-01")] * 3})
+        df = pd.DataFrame({"Per": [value] * 3})
         result = df.replace(1.0, 0.0)
-        expected = pd.DataFrame({"Per": [pd.Period("2020-01")] * 3})
+        expected = pd.DataFrame({"Per": [value] * 3})
         tm.assert_frame_equal(expected, result)
 
     def test_replace_value_category_type(self):

pandas/tests/series/methods/test_replace.py

@@ -449,3 +449,15 @@ def test_replace_with_compiled_regex(self):
         result = s.replace({regex: "z"}, regex=True)
         expected = pd.Series(["z", "b", "c"])
         tm.assert_series_equal(result, expected)
+
+    @pytest.mark.parametrize("value", [pd.Period("2020-01"), pd.Interval(0, 5)])
+    def test_replace_period_ignore_float(self, value):
+        """
+        Regression test for corrolary to GH#34871: if series.replace(1.0, 0.0)
+        is called on a Period/Interval Series, the old, faulty behavior
+        is to raise TypeError.
+        """
+        series = pd.Series([value] * 3)
+        result = series.replace(1.0, 0.0)
+        expected = pd.Series([value] * 3)
+        tm.assert_series_equal(expected, result)