Skip to content

Commit 7035a6b

Browse files
committed
importing user_services
1 parent de80c9a commit 7035a6b

File tree

7 files changed

+25
-16
lines changed

7 files changed

+25
-16
lines changed

services/web/server/src/simcore_service_webserver/exporter/_handlers.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
from ..projects._projects_service import create_user_notification_cb
1919
from ..redis import get_redis_lock_manager_client_sdk
2020
from ..security.decorators import permission_required
21-
from ..users.users_service import get_user_fullname
21+
from ..users import users_service
2222
from ._formatter.archive import get_sds_archive_path
2323
from .exceptions import SDSException
2424
from .utils import CleanupFileResponse
@@ -52,7 +52,8 @@ async def export_project(request: web.Request):
5252
project_uuid=project_uuid,
5353
status=ProjectStatus.EXPORTING,
5454
owner=Owner(
55-
user_id=user_id, **await get_user_fullname(request.app, user_id=user_id)
55+
user_id=user_id,
56+
**await users_service.get_user_fullname(request.app, user_id=user_id),
5657
),
5758
notification_cb=create_user_notification_cb(
5859
user_id, ProjectID(f"{project_uuid}"), request.app

services/web/server/src/simcore_service_webserver/functions/_functions_repository.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,9 @@ async def create_function_job( # noqa: PLR0913
208208

209209
registered_function_job = RegisteredFunctionJobDB.model_validate(row)
210210

211-
user_primary_group_id = await get_user_primary_group_id(app, user_id=user_id)
211+
user_primary_group_id = await users_service.get_user_primary_group_id(
212+
app, user_id=user_id
213+
)
212214
await set_group_permissions(
213215
app,
214216
connection=transaction,
@@ -293,7 +295,9 @@ async def create_function_job_collection(
293295
) # nosec
294296
job_collection_entries.append(entry)
295297

296-
user_primary_group_id = await get_user_primary_group_id(app, user_id=user_id)
298+
user_primary_group_id = await users_service.get_user_primary_group_id(
299+
app, user_id=user_id
300+
)
297301
await set_group_permissions(
298302
app,
299303
connection=transaction,

services/web/server/src/simcore_service_webserver/groups/_groups_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,7 @@ async def add_user_in_group(
267267

268268
if new_by_user_id is not None:
269269
new_user = await users_service.get_user(app, new_by_user_id)
270-
new_by_user_name = new_user.name
270+
new_by_user_name = new_user["name"]
271271

272272
return await _groups_repository.add_new_user_in_group(
273273
app,

services/web/server/src/simcore_service_webserver/login_accounts/_controller_rest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
from ..security import security_service, security_web
2929
from ..security.decorators import permission_required
3030
from ..session import api as session_service
31-
from ..users import api as users_service
31+
from ..users import users_service
3232
from ..users._common.schemas import PreRegisteredUserGet
3333
from ..utils import MINUTE
3434
from ..utils_rate_limiting import global_rate_limit_route

services/web/server/src/simcore_service_webserver/payments/_onetime_api.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -391,8 +391,10 @@ async def pay_with_payment_method(
391391
)
392392

393393
# user info
394-
user = await get_user_display_and_id_names(app, user_id=user_id)
395-
user_invoice_address = await get_user_invoice_address(app, user_id=user_id)
394+
user_info = await users_service.get_user_display_and_id_names(app, user_id=user_id)
395+
user_invoice_address = await users_service.get_user_invoice_address(
396+
app, user_id=user_id
397+
)
396398

397399
settings: PaymentsSettings = get_plugin_settings(app)
398400
if settings.PAYMENTS_FAKE_COMPLETION:
@@ -405,8 +407,8 @@ async def pay_with_payment_method(
405407
wallet_id=wallet_id,
406408
wallet_name=user_wallet.name,
407409
user_id=user_id,
408-
user_name=user.full_name,
409-
user_email=user.email,
410+
user_name=user_info.full_name,
411+
user_email=user_info.email,
410412
comment=comment,
411413
)
412414

@@ -421,8 +423,8 @@ async def pay_with_payment_method(
421423
wallet_id=wallet_id,
422424
wallet_name=user_wallet.name,
423425
user_id=user_id,
424-
user_name=user.full_name,
425-
user_email=user.email,
426+
user_name=user_info.full_name,
427+
user_email=user_info.email,
426428
user_address=user_invoice_address,
427429
stripe_price_id=product_stripe_info.stripe_price_id,
428430
stripe_tax_rate_id=product_stripe_info.stripe_tax_rate_id,

services/web/server/src/simcore_service_webserver/projects/_controller/nodes_rest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -562,7 +562,7 @@ async def get_project_services_access_for_gid(request: web.Request) -> web.Respo
562562

563563
# Update groups to compare based on the type of sharing group
564564
if _sharing_with_group.group_type == GroupType.PRIMARY:
565-
_user_id = await get_user_id_from_gid(
565+
_user_id = await users_service.get_user_id_from_gid(
566566
app=request.app, primary_gid=query_params.for_gid
567567
)
568568
user_groups_ids = await list_all_user_groups_ids(

services/web/server/src/simcore_service_webserver/projects/_projects_service.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ async def patch_project(
323323
"write": True,
324324
"delete": True,
325325
}
326-
user: dict = await get_user(app, project_db.prj_owner)
326+
user: dict = await users_service.get_user(app, project_db.prj_owner)
327327
_prj_owner_primary_group = f"{user['primary_gid']}"
328328
if _prj_owner_primary_group not in new_prj_access_rights:
329329
raise ProjectOwnerNotFoundInTheProjectAccessRightsError
@@ -333,7 +333,7 @@ async def patch_project(
333333
# 4. If patching template type
334334
if new_template_type := patch_project_data.get("template_type"):
335335
# 4.1 Check if user is a tester
336-
current_user: dict = await get_user(app, user_id)
336+
current_user: dict = await users_service.get_user(app, user_id)
337337
if UserRole(current_user["role"]) < UserRole.TESTER:
338338
raise InsufficientRoleForProjectTemplateTypeUpdateError
339339
# 4.2 Check the compatibility of the template type with the project
@@ -668,7 +668,9 @@ async def _start_dynamic_service( # noqa: C901
668668
raise
669669

670670
save_state = False
671-
user_role: UserRole = await get_user_role(request.app, user_id=user_id)
671+
user_role: UserRole = await users_service.get_user_role(
672+
request.app, user_id=user_id
673+
)
672674
if user_role > UserRole.GUEST:
673675
save_state = await has_user_project_access_rights(
674676
request.app, project_id=project_uuid, user_id=user_id, permission="write"

0 commit comments

Comments
 (0)