BUG: cant modify df with duplicate index (pandas-dev#17105)

Author: Saravia Rajal

doc/source/whatsnew/v0.23.0.txt

@@ -1243,6 +1243,7 @@ Indexing
 - Bug in ``Series.is_unique`` where extraneous output in stderr is shown if Series contains objects with ``__ne__`` defined (:issue:`20661`)
 - Bug in ``.loc`` assignment with a single-element list-like incorrectly assigns as a list (:issue:`19474`)
 - Bug in partial string indexing on a ``Series/DataFrame`` with a monotonic decreasing ``DatetimeIndex`` (:issue:`19362`)
+- Bug in performing in-place operations on a DataFrame with a duplicate Index (:issue:`17105`)
 
 MultiIndex
 ^^^^^^^^^^

pandas/core/indexing.py

@@ -1319,7 +1319,7 @@ def _convert_to_indexer(self, obj, axis=None, is_setter=False):
                     (indexer,
                      missing) = labels.get_indexer_non_unique(objarr)
                     # 'indexer' has dupes, create 'check' using 'missing'
-                    check = np.zeros_like(objarr)
+                    check = np.zeros(len(objarr))
                     check[missing] = -1
 
                 mask = check == -1

pandas/tests/indexing/test_loc.py

@@ -784,3 +784,47 @@ def convert_nested_indexer(indexer_type, keys):
             index=pd.MultiIndex.from_product(keys))
 
         tm.assert_series_equal(result, expected)
+
+    def test_modify_with_duplicate_index_assigning(self):
+        """ see issue #17105 """
+
+        trange = pd.date_range(start=pd.Timestamp(year=2017, month=1, day=1),
+                               end=pd.Timestamp(year=2017, month=1, day=5))
+
+        # insert a duplicate element to the index
+        trange = trange.insert(loc=5,
+                               item=pd.Timestamp(year=2017, month=1, day=5))
+
+        df = pd.DataFrame(0, index=trange, columns=["A", "B"])
+        bool_idx = np.array([False, False, False, False, False, True])
+
+        # modify the value for the duplicate index entry
+        df.loc[trange[bool_idx], "A"] = 6
+
+        expected = pd.DataFrame({'A': [0, 0, 0, 0, 6, 6],
+                                 'B': [0, 0, 0, 0, 0, 0]},
+                                index=trange)
+
+        tm.assert_frame_equal(df, expected)
+
+    def test_modify_with_duplicate_index_adding(self):
+        """ see issue #17105 """
+
+        trange = pd.date_range(start=pd.Timestamp(year=2017, month=1, day=1),
+                               end=pd.Timestamp(year=2017, month=1, day=5))
+
+        # insert a duplicate element to the index
+        trange = trange.insert(loc=5,
+                               item=pd.Timestamp(year=2017, month=1, day=5))
+
+        df = pd.DataFrame(0, index=trange, columns=["A", "B"])
+        bool_idx = np.array([False, False, False, False, False, True])
+
+        # modify the value for the duplicate index entry
+        df.loc[trange[bool_idx], "A"] += 7
+
+        expected = pd.DataFrame({'A': [0, 0, 0, 0, 7, 7],
+                                 'B': [0, 0, 0, 0, 0, 0]},
+                                index=trange)
+
+        tm.assert_frame_equal(df, expected)