Hello,
Maybe it should be issue for ajax comment module, but I'm not sure.
When I ajax comment module is enabled, Cache Status is "Caching Canceled".
See attached file.

CommentFileSizeAuthor
#10 Caching Canceled-4.jpg19.18 KBAlex_eav
#7 interdiff.txt1.35 KBznerol
#7 authcache_with_ajax-2658720-6.patch1.05 KBznerol
#7 interdiff.txt1.35 KBznerol
#7 authcache_with_ajax-2658720-6.patch1.05 KBznerol
#5 Caching Canceled-3.jpg33.9 KBAlex_eav
#4 authcache_with_ajax-2658720-4.patch874 bytesznerol
#3 Caching Canceled-2.jpg28.91 KBAlex_eav
Caching Canceled.jpg38.11 KBAlex_eav
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Alex_eav created an issue. See original summary.

znerol’s picture

znerol CreditAttribution: znerol commented

Please apply the patch linked from #2460953-20: Support Ajax Webforms. This will make the debug message a little bit more useful. After that take another screenshot from the message and post it here.

Alex_eav’s picture

Alex_eav CreditAttribution: Alex_eav commented
FileSize
Caching Canceled-2.jpg28.91 KB

Here it is.

znerol’s picture

znerol CreditAttribution: znerol commented
Category: Support request » Bug report
Status: Active » Needs review
FileSize
authcache_with_ajax-2658720-4.patch874 bytes

Oh, that looks like a bug. The comment module should be supported out of the box and that specific value element was always there. Seems that test coverage is lacking in this area.

The fix seems easy enough though.

Alex_eav’s picture

Alex_eav CreditAttribution: Alex_eav commented
FileSize
Caching Canceled-3.jpg33.9 KB

I've applied patch.
The message in Authcache debug have changed but result still the same.
See attached screenshot.

Alex_eav’s picture

Alex_eav CreditAttribution: Alex_eav commented

And I can't understand, what do you mean "The comment module should be supported out of the box "?
When "ajax comment" turned off, Authcache work good without any problem. Only together with ajax Comment Module, cache status is "Caching Canceled".

znerol’s picture

znerol CreditAttribution: znerol commented
FileSize
authcache_with_ajax-2658720-6.patch1.05 KB
interdiff.txt1.35 KB
authcache_with_ajax-2658720-6.patch1.05 KB
interdiff.txt1.35 KB

Uh :/

Let's see whether that helps. A word of warning: This is completely untested and not ready for production. Feedback welcome.

The last submitted patch, 7: authcache_with_ajax-2658720-6.patch, failed testing.

Alex_eav’s picture

Alex_eav CreditAttribution: Alex_eav commented

I don't know, maybe I've done something wrong, but I applied patch from this file:
https://www.drupal.org/files/issues/interdiff_18935.txt
And it's worked!
Result on screenshot.

Alex_eav’s picture

Alex_eav CreditAttribution: Alex_eav commented
FileSize
Caching Canceled-4.jpg19.18 KB

Screenshot

matysek145’s picture

matysek145 CreditAttribution: matysek145 commented

#7 works for me, thanks!

  • znerol committed 6df2c6c on 7.x-2.x 
    Issue #2658720 by znerol, Alex_eav: Authcache with ajax comment module
znerol’s picture

znerol CreditAttribution: znerol commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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