I installed False Account Detector on a new drupal site, no other users but me the Admin. ;-)

After enabling it, I went to the module page. I.e., the one with the tabs: Default, Blocked, Whitelisted, Search, Settings.

I tried each tab in order ( and of course nothing was found because it is a virgin system with no users ). When I got to the search tab, I got the following error:

warning: Missing argument 1 for false_account_searchfa() in
...
/sites/all/modules/false_account/false_account.module on line 189.

Then the search field shows up.

I think the error is simply an artifact of passing an empty array to the function and not checking it. (Sorry
cannot help being a programmer...)

Thanks,

Louay

CommentFileSizeAuthor
#2 false_account.module.245432.patch1012 bytesjaydub

Comments

aznboy’s picture

I got the same error.

jaydub’s picture

Status: Active » Needs review
StatusFileSize
new1012 bytes

function needs a default value to the account argument

aznboy’s picture

thanks

introfini’s picture

Hi,

I need to change the way search is made. It currently has a problem, if the users accounts are blocked they don't show up.

@jaydub: thanks for the patch

Regards,
introfini

introfini’s picture

jaydub I've committed your patch for now, until I fix the other problem.

Thanks for helping out.

introfini

everydayjones’s picture

What am I supposed to do with this patch. Very new to Drupal.

introfini’s picture

just download the dev version, it's included already in that version.

introfini’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

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