Skip to content

Commit 418f7c1

Browse files
committed
Removed unnecessary docs comments
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
1 parent b87097b commit 418f7c1

File tree

4 files changed

+21
-98
lines changed

4 files changed

+21
-98
lines changed

nc_py_api/_session.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,6 @@ def _create_adapter(self, dav: bool = False) -> Client:
321321
)
322322

323323
def _add_auth(self, request: Request):
324-
# request.url.copy_add_param()
325324
request.headers.update({
326325
"AUTHORIZATION-APP-API": b64encode(f"{self._user}:{self.cfg.app_secret}".encode("UTF=8"))
327326
})

nc_py_api/user_status.py

Lines changed: 4 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -121,11 +121,7 @@ def available(self) -> bool:
121121
return not check_capabilities("user_status.enabled", self._session.capabilities)
122122

123123
def get_list(self, limit: typing.Optional[int] = None, offset: typing.Optional[int] = None) -> list[UserStatus]:
124-
"""Returns statuses for all users.
125-
126-
:param limit: limits the number of results.
127-
:param offset: offset of results.
128-
"""
124+
"""Returns statuses for all users."""
129125
require_capabilities("user_status.enabled", self._session.capabilities)
130126
data = kwargs_to_params(["limit", "offset"], limit=limit, offset=offset)
131127
result = self._session.ocs("GET", f"{self._ep_base}/statuses", params=data)
@@ -137,10 +133,7 @@ def get_current(self) -> CurrentUserStatus:
137133
return CurrentUserStatus(self._session.ocs("GET", f"{self._ep_base}/user_status"))
138134

