Option to disable captcha if logged in [SOLVED] with answer

[microbe]

Since people are manually approved when they register, we can assume that they are trusted. How about that if logged in no captcha for adding items. This also enables admins to add items without the pain of doing the captcha every time.

[microbe]

Hmmm...poking along because I am keen to make this happen I found this

/inc/PMF_Helper/Captcha.php

Code: Select all

if (PMF_Configuration::getInstance()->get('spam.enableCaptchaCode')) {

based on

Code: Select all

if (isset($auth)) {
    $loginTemplate = 'loggedin.tpl';

in index.php

I thought that using this might work && !isset($auth)

Code: Select all

if (PMF_Configuration::getInstance()->get('spam.enableCaptchaCode')  && !isset($auth) ) {

but it doesn't.

Am I heading in the right direction?

[microbe]

Oh how I love solving my own questions

I found that the $auth value wasn't being passed into the function and without searching on how to make it global (with maybe some implications elsewhere) I decided to pass the value through.

Since I only really want this behaviour on the add page, in /add.php I have made this change on line 78 - it should work in any other page too

Code: Select all

'captchaFieldset'       => PMF_Helper_Captcha::getInstance()->renderCaptcha(
            $captcha,
            'add',
            $PMF_LANG['msgCaptcha']
        ),

Change the code to

Code: Select all

'captchaFieldset'       => PMF_Helper_Captcha::getInstance()->renderCaptcha(
            $captcha,
            'add',
            $PMF_LANG['msgCaptcha'],'',$auth
        ),

to catch the value

and pass the value into the function in /inc/PMF_Helper/Captcha.php

Code: Select all

public function renderCaptcha(PMF_Captcha $captcha, $action, $legend, $error = '')

becomes

Code: Select all

public function renderCaptcha(PMF_Captcha $captcha, $action, $legend, $error = '', $auth)

and then test for it being set...

Code: Select all

if (PMF_Configuration::getInstance()->get('spam.enableCaptchaCode')   ) {

becomes

Code: Select all

if (PMF_Configuration::getInstance()->get('spam.enableCaptchaCode')  && !isset($auth)  ) {

Now there is a captcha on the add FAQ page for those not logged in and it is not rendered for those who are.

Sweet!

Hope it helps someone else.

[Thorsten]

Hi,

very nice... will add this to 2.8

bye
Thorsten

[Thorsten]

Hi,

added for 2.8.0-alpha2: https://github.com/thorsten/phpMyFAQ/co ... 97f78d7a94

bye
Thorsten

[microbe]

Cool, glad to be able to contribute.

I noticed your code is different to mine, mine works so do you see any problems leaving it like that?

[Thorsten]

Hi,

it's different because I added to the upcoming phpMyFAQ 2.8 alpha2 release which has a lot of improvements.

bye
Thorsten

[Clemendo]

Hi,

added for 2.8.0-alpha2: https://github.com/thorsten/phpMyFAQ/co ... 97f78d7a94

bye
Thorsten

It was a great relief and since 2.8.0. it is much more flexible. The improvements are significant

[jack9091]

very nice...