Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added reCAPTCHA && Google Analytics support #33

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lyserbin
Copy link

With these patches I added reCAPTCHA and Google Analytics support to phpsqlitecms.

You can edit the reCAPTCHA settings under 'Spam protection' section in the administration panel, for Google Analytics I added a section under the administration menu.

After adding your site and your secret reCAPTCHA keys, you can select
where do you want to add the reCAPTCHA check, by now you can add it to:

  • users login (/cms/)
  • users comments on commentable posts
  • mail sent through formmailer

In the same way you just have to add your Analytics Tracking ID, check the checkbox and you are right.
Google Analytics script will be loaded in every page with the 'default.tpl' template (so administration pages won't be tracked).

It also prevents to randomly add Google's script in the page if you haven't enabled the feature that needs them, so it should be good also for the ones who don't want Google in their way ;)

In the end I edited the scripts and stylesheet position, so the page can be readden 'above the fold' and gets loaded even if cdns are having problems.

lyserbin added 4 commits June 28, 2015 22:27
With this *huge* patch I added reCAPTCHA support to phpsqlitecms.
You can edit the reCAPTCHA settings under 'Spam protection' section in
the administration panel.
After adding your site and your secret reCAPTCHA keys, you can select
where do you want to add the reCAPTCHA check, by now you can add it to:
* users login (/cms/)
* users comments on commentable posts
* mail sent through formmailer
It also prevents to randomly add Google's script in the page if it isn't
enabled, so it should be good for the ones who don't want Google in
their way ;)
fixed some bugs too
This time I added an option in the administration menu.

If enabled it adds Google Analytics snippet code to all pages except
administration's ones.

If disabled any additional code will be included in the page.
@lyserbin lyserbin changed the title Added reCAPTCHA support Added reCAPTCHA && Google Analytics support Jun 30, 2015
@theAsgar
Copy link

theAsgar commented Jul 7, 2015

Captcha-inventor of the possibilities of modern text recognition are aware and already hindering therefore (even completely unnecessary) human visitors with barely clearly recognizable spelling - and they have the even factored in since they equally provide a link for another Captcha hoping that hidden string might therefrom be guessed correctly.
Blind are hardest hit by the Captcha-delusion. Their access is therefore entirely denied, why captchas on barrier-free pages and may not be used. It should also be clear that there can be no barrier-free captcha. If the string in addition to the Captcha for disabled in other ways - for example, quite simply as an alternative text - would be presented, could be equal to waive the Captcha to the delight of all.
Please no Captcha.

@lyserbin
Copy link
Author

lyserbin commented Jul 7, 2015

TL;DR: You are free to not enable it

@theAsgar
Copy link

theAsgar commented Jul 7, 2015

I know there are so many beautiful barrier-free alternative without captcha. :)

@lyserbin
Copy link
Author

lyserbin commented Jul 7, 2015

You can still not enable it and implement your own way to block bots 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants