Closed (fixed)
Project:
Comment Upload
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Jun 2009 at 19:03 UTC
Updated:
10 Sep 2010 at 15:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
toemaz commentedI had the same problem.
Find the patch attached.
Comment #2
toemaz commented@ankheg I made a patch based on your initial post but I didn't get it to work yet. Have you been able to actually solve the issue or was your proposed solution just a hunch?
Comment #3
ankheg commentedYes, the problem's solved. Your patch seems ok.
Comment #4
netaustin commentedPatch looks good. Committed, thanks!
Comment #6
EllECTRONC commentedThis patch works great! Thanx toemaz.
Comment #7
lewmich commentedProblem came back after updating Drupal to 6.14... It is weird because I have the newest Comment Upload module with patch commited... Someting else changed?
m.
Comment #8
toemaz commented@lewmich
It has been committed: http://drupal.org/cvs?commit=237422 and so it's available in the D6 dev package.
Comment #9
EllECTRONC commentedupdating Drupal to 6.14 and problem came back...
tested: 6.x-1.0-alpha5, 6.x-1.x-dev
No preview if text olny and if text and attach
don't wanna back to 6.13...
Comment #10
toemaz commentedThere has been an FAPI change in 6.14. See for similar bug report http://drupal.org/node/579900
This 'new' issue has nothing to do with this one so resubmit a new issue.
Comment #11
boobaaSubscribing. Any chance for a release soonish with this change included?
Comment #12
tilmankoester commentedStill isn't working in D6.16 with the latest dev version of comment_upload.
It's definitely the language, because switching to English enables comment preview again.
Comment #13
.zLaW commentedI just applied the patch to 6.x-1.0-alpha5 on a german site running Drupal 6.19
The patch works fine here. Users with de as language are now able to preview their comments.
Thank you!
Helmut
Comment #14
toemaz commented@zLaW it would have been more useful if you use the dev rather than alpha 5 and see whether it was still applicable. That way we could place this issue on RTBC.
Comment #15
.zLaW commented@toemaz - sorry, I didn't mean to be useless ;-)
I dl'd 6.x-1.x-dev today and tried it on my Drupal 6.19 site.
The dev version contains the changes found in the patch and preview works fine for users in german on this site.
Helmut
Comment #16
toemaz commentedSo let's put it on fixed then.