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.

Files: 

Comments

GuyPaddock’s picture

Status:Active» Needs review
fago’s picture

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

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

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

Thank you. It's very much appreciated!

Status:Fixed» Closed (fixed)

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