Needs review
Project:
Comment form above comments
Version:
6.x-1.1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Aug 2009 at 07:24 UTC
Updated:
5 Aug 2011 at 09:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
if and a commentedHeh. Similar problem here and now.
I'm trying to theme the comment form. It is done but when I send some text, drupal redirect me to /comment/reply/%node
BUT there are no more comment-wrapper defined in comment-wrapper.tpl.php
AND no comment has been sent
Thanks
http://ifanda.cz
Comment #2
drupalftw commentedI don't really know the inner workings of the comment module nor why certain variables is set in comment_form_above_comments. But I have a check if we have a comment/reply request. This is proberbly not that nice but it will have to work for now, atleast for me.
original:
fix:
Comment #3
matthew petty commentedThe fix above is working for me. Will this be committed or is there a better way of doing it?
Comment #4
rapsli commentedthis worked for me too...
Comment #5
rapsli commentedhere is the patch
Comment #6
denes.szabo commentedI just have used this module and I found a strange thing:
I am using the better_formats module (http://drupal.org/project/better_formats). When I used it with the CMAC module the filter formats did not work.
I found: CFAC module gets the comment form twice. But the system caches the form and in the second case the better formats can not able to run (not deeply understand, but it is not matter now).
So I changed this module a little bit:
In the preprocess I not just drop the box by $vars = array() but I saved it as the whole form.
Then in the wrapper preprocess I put the saved form into the $variables. So does not need to recreate the whole comment form and it is works with the better formats module too.
I made a patch. (My first patch...) I am inserted #2 patch in it (It had a small code typo).
Comment #7
jefftrnr commentedI'm having trouble viewing the preview from a comment/edit form. It submits fine.
Comment #8
jefftrnr commentedI added the following change to your patch - is this correct?
from
to
Comment #9
denes.szabo commentedIt is probably correct but your solution uses two preg match. I think, we can use a simpler solution:
the pattern can be this: '`(comment/edit|comment/reply)(.*)`'
So the line can be: if (!preg_match('`(comment/edit|comment/reply)(.*)`', $_REQUEST['q'])) { ...
Comment #10
video izle commentedThank you
http://www.videoizle18.org
http://www.magazintr.net
Comment #12
fax8 commentedSame issue here. Patch #5 worked great.
Setting this as a bug and as critical.
Comment #15
denes.szabo commentedDoes the patch #5 work with the better formats modul? http://drupal.org/project/better_formats
If not (I think not) please see the #6 patch.
Comment #18
Rhino commentedSubscribing. Solution #2 worked for me.
Comment #21
bymegacom commentedI'm having trouble viewing the preview from a comment/edit form. It submits fine.
dizi izle
Comment #22
vincentcarl commentedUs here in scottsdale carpet cleaning trying to use the comment-wrapper.tpl.php template to wrap the comment area in a specific div. I would also have just assumed that the comment/reply page would be wrapped in this same template, but it doesn't appear to be. I need to wrap this comment/reply output in the same div. Have been struggling with what shouldn't really be happening for hours.
I do see this comment-wrapper.tpl.php being used on the actual node page where the inline comments are. Just not on the comment/reply page - it seems to be assembled differently?
Comment #24
Rhino commentedI want a "report for spam" button. See above spam comment. The issue tags were full of strange keywords too.
Edit: (and I see now that I didn't manage to delete them all, sorry about that)
Comment #30
computersimple commented1 - I am not sure why a module that does not work is available
2 - I am not sure why this adding comments form above comments isn't an option in drupal core install
I installed this module and it doesn't work it should just say this is on the module page (i.e. This module is not a complete fix, it only works in certain cases like if you only want the module above and only if your user clicks save and not preview) and perhaps adding beta or dev to the download listing would be helpful so that people aren't relying on it as a fully developed module.
Comment #31
fax8 commentedIt looks that the maintainer of this module is not working anymore on it, that's why this bug hasn't been fixed. This happens sometimes.
Anyway, there are some patches above which I used and worked.
If you want to help, you could test the above patches and report if they work or not. If you have programming experience you can ask to became this module maintainer... usually complaining in the FLOSS community doesn't work, it's better to roll up your sleeves and give your help.