Move app member creation business logic from route
This commit is contained in:
parent
4aa0afdb67
commit
0ed782f4e8
@ -154,6 +154,41 @@ def send_application_invitation(invitee_email, inviter_name, token):
|
|||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def handle_create_member(application_id, http_request):
|
||||||
|
application = Applications.get(application_id)
|
||||||
|
form = NewMemberForm(http_request.form)
|
||||||
|
|
||||||
|
if form.validate():
|
||||||
|
try:
|
||||||
|
invite = Applications.invite(
|
||||||
|
application=application,
|
||||||
|
inviter=g.current_user,
|
||||||
|
user_data=form.user_data.data,
|
||||||
|
permission_sets_names=form.data["permission_sets"],
|
||||||
|
environment_roles_data=form.environment_roles.data,
|
||||||
|
)
|
||||||
|
|
||||||
|
send_application_invitation(
|
||||||
|
invitee_email=invite.email,
|
||||||
|
inviter_name=g.current_user.full_name,
|
||||||
|
token=invite.token,
|
||||||
|
)
|
||||||
|
|
||||||
|
flash(
|
||||||
|
"new_application_member",
|
||||||
|
user_name=invite.user_name,
|
||||||
|
application_name=application.name,
|
||||||
|
)
|
||||||
|
|
||||||
|
except AlreadyExistsError:
|
||||||
|
return render_template(
|
||||||
|
"error.html", message="There was an error processing your request."
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
pass
|
||||||
|
# TODO: flash error message
|
||||||
|
|
||||||
|
|
||||||
@applications_bp.route("/applications/<application_id>/settings")
|
@applications_bp.route("/applications/<application_id>/settings")
|
||||||
@user_can(Permissions.VIEW_APPLICATION, message="view application edit form")
|
@user_can(Permissions.VIEW_APPLICATION, message="view application edit form")
|
||||||
def settings(application_id):
|
def settings(application_id):
|
||||||
@ -284,39 +319,7 @@ def delete_environment(environment_id):
|
|||||||
Permissions.CREATE_APPLICATION_MEMBER, message="create new application member"
|
Permissions.CREATE_APPLICATION_MEMBER, message="create new application member"
|
||||||
)
|
)
|
||||||
def create_member(application_id):
|
def create_member(application_id):
|
||||||
application = Applications.get(application_id)
|
handle_create_member(application_id, http_request)
|
||||||
form = NewMemberForm(http_request.form)
|
|
||||||
|
|
||||||
if form.validate():
|
|
||||||
try:
|
|
||||||
invite = Applications.invite(
|
|
||||||
application=application,
|
|
||||||
inviter=g.current_user,
|
|
||||||
user_data=form.user_data.data,
|
|
||||||
permission_sets_names=form.data["permission_sets"],
|
|
||||||
environment_roles_data=form.environment_roles.data,
|
|
||||||
)
|
|
||||||
|
|
||||||
send_application_invitation(
|
|
||||||
invitee_email=invite.email,
|
|
||||||
inviter_name=g.current_user.full_name,
|
|
||||||
token=invite.token,
|
|
||||||
)
|
|
||||||
|
|
||||||
flash(
|
|
||||||
"new_application_member",
|
|
||||||
user_name=invite.user_name,
|
|
||||||
application_name=application.name,
|
|
||||||
)
|
|
||||||
|
|
||||||
except AlreadyExistsError:
|
|
||||||
return render_template(
|
|
||||||
"error.html", message="There was an error processing your request."
|
|
||||||
)
|
|
||||||
else:
|
|
||||||
pass
|
|
||||||
# TODO: flash error message
|
|
||||||
|
|
||||||
return redirect(
|
return redirect(
|
||||||
url_for(
|
url_for(
|
||||||
"applications.settings",
|
"applications.settings",
|
||||||
|
Loading…
x
Reference in New Issue
Block a user