onFormProcessed not being called

[getgrav]

onFormProcessed not being called

[flaviocopes]

Are you getting an error about captcha above the comment form, once you enter the comment?
You must configure the recaptcha parameters, or delete the captcha processing.
I'll disable captcha by default in the next update, with instructions on how to activate it, as it needs the site/secret keys to work.

[flaviocopes]

f25fbd7

[superlou]

No error on captcha before or after submitting the form. I gave it my Recaptcha keys and the check itself seems to work fine.

[flaviocopes]

If you just add the 'addComment' process action to the form, is it called?

[superlou]

Sorry, how would I do that? In user/config/plugins/comments.yaml I see a form: process: addComment: item, but it's empty.

[flaviocopes]

This should work, for example:

enabled: true
form:
    name: comments
    fields:
.......
    process:
        - addComment:

(mind the correct tabs indentation)

[superlou]

I'm not following, sorry. In comments.yaml, I have the addComment item, but I think that came by default. As shown in the first code snippet in my first post, a log message in onFormProcessed doesn't show up by tailing the log.

[flaviocopes]

I'm not sure why it should not! PM me your user/ folder via Gitter so I can test

[superlou]

I recreated the issue on Linux Mint 17, Apache/2.4.7, PHP 5.5.9. I used the grav core + admin package for version 1.0.4. I installed the current comments package with recaptcha disabled. I had the same symptoms with no comments being added. It seems an unlikely thing, but is anyone successful on a LAMP server?

[khanduras]

I'm getting something similar - it doesn't look like it's processing the 'addcomment'
I'm using nginx/1.4.6 PHP 5.5.9-1ubuntu4.5

[flaviocopes]

Just spinned up a clean site on Linux and I recreated the problem. Not sure why it didn't happen on my test environment before.
Can you test my fix? a6476ab

[superlou]

I tested it on my localhost development site and on my blog, both running LAMP on Mint/Ubuntu, and the fix seems to work.