"Inline Registration allows anonymous users to register via the node/add page, thus removing a step/barrier from the user actually publishing content." However, when the registration form is displayed via this module, the Terms of Use fieldset is not displayed.

Please consider making the change necessary to ensure compatibility with Inline Registration.

CommentFileSizeAuthor
#1 728012-terms_of_use-inline_registration.patch3.91 KBmrtoner

Comments

mrtoner’s picture

mrtoner commented
Status: Active » Needs review
StatusFileSize
new 728012-terms_of_use-inline_registration.patch3.91 KB

This patch seems to work well to cause the terms of use to show when a registration form is displayed by Inline Registration.

kars-t’s picture

kars-t commented
Status: Needs review » Needs work 
+++ terms_of_use.module	2010-02-27 17:56:28.000000000 -0800
@@ -126,53 +126,55 @@ function terms_of_use_admin_settings_val
- * Implementation of hook_form_form_id_alter().
+ * Implementation of hook_user().

Could you please explain why are you changing hooks here?
I can't get a grip on what you are doing because it seems you are altering TOU by it's core. Please explain what are you doing and why.

Powered by Dreditor.

mrtoner’s picture

mrtoner commented

Because I don't know enough about Drupal's Forms API, I'll commence talking out of my butt here. Since Inline Registration uses drupal_retrieve_form() to display the registration form, hook_form_FORM_ID_alter() is never fired. Or maybe it's getting fired out of sequence with hook_form_alter, which Inline Registration implements. In any case, TOU never alters the user_registration form when it's displayed by IR.

By changing this to hook_user, TOU displays on the form when retrieved by IR.

Clément’s picture

Clément commented
Version: 6.x-1.10 » 6.x-1.12

Hello,

I have the same problem. Does this patch also works for version 6.x-1.12? Thank you.

kars-t’s picture

kars-t commented
Status: Needs work » Closed (won't fix)