Closed (won't fix)
Project:
AJAX Comments
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2012 at 21:03 UTC
Updated:
27 Oct 2014 at 06:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
nathanhilbert commentedTry attached patch to correct the target ajax callback for forums.
Comment #2
muschpusch commentedWhat exactly is the problem when using advanced forums?
Comment #3
chimir commentedВ advanced forum на срабатывает кнопка "ответить", не появляется форма для ответа
Comment #4
chimir commented@ nathanhilbert thank you
Comment #5
formatC'vt commentedtry latest dev, please
Comment #6
edaa commented@formatC'vt
Thanks for your work, I have tried the latest 7.x-1.x-dev, the new comment can added, but there are 2 issues:
1) The comment order is wrong when I post multiple comments, since Advanced Forum resets $variables['class_array']:
so no 'ajax-comment-wrapper' class in real comment.
2) Advanced Forum changes the comment permalink to something such as #1, #2, #3 ..., when posting new comments, the permalink of ajax response is always '#2'.
Comment #7
formatC'vt commented1) I think the best way is ask a maintainers of Advanced Forum do not do this
$variables['classes_array'] = array(), because this is can crash any other modules too, or add for this option in admin interface (this option can be enabled by default for overwriting existing classes).2) We are using default comment load feature, other modules who changing/altering something must do this every time.
Comment #8
formatC'vt commented2) May be we can fix it, i check it on next week
Comment #9
formatC'vt commented2) Hmm, this is can be easy to fix. Can you please find in
ajax_comments_submit_jsfunction this line:$node = $form['#node'];and replace it with:
$node = node_load($comment->nid, NULL, TRUE);and check after this permalink.
Comment #10
edaa commentedThank you! I will give it a try.
Comment #11
edaa commentedduplicate of Keep default classes from Drupal core comments
Comment #12
formatC'vt commented1. I submit a patch
2. Do you test it?
Comment #13
edaa commented@formatC'vt
1. the patch works well.
2. sorry, I test it many times, it does not work.
Comment #14
formatC'vt commented2. Advanced forum don't support ajax content update at this time (they need a javascript behavior for update permalink value for all of comments on content update)