Enabling fb_registration causes WSOD due to a syntax error.
Patch removes superfluous )

CommentFileSizeAuthor
#2 fb_reg_port.patch2.03 KBalasda
fb_registration.patch610 bytesalasda

Comments

Dave Cohen’s picture

Dave Cohen commented
Title: WSOD when fb_registration is enabled » fb_registration.module not upgraded to D7
Category: bug » task
Status: Needs review » Active
Issue tags: +fb-volunteer, +fb-d7

Not sure how that got in there. But don't expect the module to work. It needs to be upgraded to D7.

alasda’s picture

alasda commented
StatusFileSize
new fb_reg_port.patch2.03 KB

Worked surprisingly well given the circumstances!.

See attached patch for port to d7

MacRonin’s picture

MacRonin commented

The FB modules I downloaded recently have a later date than your patch March 23 instead of March 22.
When I run the patch I get the following error.

***************
*** 97,103 ****
  
      _fb_registration_extract_fb_fields($fb_fields, $form);
  
-     $redirect_url = url(request_path()), array('absolute' => TRUE));
      $fields = json_encode($fb_fields);
  
      if (FALSE) {
--- 98,104 ----
  
      _fb_registration_extract_fb_fields($fb_fields, $form);
  
+     $redirect_url = url(request_path(), array('absolute' => TRUE));
      $fields = json_encode($fb_fields);
  
      if (FALSE) {

This would appear to be because the extra paren after request_path was already fixed in the distributed file.

Now if I can get this to actually do anything I'll uploaded a updated patch file. But till then I just wanted to log this for anyone else who might also encounter the error.