Skip to content

Commit 9a8a9e5

Browse files
committed
refactor: rename handle_patronymic_name_order to handle_east_slavic_patronymic_name_order
Consistency fix: this handler only ever implemented East-Slavic rotation logic (sibling to handle_turkic_patronymic_name_order()), so the generic name was misleading now that a family-specific sibling exists — the same asymmetry that is_patronymic -> is_east_slavic_patronymic already fixed for the helper method. The patronymic_name_order flag itself stays generic, since it's the public umbrella opt-in switch covering both families by design. Pure rename, zero behavior change — single call site, not referenced by name in shipped docs.
1 parent 96e6186 commit 9a8a9e5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

nameparser/parser.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -757,7 +757,7 @@ def pre_process(self) -> None:
757757
self.parse_nicknames()
758758
self.squash_emoji()
759759

760-
def handle_patronymic_name_order(self) -> None:
760+
def handle_east_slavic_patronymic_name_order(self) -> None:
761761
"""
762762
When patronymic_name_order is enabled, detect Russian formal order
763763
(Surname GivenName Patronymic) and rotate to Western order.
@@ -818,7 +818,7 @@ def post_process(self) -> None:
818818
"""
819819
self.handle_firstnames()
820820
if self.C.patronymic_name_order:
821-
self.handle_patronymic_name_order()
821+
self.handle_east_slavic_patronymic_name_order()
822822
self.handle_turkic_patronymic_name_order()
823823
if self.C.middle_name_as_last:
824824
self.handle_middle_name_as_last()

0 commit comments

Comments
 (0)