Ucreate breaks on submit.

rodanx86 - March 3, 2009 - 19:56
Project:U Create
Version:6.x-1.0-beta1
Component:ucreate.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs work
Description

When I try to create a user, when I hit submit, I get:

Fatal error: Call to undefined function dprint_r() in /sites/all/modules/ucreate/ucreate_og.module on line 42

I don't know why this is happening, so any help would be nice. Thanks.

#1

rodanx86 - March 4, 2009 - 02:49

So I commented out line 42 to see what happens. The page loads and reports that it has sent instructions but it displays this error right after it:

* warning: Illegal offset type in isset or empty in includes/path.inc on line 65.
* warning: Illegal offset type in includes/path.inc on line 70.
* Unable to send e-mail. Please contact the site admin, if the problem persists.

I am not a coder, so I don't know if this is caused by commenting out line 42. Hope this is more informative to someone else than it is to me.

#2

zengenuity - March 11, 2009 - 14:24

I have fixed these issues in my version. Here is a patch for ucreate.module that gets rid of the dprint_r() and email errors.

AttachmentSize
email_fix.6x.patch 3.23 KB

#3

rodanx86 - March 11, 2009 - 15:35

I applied the patch with no success.

I noticed that the patch is for ucreate.module, but the error is from the ucreate_og.module. Does that make a difference?

#4

alex_b - March 11, 2009 - 15:47
Status:active» needs review

Does the patch apply to 6.x ?

#5

JamesAn - March 12, 2009 - 22:01

#6

JamesAn - March 12, 2009 - 22:10
Component:ucreate_og.module» ucreate.module
Status:needs review» active

The patch in #2 looks like the changes put forward in #337800: Blank screen (ie. not for this issue).

After commenting out line 42: ucreate_og.module (the dprint_r function), change ucreate.module:
Line 193:
drupal_goto($_GET['q']);
to
drupal_goto();

The problem is that the variable $_GET['q'] might not be set, as in the URL doesn't have a "q=some/url" in it. The module doesn't check this and inadvertently sends in a NULL value (when the variable is not set) which trips up the path module.

The good thing about drupal_goto() is that it can automatically pick up the destination string from the URL, so you don't need to feed it the 'q' value. It'll get the destination value itself if you send nothing in, and - if there is no destination - it just brings you back to the original page: user/add.

#7

xcorex - March 19, 2009 - 20:35

#6 didn't work for me.

#8

alex_b - March 19, 2009 - 21:48
Status:active» needs work

Patch in #2 does not comply with coding standards.

#9

timl - April 6, 2009 - 23:47
Category:support request» bug report

subscribe +1

this is a show stopper for me

 
 

Drupal is a registered trademark of Dries Buytaert.