My patch fix the bug.

CommentFileSizeAuthor
autoassignrole_content_profile_admin_form_submit.diff232 bytesthehong

Comments

pedrorocha’s picture

pedrorocha commented
Priority: Normal » Critical
Status: Active » Reviewed & tested by the community

I had the same error and this code, despite not being a patch file, fixed the error after i manually merged with my code.

It's ok to merge with the project.

ps: i'm puting this as critical, because it's an important feature that simply doesn't work with the current version of the module, so really needs this patch

Status: Reviewed & tested by the community » Needs work

The last submitted patch, autoassignrole_content_profile_admin_form_submit.diff, failed testing.

maddentim’s picture

maddentim commented
Priority: Critical » Normal

pedrorocha, I am not sure, but this diff (mostly because it is not a valid drupal patch...) just seems to the same thing that is there already. I don't think we need to call the function " content_profile_set_settings" because the submit function from content_profile already calls it. This function just needs to set up the autoassignrole settings to be properly saved by the submit. There is another issue that seems to have been my root cause. Maybe look into this one: http://drupal.org/node/1220228

I will try to submit a valid patch to the issue today (it doesn't have one either). I am removing the critical as this issue doesn't have much in terms of detail (like a full description of what the error really consists of... maybe it is really a dup of the other).