Comment attachment module enables comments to have files attached to it.
Suppose you have a forum and your forum members submits a post/comment and he/she want to upload a PDF or an image file attched to his post/comment.
The forum module which comes with the drupal core does not provide any option to attach an image or any file to a post/comment.
This module solves this simple problem by allowing an UI for uploading files as an attachment to a comment and also lists the files
attached to a particular comment.
We are using this module on our production site Debt Consolidation Care with drupal 7
Project's Page: http://drupal.org/sandbox/DebtConsolidationCare/1923150
Repository: git clone --recursive --branch 7.x-1.x http://git.drupal.org/sandbox/DebtConsolidationCare/1923150.git comment_attachment
Reviews of other projects:-
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | comments-fields.png | 24.3 KB | mnico |
Comments
Comment #1
abhijeetkalsi commentedHi,
there few error related to indentation and spacing, see this
http://ventral.org/pareview/httpgitdrupalorgsandboxdebtconsolidationcare...
Comment #2
DebtConsolidationCare commentedHi,
I have fixed the errors and warnings.
Please review this module.
Comment #3
cackle commentedHello,
from manual review:
1. You should add hook_uninstall to your_module.install
2. You should add comment module dependency because you use hook_comment_view_alter() which is providing by comment module.
3. In function theme_file_attachment you generate a lot of html. By drupal coding standarts it needs to divide business login from the view using templates. See hook_theme to understand how templates works.
In line 171
you hardcoded width of the image as php variable and it looks bad. I recommend you try to make it more responsive by styles.
Cackle
Comment #4
DebtConsolidationCare commentedHi,
I have fixed those three issues as per manual review by cackle.
Please review for further issues.
Comment #5
klausiWe are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)
Comment #6
mayank-kamothi commentedHi,
Manual review:
1] There is no any lable for file attachment field and it is better to add filedset for your attchment.
Mayank
Comment #6.0
DebtConsolidationCare commentedAdded 'Reviews of other projects' section with 3 links to 2 projects
Comment #7
DebtConsolidationCare commentedHi,
I have added a label and a description to the file attachment field.
Also I have completed manual reviews on 3 other project applications.
Please review this module once more.
Comment #8
veeraprasadd commentedHi,
I have setup your module in my local environment and manually reviewed. It works well.
I just found minor things :
Other than that everything is good.
Regards,
D. Veera Prasad.
www.drup-all.com
Comment #9
mnico commentedHello,
I have a question. What difference has this module with attaching the file upload field with field UI? In the administration of content types, you can attach multiple fields to comments, including file upload fields (I uploaded a screenshot).
Comment #10
klausiI agree with comment #9, why do you need to write a single line of code if Drupal core provides you with that functionality already?
Comment #11
veeraprasadd commentedYes klausi..I do have the same question. I thought he can go with version 6. But still we have already a module (http://drupal.org/project/comment_upload) in version 6.
Comment #13
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #13.0
PA robot commentedAdded review of mypuzzle_sudoku module