We have a custom module that implements hook_nodeapi(), and we noticed that when the user profile content type was being validated during registration, we were getting an empty array passed to us during the "validate" OP.

We've traced the problem to the call to node_validate() in content_profile_registration_user_register_validate(); it's not passing the $form array in the call. The attached patch fixes this.

CommentFileSizeAuthor
content_profile_registration_node_validate.patch454 bytesguypaddock

Comments

guypaddock’s picture

guypaddock commented
Status: Active » Needs review
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Closed (won't fix)

Well it doesn't as the form isn't the node form. node_validate() has $form as optional argument, so it should be fine that way.

guypaddock’s picture

guypaddock commented
Status: Closed (won't fix) » Active

Sorry, but it has the same structure as the node form... if you can use CCK in the content type for the profile, which this module allows you to do, then CCK should have the ability to see the form.

As far as I know, there aren't any side effects to passing this along... plus, we need this patch for an upcoming installation profile, or we'll need to make a fork.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Active » Fixed

Very polite.. Anyway, when its important to you, I can commit it. I hope it won't cause bug for other modules.

committed.

guypaddock’s picture

guypaddock commented

Thank you. It's very much appreciated!

Status: Fixed » Closed (fixed)

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