Closed (fixed)
Project:
U Create
Version:
6.x-1.0-beta4
Component:
ucreate.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Mar 2009 at 19:56 UTC
Updated:
30 Aug 2010 at 17:22 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rodanx86 commentedSo 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.
Comment #2
zengenuity commentedI have fixed these issues in my version. Here is a patch for ucreate.module that gets rid of the dprint_r() and email errors.
Comment #3
rodanx86 commentedI 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?
Comment #4
alex_b commentedDoes the patch apply to 6.x ?
Comment #5
JamesAn commentedComment #6
JamesAn commentedThe 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.
Comment #7
xcorex commented#6 didn't work for me.
Comment #8
alex_b commentedPatch in #2 does not comply with coding standards.
Comment #9
timl commentedsubscribe +1
this is a show stopper for me
Comment #10
Anonymous (not verified) commentedLooks like dprint_r() has already been removed. Everything should work fine now.