Hoovilation

Sent, thanks again for the quick updates.

Excellent, I have the patch to support multiple ips in the “Block my IP” option, how can I submit this to you?

Sounds good I should be able to handle that. I do have another question for you though. In cfenqueue_plugin_scripts why is the cfmonitor.css enqueued? It seems like this css is only needed for the Administration settings page where it is already added. The css also has some generic things like .title with !important attributes that overrides the css we have on our site.

And on a side note the “Block my IP” currently only supports one IP address. I’m thinking of adding the ability to allow for multiple via a comma separated list. Is this something you would like to push into your code base? I could submit a SVN patch.

Hello ReneHermi,

So let us say that in one of my template files like the header I am outputting an ad. Before I do that I would rather perform a check on the user to see if they pass all the criteria in the Click Fraud settings like:

if (clickfraud_user_can_see_ads()) {
  echo 'ad code';
}

Currently this is all handled client side by the inclusion of the hideads_min.js. I suppose I can add a method myself that performs the same checks you have in cfenqueue_plugin_scripts but I just wanted to see if maybe this already exists or is something you were planning on adding. I hope that is more clear.

Hello ReneHermi,

I applied the plugin update and yes that did the trick and corrected the issue ??

I haven’t looked through the plugin code so maybe this already exists, but have you considered creating some method that can be executed server side to check if ads should be displayed for the user? This could be useful to drop in templates if it will return true/false based on the Click Fraud settings and user information.

Thanks!