@autorest/python 6.4.13 → 6.4.15

package/autorest/codegen/models/operation.py

@@ -36,6 +36,7 @@ from .parameter import (
 )
 from .parameter_list import ParameterList
 from .model_type import ModelType
+from .base import BaseType
 from .request_builder import OverloadedRequestBuilder, RequestBuilder
 
 if TYPE_CHECKING:
@@ -49,6 +50,10 @@ ResponseType = TypeVar(
 )
 
 
+def is_internal(target: Optional[BaseType]) -> bool:
+    return isinstance(target, ModelType) and target.base == "dpg" and target.internal
+
+
 class OperationBase(  # pylint: disable=too-many-public-methods
     Generic[ResponseType], BaseBuilder[ParameterList]
 ):
@@ -130,6 +135,13 @@ class OperationBase(  # pylint: disable=too-many-public-methods
         if self.response_type_annotation(async_mode=False) == "None":
             # doesn't matter if it's async or not
             retval = add_to_pylint_disable(retval, "inconsistent-return-statements")
+        try:
+            if any(is_internal(r.type) for r in self.responses) or is_internal(
+                self.parameters.body_parameter.type
+            ):
+                retval = add_to_pylint_disable(retval, "protected-access")
+        except ValueError:
+            pass
         return retval
 
     def cls_type_annotation(self, *, async_mode: bool) -> str:

package/autorest/codegen/models/request_builder_parameter.py

@@ -40,7 +40,11 @@ class RequestBuilderBodyParameter(BodyParameter):
 
     @property
     def in_method_signature(self) -> bool:
-        return super().in_method_signature and not self.is_partial_body
+        return (
+            super().in_method_signature
+            and not self.is_partial_body
+            and self.code_model.options["models_mode"] != "dpg"
+        )
 
     @property
     def method_location(self) -> ParameterMethodLocation:

package/autorest/codegen/templates/model_base.py.jinja2

@@ -58,32 +58,35 @@ def _timedelta_as_isostr(td: timedelta) -> str:
     if days:
         date_str = "%sD" % days
 
-    # Build time
-    time_str = "T"
+    if hours or minutes or seconds:
+        # Build time
+        time_str = "T"
 
-    # Hours
-    bigger_exists = date_str or hours
-    if bigger_exists:
-        time_str += "{:02}H".format(hours)
+        # Hours
+        bigger_exists = date_str or hours
+        if bigger_exists:
+            time_str += "{:02}H".format(hours)
 
-    # Minutes
-    bigger_exists = bigger_exists or minutes
-    if bigger_exists:
-        time_str += "{:02}M".format(minutes)
+        # Minutes
+        bigger_exists = bigger_exists or minutes
+        if bigger_exists:
+            time_str += "{:02}M".format(minutes)
 
-    # Seconds
-    try:
-        if seconds.is_integer():
-            seconds_string = "{:02}".format(int(seconds))
-        else:
-            # 9 chars long w/ leading 0, 6 digits after decimal
-            seconds_string = "%09.6f" % seconds
-            # Remove trailing zeros
-            seconds_string = seconds_string.rstrip("0")
-    except AttributeError:  # int.is_integer() raises
-        seconds_string = "{:02}".format(seconds)
-
-    time_str += "{}S".format(seconds_string)
+        # Seconds
+        try:
+            if seconds.is_integer():
+                seconds_string = "{:02}".format(int(seconds))
+            else:
+                # 9 chars long w/ leading 0, 6 digits after decimal
+                seconds_string = "%09.6f" % seconds
+                # Remove trailing zeros
+                seconds_string = seconds_string.rstrip("0")
+        except AttributeError:  # int.is_integer() raises
+            seconds_string = "{:02}".format(seconds)
+
+        time_str += "{}S".format(seconds_string)
+    else:
+        time_str = ""
 
     return "P" + date_str + time_str
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autorest/python",
-  "version": "6.4.13",
+  "version": "6.4.15",
   "description": "The Python extension for generators in AutoRest.",
   "main": "index.js",
   "repository": {