139135
def get(self, user_id: str) -> typing.Optional[UserStatus]:
140-
"""Returns the user status for the specified user.
141-
142-
:param user_id: User ID for getting status.
143-
"""
136+
"""Returns the user status for the specified user."""
144137
require_capabilities("user_status.enabled", self._session.capabilities)
145138
try:
146139
return UserStatus(self._session.ocs("GET", f"{self._ep_base}/statuses/{user_id}"))
@@ -195,21 +188,15 @@ def set_status(self, message: typing.Optional[str] = None, clear_at: int = 0, st
195188
self._session.ocs("PUT", f"{self._ep_base}/user_status/message/custom", params=params)
196189

197190
def get_backup_status(self, user_id: str = "") -> typing.Optional[UserStatus]:
198-
"""Get the backup status of the user if any.
199-
200-
:param user_id: User ID for getting status.
201-
"""
191+
"""Get the backup status of the user if any."""
202192
require_capabilities("user_status.enabled", self._session.capabilities)
203193
user_id = user_id if user_id else self._session.user
204194
if not user_id:
205195
raise ValueError("user_id can not be empty.")
206196
return self.get(f"_{user_id}")
207197

208198
def restore_backup_status(self, status_id: str) -> typing.Optional[CurrentUserStatus]:
209-
"""Restores the backup state as current for the current user.
210-
211-
:param status_id: backup status ID.
212-
"""
199+
"""Restores the backup state as current for the current user."""
213200
require_capabilities("user_status.enabled", self._session.capabilities)
214201
require_capabilities("user_status.restore", self._session.capabilities)
215202
result = self._session.ocs("DELETE", f"{self._ep_base}/user_status/revert/{status_id}")

nc_py_api/users.py

Lines changed: 10 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -170,21 +170,13 @@ def __init__(self, session: NcSessionBasic):
170170
def get_list(
171171
self, mask: typing.Optional[str] = "", limit: typing.Optional[int] = None, offset: typing.Optional[int] = None
172172
) -> list[str]:
173-
"""Returns list of user IDs.
174-
175-
:param mask: user ID mask to apply.
176-
:param limit: limits the number of results.
177-
:param offset: offset of results.
178-
"""
173+
"""Returns list of user IDs."""
179174
data = kwargs_to_params(["search", "limit", "offset"], search=mask, limit=limit, offset=offset)
180175
response_data = self._session.ocs("GET", self._ep_base, params=data)
181176
return response_data["users"] if response_data else {}
182177

183178
def get_user(self, user_id: str = "") -> UserInfo:
184-
"""Returns detailed user information.
185-
186-
:param user_id: the identifier of the user about which information is to be returned.
187-
"""
179+
"""Returns detailed user information."""
188180
return UserInfo(self._session.ocs("GET", f"{self._ep_base}/{user_id}" if user_id else "/ocs/v1.php/cloud/user"))
189181

190182
def create(self, user_id: str, display_name: typing.Optional[str] = None, **kwargs) -> None:
@@ -216,31 +208,19 @@ def create(self, user_id: str, display_name: typing.Optional[str] = None, **kwar
216208
self._session.ocs("POST", self._ep_base, json=data)
217209

218210
def delete(self, user_id: str) -> None:
219-
"""Deletes user from the Nextcloud server.
220-
221-
:param user_id: id of the user.
222-
"""
211+
"""Deletes user from the Nextcloud server."""
223212
self._session.ocs("DELETE", f"{self._ep_base}/{user_id}")
224213

225214
def enable(self, user_id: str) -> None:
226-
"""Enables user on the Nextcloud server.
227-
228-
:param user_id: id of the user.
229-
"""
215+
"""Enables user on the Nextcloud server."""
230216
self._session.ocs("PUT", f"{self._ep_base}/{user_id}/enable")
231217

232218
def disable(self, user_id: str) -> None:
233-
"""Disables user on the Nextcloud server.
234-
235-
:param user_id: id of the user.
236-
"""
219+
"""Disables user on the Nextcloud server."""
237220
self._session.ocs("PUT", f"{self._ep_base}/{user_id}/disable")
238221

239222
def resend_welcome_email(self, user_id: str) -> None:
240-
"""Send welcome email for specified user again.
241-
242-
:param user_id: id of the user.
243-
"""
223+
"""Send welcome email for specified user again."""
244224
self._session.ocs("POST", f"{self._ep_base}/{user_id}/welcome")
245225

246226
def editable_fields(self) -> list[str]:
@@ -257,35 +237,19 @@ def edit(self, user_id: str, **kwargs) -> None:
257237
self._session.ocs("PUT", f"{self._ep_base}/{user_id}", params={"key": k, "value": v})
258238

259239
def add_to_group(self, user_id: str, group_id: str) -> None:
260-
"""Adds user to the group.
261-
262-
:param user_id: ID of the user.
263-
:param group_id: the destination group to which add user to.
264-
"""
240+
"""Adds user to the group."""
265241
self._session.ocs("POST", f"{self._ep_base}/{user_id}/groups", params={"groupid": group_id})
266242

267243
def remove_from_group(self, user_id: str, group_id: str) -> None:
268-
"""Removes user from the group.
269-
270-
:param user_id: ID of the user.
271-
:param group_id: group from which remove user.
272-
"""
244+
"""Removes user from the group."""
273245
self._session.ocs("DELETE", f"{self._ep_base}/{user_id}/groups", params={"groupid": group_id})
274246

275247
def promote_to_subadmin(self, user_id: str, group_id: str) -> None:
276-
"""Makes user admin of the group.
277-
278-
:param user_id: ID of the user.
279-
:param group_id: group where user should become administrator.
280-
"""
248+
"""Makes user admin of the group."""
281249
self._session.ocs("POST", f"{self._ep_base}/{user_id}/subadmins", params={"groupid": group_id})
282250

283251
def demote_from_subadmin(self, user_id: str, group_id: str) -> None:
284-
"""Removes user from the admin role of the group.
285-
286-
:param user_id: ID of the user.
287-
:param group_id: group where user should be removed from administrators.
288-
"""
252+
"""Removes user from the admin role of the group."""
289253
self._session.ocs("DELETE", f"{self._ep_base}/{user_id}/subadmins", params={"groupid": group_id})
290254

291255
def get_avatar(

nc_py_api/users_groups.py

Lines changed: 7 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -62,67 +62,40 @@ def __init__(self, session: NcSessionBasic):
6262
def get_list(
6363
self, mask: typing.Optional[str] = None, limit: typing.Optional[int] = None, offset: typing.Optional[int] = None
6464
) -> list[str]:
65-
"""Returns a list of user groups IDs.
66-
67-
:param mask: group ID mask to apply.
68-
:param limit: limits the number of results.
69-
:param offset: offset of results.
70-
"""
65+
"""Returns a list of user groups IDs."""
7166
data = kwargs_to_params(["search", "limit", "offset"], search=mask, limit=limit, offset=offset)
7267
response_data = self._session.ocs("GET", self._ep_base, params=data)
7368
return response_data["groups"] if response_data else []
7469

7570
def get_details(
7671
self, mask: typing.Optional[str] = None, limit: typing.Optional[int] = None, offset: typing.Optional[int] = None
7772
) -> list[GroupDetails]:
78-
"""Returns a list of user groups with detailed information.
79-
80-
:param mask: group ID mask to apply.
81-
:param limit: limits the number of results.
82-
:param offset: offset of results.
83-
"""
73+
"""Returns a list of user groups with detailed information."""
8474
data = kwargs_to_params(["search", "limit", "offset"], search=mask, limit=limit, offset=offset)
8575
response_data = self._session.ocs("GET", f"{self._ep_base}/details", params=data)
8676
return [GroupDetails(i) for i in response_data["groups"]] if response_data else []
8777

8878
def create(self, group_id: str, display_name: typing.Optional[str] = None) -> None:
89-
"""Creates the users group.
90-
91-
:param group_id: the ID of group to be created.
92-
:param display_name: display name for a created group.
93-
"""
79+
"""Creates the users group."""
9480
params = {"groupid": group_id}
9581
if display_name is not None:
9682
params["displayname"] = display_name
9783
self._session.ocs("POST", f"{self._ep_base}", params=params)
9884

9985
def edit(self, group_id: str, display_name: str) -> None:
100-
"""Edits users group information.
101-
102-
:param group_id: the ID of group to edit info.
103-
:param display_name: new group display name.
104-
"""
86+
"""Edits users group information."""
10587
params = {"key": "displayname", "value": display_name}
10688
self._session.ocs("PUT", f"{self._ep_base}/{group_id}", params=params)
10789

10890
def delete(self, group_id: str) -> None:
109-
"""Removes the users group.
110-
111-
:param group_id: the ID of group to remove.
112-
"""
91+
"""Removes the users group."""
11392
self._session.ocs("DELETE", f"{self._ep_base}/{group_id}")
11493

11594
def get_members(self, group_id: str) -> list[str]:
116-
"""Returns a list of group users.
117-
118-
:param group_id: Group ID to get the list of members.
119-
"""
95+
"""Returns a list of group users."""
12096
response_data = self._session.ocs("GET", f"{self._ep_base}/{group_id}")
12197
return response_data["users"] if response_data else {}
12298

12399
def get_subadmins(self, group_id: str) -> list[str]:
124-
"""Returns list of users who is subadmins of the group.
125-
126-
:param group_id: group ID to get the list of subadmins.
127-
"""
100+
"""Returns list of users who is subadmins of the group."""
128101
return self._session.ocs("GET", f"{self._ep_base}/{group_id}/subadmins")

0 commit comments

Comments
 (0)