change inplace check position

Author: yl2526

pandas/core/generic.py

@@ -4415,26 +4415,25 @@ def _clip_with_scalar(self, lower, upper, inplace=False):
 
     def _clip_with_one_bound(self, threshold, method, axis, inplace):
 
+        inplace = validate_bool_kwarg(inplace, 'inplace')
+
         if np.any(isnull(threshold)):
             raise ValueError("Cannot use an NA value as a clip threshold")
 
         # method is self.le for upper bound and self.ge for lower bound
         if is_scalar(threshold) and is_number(threshold):
             if method.__name__ == 'le':
                 return self._clip_with_scalar(None, threshold, inplace=inplace)
-            else:
-                return self._clip_with_scalar(threshold, None, inplace=inplace)
-
-        inplace = validate_bool_kwarg(inplace, 'inplace')
+            return self._clip_with_scalar(threshold, None, inplace=inplace)
 
         subset = method(threshold, axis=axis) | isnull(self)
 
         # GH #15390
-        if is_scalar(threshold) or is_number(threshold):
+        if is_scalar(threshold):
             return self.where(subset, threshold, axis=axis, inplace=inplace)
 
         # For arry_like threshold, convet it to Series with corret index
-        # `where` only takes
+        # `where` takes scalar, NDFrame, or callable for argument "other"
         try:
             if isinstance(subset, ABCSeries):
                 threshold = pd.Series(threshold, index=subset.index)

pandas/tests/series/test_analytics.py

@@ -1021,7 +1021,7 @@ def test_clip_against_list(self, inplace):
         original = pd.Series([5, 6, 7])
         result = original.clip(upper=[1, 2, 3], inplace=inplace)
         expected = pd.Series([1, 2, 3])
-        
+
         if inplace:
             tm.assert_series_equal(original, expected, check_exact=True)
         else: