Submitting the admin/user/user/create form does nothing, no user account is created.

CommentFileSizeAuthor
#4 bakery-admin-D7-957242-4.patch1.18 KBDavid_Rothstein
#2 957242_admin_create_users-2.patch2.19 KBcoltrane
#1 957242_admin_create_users.patch2 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new 957242_admin_create_users.patch2 KB
coltrane’s picture

coltrane
he/him
commented
StatusFileSize
new 957242_admin_create_users-2.patch2.19 KB

#1 had the submit handler outside of the if statement, so it didn't fix the bug. I also added a few comments.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Committed to 6.x-2.x. Needs porting.

David_Rothstein’s picture

David_Rothstein commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new bakery-admin-D7-957242-4.patch1.18 KB

This appears to have been mostly ported already, except there's a small bug in the D7 version that prevents it from working.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

We now have admin/user/bakery so this help text should probably reference that, no?

David_Rothstein’s picture

David_Rothstein commented

In D7 I think that's admin/config/people/bakery?

David_Rothstein’s picture

David_Rothstein commented
Title: Admin is unable to create user accounts on slave » Admin is unable to create user accounts on slave (mostly works, finish porting patch to D7)

Retitling since I think the main functionality works; this is just a small followup.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Needs work

Right.

purencool’s picture

purencool commented
Issue summary: View changes
Status: Needs work » Closed (outdated)