Hello,

I just installed this module. When I save the configuration at admin/config/people/fboauth, I get the following error message.

Notice: Undefined index: fboauth_user_fields in fboauth_field_form_submit() (line 60 of C:\wamp\www\mysite\sites\all\modules\contrib\fboauth\includes\fboauth.field.inc).
Warning: array_filter() expects parameter 1 to be array, null given in fboauth_field_form_submit() (line 60 of C:\wamp\www\mysite\sites\all\modules\contrib\fboauth\includes\fboauth.field.inc).

BTW, I noticed that the function fboauth_field_form_submit() is receiving two arguments, but using only one.

CommentFileSizeAuthor
#5 fboauth_no_user_fields-d7.patch642 bytesquicksketch
#3 error.jpg40.39 KBdjmy

Comments

djmy’s picture

djmy commented

just installed it as well and i get the same issue too

areynolds’s picture

areynolds commented
Status: Active » Closed (duplicate)

The patch and advice in http://drupal.org/node/1305340 should resolve this issue; try those out and re-open this issue if it's a problem. Quicksketch should be committing that code soon, but let us know on the other issue if the patches work.

Thanks for the report!

djmy’s picture

djmy commented
Status: Closed (duplicate) » Active
StatusFileSize
new error.jpg40.39 KB

hi areynolds! Unfortunately the patch didn't fix it. I attached a screenshot of the error.

phantom21’s picture

phantom21 commented

I had already applied your patch. In the absence of the patch, I could not even get the configuration screen to show. After applying your patch I could see the config screen, but as soon as I click the save button, I get the above errors.

quicksketch’s picture

quicksketch
he/him
commented
Title: admin/config/people/fboauth throws an error message when configuration is saved » Settings page throws an array_filter() warning when saving if there are no fields on users
Priority: Major » Minor
Status: Active » Fixed
StatusFileSize
new fboauth_no_user_fields-d7.patch642 bytes

I found that this problem is easy to reproduce on a clean install of Drupal 7. The error occurs whenever there are no fields assigned to the User object, so the $form_state['values']['fboauth_user_fields'] value is not even set, which causes the array_filter() warning.

This patch fixes the warning, but even without the patch the warning is merely cosmetic and doesn't affect the functionality of the module. I'll be putting out a 1.2 release after addressing other issues in the queue. I've committed the attached patch.

djmy’s picture

djmy commented

Thank you! Awesome module!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

removed the site name