From 889e8064dd914103e6b7a7ba571ac057d489bc3a Mon Sep 17 00:00:00 2001 From: Suyash Vardhan Mathur Date: Mon, 16 Aug 2021 12:07:11 +0530 Subject: [PATCH] zulip: Standardize the names of bindings with OperationIDs. Some bindings had names that differed from the names that were used in the OperationID. This commit deprecates the older names, and changes the names of the bindings to match operationIDs. --- zulip/zulip/__init__.py | 101 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 99 insertions(+), 2 deletions(-) diff --git a/zulip/zulip/__init__.py b/zulip/zulip/__init__.py index 725896e34..92b3c671a 100644 --- a/zulip/zulip/__init__.py +++ b/zulip/zulip/__init__.py @@ -951,6 +951,10 @@ def remove_reaction(self, reaction_data: Dict[str, Any]) -> Dict[str, Any]: ) def get_realm_emoji(self) -> Dict[str, Any]: + logger.warning("get_realm_emoji() is deprecated." " Please use get_custom_emoji() instead.") + return self.get_custom_emoji() + + def get_custom_emoji(self) -> Dict[str, Any]: """ See examples/realm-emoji for example usage. """ @@ -978,6 +982,12 @@ def delete_custom_emoji(self, emoji_name: str) -> Dict[str, Any]: ) def get_realm_linkifiers(self) -> Dict[str, Any]: + logger.warning( + "get_realm_linkifiers() is deprecated." " Please use get_linkifiers() instead." + ) + return self.get_linkifiers() + + def get_linkifiers(self) -> Dict[str, Any]: """ Example usage: @@ -1000,6 +1010,10 @@ def get_realm_linkifiers(self) -> Dict[str, Any]: ) def add_realm_filter(self, pattern: str, url_format_string: str) -> Dict[str, Any]: + logger.warning("get_members() is deprecated." " Please use get_users() instead.") + return self.add_linkifier(pattern, url_format_string) + + def add_linkifier(self, pattern: str, url_format_string: str) -> Dict[str, Any]: """ Example usage: @@ -1016,6 +1030,12 @@ def add_realm_filter(self, pattern: str, url_format_string: str) -> Dict[str, An ) def remove_realm_filter(self, filter_id: int) -> Dict[str, Any]: + logger.warning( + "remove_realm_filter() is deprecated." " Please use remove_linkifier() instead." + ) + return self.remove_linkifier(filter_id) + + def remove_linkifier(self, filter_id: int) -> Dict[str, Any]: """ Example usage: @@ -1028,6 +1048,13 @@ def remove_realm_filter(self, filter_id: int) -> Dict[str, Any]: ) def get_realm_profile_fields(self) -> Dict[str, Any]: + logger.warning( + "get_realm_profile_fields() is deprecated." + " Please use get_custom_profile_fields() instead." + ) + return self.get_custom_profile_fields() + + def get_custom_profile_fields(self) -> Dict[str, Any]: """ Example usage: @@ -1040,6 +1067,13 @@ def get_realm_profile_fields(self) -> Dict[str, Any]: ) def create_realm_profile_field(self, **request: Any) -> Dict[str, Any]: + logger.warning( + "create_realm_profile_field() is deprecated." + " Please use create_custom_profile_field() instead." + ) + return self.create_custom_profile_field(**request) + + def create_custom_profile_field(self, **request: Any) -> Dict[str, Any]: """ Example usage: @@ -1065,6 +1099,13 @@ def remove_realm_profile_field(self, field_id: int) -> Dict[str, Any]: ) def reorder_realm_profile_fields(self, **request: Any) -> Dict[str, Any]: + logger.warning( + "reorder_realm_profile_fields() is deprecated." + " Please use reorder_custom_profile_fields() instead." + ) + return self.reorder_custom_profile_fields(**request) + + def reorder_custom_profile_fields(self, **request: Any) -> Dict[str, Any]: """ Example usage: @@ -1078,6 +1119,12 @@ def reorder_realm_profile_fields(self, **request: Any) -> Dict[str, Any]: ) def update_realm_profile_field(self, field_id: int, **request: Any) -> Dict[str, Any]: + logger.warning( + "update_realm_profile_field() is deprecated." " Please use update_linkifier() instead." + ) + return self.update_linkifier(field_id, **request) + + def update_linkifier(self, field_id: int, **request: Any) -> Dict[str, Any]: """ Example usage: @@ -1118,6 +1165,15 @@ def register( event_types: Optional[Iterable[str]] = None, narrow: Optional[List[List[str]]] = None, **kwargs: object, + ) -> Dict[str, Any]: + logger.warning("register() is deprecated." " Please use register_queue() instead.") + return self.register_queue(event_types, narrow, **kwargs) + + def register_queue( + self, + event_types: Optional[Iterable[str]] = None, + narrow: Optional[List[List[str]]] = None, + **kwargs: object, ) -> Dict[str, Any]: """ Example usage: @@ -1139,6 +1195,10 @@ def register( ) def deregister(self, queue_id: str, timeout: Optional[float] = None) -> Dict[str, Any]: + logger.warning("deregister() is deprecated." " Please use delete_queue() instead.") + return self.delete_queue(queue_id, timeout) + + def delete_queue(self, queue_id: str, timeout: Optional[float] = None) -> Dict[str, Any]: """ Example usage: @@ -1157,6 +1217,10 @@ def deregister(self, queue_id: str, timeout: Optional[float] = None) -> Dict[str ) def get_profile(self, request: Optional[Dict[str, Any]] = None) -> Dict[str, Any]: + logger.warning("get_profile() is deprecated." " Please use get_own_user() instead.") + self.get_own_user(request) + + def get_own_user(self, request: Optional[Dict[str, Any]] = None) -> Dict[str, Any]: """ Example usage: @@ -1232,6 +1296,10 @@ def update_stream(self, stream_data: Dict[str, Any]) -> Dict[str, Any]: ) def delete_stream(self, stream_id: int) -> Dict[str, Any]: + logger.warning("delete_stream() is deprecated." " Please use archive_stream() instead.") + return self.archive_stream(stream_id) + + def archive_stream(self, stream_id: int) -> Dict[str, Any]: """ See examples/delete-stream for example usage. """ @@ -1255,6 +1323,10 @@ def add_default_stream(self, stream_id: int) -> Dict[str, Any]: ) def get_user_by_id(self, user_id: int, **request: Any) -> Dict[str, Any]: + logger.warning("get_user_by_id() is deprecated." " Please use get_user() instead.") + return self.get_user(user_id, **request) + + def get_user(self, user_id: int, **request: Any) -> Dict[str, Any]: """ Example usage: @@ -1269,6 +1341,12 @@ def get_user_by_id(self, user_id: int, **request: Any) -> Dict[str, Any]: ) def deactivate_user_by_id(self, user_id: int) -> Dict[str, Any]: + logger.warning( + "deactivate_user_by_id() is deprecated." " Please use deactivate_user() instead." + ) + return self.deactivate_user(user_id) + + def deactivate_user(self, user_id: int) -> Dict[str, Any]: """ Example usage: @@ -1282,6 +1360,12 @@ def deactivate_user_by_id(self, user_id: int) -> Dict[str, Any]: ) def reactivate_user_by_id(self, user_id: int) -> Dict[str, Any]: + logger.warning( + "reactivate_user_by_id() is deprecated." " Please use reactivate_user() instead." + ) + return self.reactivate_user(user_id) + + def reactivate_user(self, user_id: int) -> Dict[str, Any]: """ Example usage: @@ -1295,6 +1379,10 @@ def reactivate_user_by_id(self, user_id: int) -> Dict[str, Any]: ) def update_user_by_id(self, user_id: int, **request: Any) -> Dict[str, Any]: + logger.warning("update_user_by_id() is deprecated." " Please use update_user() instead.") + return self.update_user(user_id, **request) + + def update_user(self, user_id: int, **request: Any) -> Dict[str, Any]: """ Example usage: @@ -1321,8 +1409,7 @@ def get_users(self, request: Optional[Dict[str, Any]] = None) -> Dict[str, Any]: def get_members(self, request: Optional[Dict[str, Any]] = None) -> Dict[str, Any]: # This exists for backwards-compatibility; we renamed this # function get_users for consistency with the rest of the API. - # Later, we may want to add a warning for clients using this - # legacy name. + logger.warning("get_members() is deprecated." " Please use get_users() instead.") return self.get_users(request=request) def get_alert_words(self) -> Dict[str, Any]: @@ -1364,6 +1451,10 @@ def list_subscriptions(self, request: Optional[Dict[str, Any]] = None) -> Dict[s return self.get_subscriptions(request) def add_subscriptions(self, streams: Iterable[Dict[str, Any]], **kwargs: Any) -> Dict[str, Any]: + logger.warning("add_subscriptions() is deprecated." " Please use subscribe() instead.") + return self.subscribe(streams, **kwargs) + + def subscribe(self, streams: Iterable[Dict[str, Any]], **kwargs: Any) -> Dict[str, Any]: """ See examples/subscribe for example usage. """ @@ -1376,6 +1467,12 @@ def add_subscriptions(self, streams: Iterable[Dict[str, Any]], **kwargs: Any) -> def remove_subscriptions( self, streams: Iterable[str], principals: Union[Sequence[str], Sequence[int]] = [] + ) -> Dict[str, Any]: + logger.warning("remove_subscriptions() is deprecated." " Please use unsubscribe() instead.") + return self.unsubscribe(streams, principals) + + def unsubscribe( + self, streams: Iterable[str], principals: Union[Sequence[str], Sequence[int]] = [] ) -> Dict[str, Any]: """ See examples/unsubscribe for example usage.