With the latest version of 8.x-1.x-dev I can create a registration page for a role, but when creating an account based, on this, I get:

Uncaught PHP Exception Drupal\Core\Entity\EntityStorageException: "Entity validation was skipped."

I tracked this down to the validation handler attached to the submit button instead of the form (A similar issue is documented here: https://www.drupal.org/node/2621856#comment-10654312 )

The attached patch fixes this.

CommentFileSizeAuthor
#11 multiple_registration-entity_validation_error-2823974-11.patch670 byteswelly
#2 multiple_registration-entity_validation_error-2823974-2.patch1.2 KBrjjakes

Comments

rjjakes created an issue. See original summary.

rjjakes’s picture

rjjakes commented
StatusFileSize
new multiple_registration-entity_validation_error-2823974-2.patch1.2 KB
rjjakes’s picture

rjjakes commented
Status: Active » Needs review
MarcoDeJong’s picture

MarcoDeJong commented

The patch seems to enable registration with the correct profile. Yay! However, before I applied the patch, the 'registration page' would not show me all user related tabs (e.g. create new account/forgot passw). After applying the patch, I see all user related tabs (which I now hide through some block permissions).

rjjakes’s picture

rjjakes commented
Status: Needs review » Needs work
rjjakes’s picture

rjjakes commented

@MarcoDeJong this is working as one would expect it to.

On the standard /user/register page, I see: "Log in", "Create new xxxx account", "Create new account(active tab)", "Reset your password"
On my custom page I see: "Log in", "Create new xxxx account(active tab)", "Create new account", "Reset your password"

The default is to show the "Log in" and "Reset your password" on the /user/register page, so why not on the custom registration page?

rjjakes’s picture

rjjakes commented
Status: Needs work » Needs review
rjjakes’s picture

rjjakes commented
Status: Needs review » Needs work
welly’s picture

welly commented
Version: 8.x-1.x-dev » 8.x-1.4
Status: Needs work » Reviewed & tested by the community

Tested this patch on latest release which also had the same problem, works well.

Valvog’s picture

Valvog commented

Comment on applique un patch ?

welly’s picture

welly commented
StatusFileSize
new multiple_registration-entity_validation_error-2823974-11.patch670 bytes

After testing the patch again, it looks like it didn't work. It fires an error on

$aliasStorage->validate();

Error being:

Call to undefined method Drupal\Core\Path\AliasStorage::validate()

Not sure where the validate method came from, however it doesn't exist in Drupal\Core\Path\AliasStorage.

I've attached a patch without the validate function and it seems to be working. Would appreciate a test/feedback.

Not knowing this codebase very well, I'm not actually sure if there is any actual validation going on but at the very least this module now works without triggering the original error.

rjjakes’s picture

rjjakes commented
Status: Reviewed & tested by the community » Needs review
Oleks Iv’s picture

Oleks Iv commented

Fixed in the new 8.x-1.5 version

Oleks Iv’s picture

Oleks Iv commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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