Using Drupal 8.0.0-beta15 and the dev version of mollom (8.x-1.x-dev) I am able to add my API keys on the admin config page (admin/config/content/mollom/settings) and add the user_register_form . When I go to user/register, I get:









Same thing happens when adding user_pass to mollom configuration and going to user/pass.

BTW, with 8.x-1.0-alpha2 the admin page errors out. I'll add another ticket for that version.

CommentFileSizeAuthor
#4 2569603-4.patch560 bytesmbaynton
mollom-user-register.png71.82 KBesod

Comments

esod created an issue. See original summary.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

With the same beta15 and mollom 8.x-1.x HEAD, I do not have this problem (PHP 5.6.13), the module triggers no erreor anywhere, but does not appears to do anything on any form where it is added, even in forced captcha mode. No mollom markup or js is visible, all content is accepted (even from non-admin users), even marked "spam" in testing mode, and no entries appear in the dblog even with "log all events".

esod’s picture

esod commented

With mollom branch 8.x-1.x, user/register and user/pass still fatal error for me when I submit either form (PHP 5.5.28). The same forms submit without problem when I delete the form's protection at admin/config/content/mollom/form/user_register_form/edit. The Unprotect drop down doesn't get the form working again for me either. Also the form fields on user/register are sticky with previously input values.

I don't see an 8.x-1.x HEAD branch or tag in the list of branches.

I also don't get the captcha or text analysis on the form. Console is clear of errors.

mbaynton’s picture

mbaynton
he/him
Location Minneapolis, US
commented
Status: Active » Needs review
StatusFileSize
new 2569603-4.patch560 bytes

$protected_form_id is sent on to EntityStorageBase::load, which handles the scalar-to-arrayness before delegating to loadMultiple. As it is EntityStorageBase::loadMultiple attempts to load an array of an array, not an array of an id.

tetranz’s picture

tetranz commented

This patch is a duplicate of https://www.drupal.org/node/2560147

mbaynton’s picture

mbaynton
he/him
Location Minneapolis, US
commented

Ha, hows about that. Seems the problem has more than one symptom. Now the real hard part is deciding which patch is better...

eshta’s picture

eshta commented
Status: Needs review » Fixed

Darn... this was fixed while I was doing some other stuff....
http://cgit.drupalcode.org/mollom/commit/?h=8.x-1.x&id=73fd7c7aa428bba7c...
But you and tetranz win the I'll buy you a beer at the next Drupalcon commit award.

Status: Fixed » Closed (fixed)

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