Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

Commit 00db7ce

Browse files
committed
updates paged_result_field with a validation function
1 parent 34b6490 commit 00db7ce

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

gapic/schema/wrappers.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1830,7 +1830,7 @@ def ident(self) -> metadata.Address:
18301830
"""Return the identifier data to be used in templates."""
18311831
return self.meta.address
18321832

1833-
def _validate_paged_field_size_type(value):
1833+
def _validate_paged_field_size_type(page_field_size):
18341834
"""Validates allowed paged_field_size type(s).
18351835
18361836
Confirms whether the paged_field_size.type is an allowed wrapper type:
@@ -1839,9 +1839,11 @@ def _validate_paged_field_size_type(value):
18391839
Int32Value.
18401840
"""
18411841

1842-
if page_field_size.type == int or (
1843-
isinstance(value.type, MessageType)
1844-
and value.type.message_pb.name in {"UInt32Value", "Int32Value"}
1842+
_type = page_field_size.type
1843+
1844+
if _type == int or (
1845+
isinstance(_type, MessageType)
1846+
and _type.message_pb.name in {"UInt32Value", "Int32Value"}
18451847
):
18461848
return True
18471849

@@ -1871,11 +1873,12 @@ def paged_result_field(self) -> Optional[Field]:
18711873
self.input.fields.get("page_size", None),
18721874
)
18731875
page_field_size = next((field for field in page_fields if field), None)
1876+
18741877
if not page_field_size:
18751878
return None
18761879

18771880
# Confirm whether the paged_field_size is an allowed type.
1878-
if not _validate_paged_field_size_type(value):
1881+
if not self._validate_paged_field_size_type(page_field_size=page_field_size):
18791882
return None
18801883

18811884
# Return the first repeated field.

0 commit comments

Comments
 (0)