[fix] save() -> insert() for single doc, use doc.append in child table
This commit is contained in:
parent
a8065ff725
commit
747c464248
@ -20,20 +20,23 @@ def register_seller(**kwargs):
|
|||||||
settings.users = []
|
settings.users = []
|
||||||
|
|
||||||
validate_registerer()
|
validate_registerer()
|
||||||
|
|
||||||
message = settings.register()
|
message = settings.register()
|
||||||
|
|
||||||
if message.get('email'):
|
if message.get('email'):
|
||||||
settings.registered = 1
|
settings.registered = 1
|
||||||
|
|
||||||
user_emails = kwargs.get('users').strip()[:-1].split(', ')
|
settings.append('users', {
|
||||||
settings.users = get_users_object_list(
|
'user': frappe.session.user,
|
||||||
user_emails,
|
'name': frappe.session.user,
|
||||||
kwargs.get('username'),
|
'username': kwargs.get('username'),
|
||||||
message.get('password')
|
'password': message.get('password')
|
||||||
)
|
})
|
||||||
|
|
||||||
settings.save()
|
user_emails = kwargs.get('users').strip()[:-1].split(', ')
|
||||||
|
for email in user_emails:
|
||||||
|
settings.append('users', {'user': email, 'name': email})
|
||||||
|
|
||||||
|
settings.insert()
|
||||||
|
|
||||||
return message
|
return message
|
||||||
|
|
||||||
@ -44,20 +47,6 @@ def validate_registerer():
|
|||||||
if 'System Manager' not in frappe.get_roles():
|
if 'System Manager' not in frappe.get_roles():
|
||||||
frappe.throw(_('Only users with System Manager role can register on Marketplace'), frappe.PermissionError)
|
frappe.throw(_('Only users with System Manager role can register on Marketplace'), frappe.PermissionError)
|
||||||
|
|
||||||
def get_users_object_list(user_emails, session_user_username, session_user_password):
|
|
||||||
users = []
|
|
||||||
|
|
||||||
for user_email in user_emails:
|
|
||||||
users.append({ 'user': user_email })
|
|
||||||
|
|
||||||
users.insert(0, {
|
|
||||||
'user': frappe.session.user,
|
|
||||||
'username': session_user_username,
|
|
||||||
'password': session_user_password
|
|
||||||
})
|
|
||||||
|
|
||||||
return users
|
|
||||||
|
|
||||||
@frappe.whitelist()
|
@frappe.whitelist()
|
||||||
def call_hub_method(method, params=None):
|
def call_hub_method(method, params=None):
|
||||||
connection = get_hub_connection()
|
connection = get_hub_connection()
|
||||||
|
Loading…
x
Reference in New Issue
Block a user