Sometimes we may need to use different form mode for checkout user registration form than site's main user registration form.

But that is not possible now, as it is hard-coded to use "Register" form mode from core user module, for the user register form in the Login checkout pane.

Please bring this feature so we can user any custom form mode we configure to user instead of default Register form mode.

CommentFileSizeAuthor
#4 3328849-4.patch7.53 KBjsacksick
#3 3328849-interdiff-2-to-3.patch902 bytesjunaidpv
#3 3328849-3.patch3.46 KBjunaidpv
#2 3328849-2.patch2.87 KBjunaidpv

Comments

junaidpv created an issue. See original summary.

junaidpv’s picture

junaidpv
Location Kannur, Kerala
commented
Status: Active » Needs review
StatusFileSize
new 3328849-2.patch2.87 KB

Here is a patch implementing that feature.

junaidpv’s picture

junaidpv
Location Kannur, Kerala
commented
StatusFileSize
new 3328849-3.patch3.46 KB
new 3328849-interdiff-2-to-3.patch902 bytes

For got use configured form mode for validating submitted user profile fields. Here is the updated patch with interdiff.

jsacksick’s picture

jsacksick commented
StatusFileSize
new 3328849-4.patch7.53 KB

Made several improvements to the patch:

  1. Renamed "register_form_mode" to "registration_form_mode".
  2. The pane summary now shows the registration form mode selected.
  3. The registration form mode setting only shows when "Registration" is allowed.
  4. I'm injecting the entity display repository rather than getting it from the service container.
  5. I'm calling the getFormModeOptions() rather than building the array of options ourselves.

Will go ahead and commit the patch if the tests are green.

  • jsacksick committed 5494273d on 8.x-2.x authored by junaidpv 
    Issue #3328849 by junaidpv, jsacksick: Allow to use different form mode...

  • jsacksick committed a3407288 on 3.0.x authored by junaidpv 
    Issue #3328849 by junaidpv, jsacksick: Allow to use different form mode...

  • jsacksick committed ca939311 on 8.x-2.x 
    Issue #3328849 followup by jsacksick: Fix the  variable assignment in...
jsacksick’s picture

jsacksick commented
Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.