Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
There is something wrong with the attach button in the issues comments. Happened twice today:
Any change in the issue description is discarded after clicking on the attach button.
possible repro:
1. open an issue
(add description if you want)
2. attach a file to the issue comment
3. change the description field
(I am not sure if I clicked preview here or before)
4. save the comment
bug: the changes in point 3 are not saved.
Comment | File | Size | Author |
---|---|---|---|
#3 | 379470-test.txt | 12 bytes | dww |
#2 | 379470-test.txt | 12 bytes | dww |
Comments
Comment #1
Pasquallesorry, I mean the comment field, not description field..
Comment #2
dwwTesting if I can reproduce this
Comment #3
dwwTesting again.
This time with a preview...
Comment #4
dwwWow, yup. #2 was supposed to have another line that I added between when I attached the file and pressed "Save". #3 has the extra line I added, but only because I previewed the comment first.
Furthermore, I could reproduce this locally, not just on d.o. Which means it really is a bug in the code somewhere, not just a d.o configuration problem. ;)
Given that we're talking the attach button here, I'm assuming this is a comment_upload bug, not project_issue. But, I could be wrong... ;) If so, we can reclassify to project_issue.
Comment #5
Heine CreditAttribution: Heine commentedComment #6
netaustin CreditAttribution: netaustin commentedComment #7
dwwNote: netaustin was able to reproduce this a) on his laptop and b) not using project issue nodes at all. Definitely a comment upload bug. He's pretty sure he knows how to fix it, and we should have a patch deployed tomorrow afternoon for testing.
Comment #8
sunConfirming.
Comment #9
introfini CreditAttribution: introfini commentedI also can confirm this issue without using project module.
introfini
Comment #10
hass CreditAttribution: hass commented+
Comment #11
netaustin CreditAttribution: netaustin commentedFixed in 1.7.2.22, new alpha release cut.
Comment #12
dwwThanks for the fix! Now we just have to test this and deploy it on d.o. ;)
Comment #13
dwwFrom Druplicon in IRC:
@netaustin: Please use issues to communicate such things -- there's no guarantee I'll see such messages in IRC... ;)
I couldn't replicate what you're talking about in the above warning with a "minor problem with the project changer". Seemed to work for me... Can you be more specific about what's not working?
However, the thing that seems to be completely broken with revision 1.7.2.22 of comment_upload.module is that if you try to attach multiple files to the same comment, the first instance of the attach button works fine, but then the 2nd one hangs forever. With revision 1.7.2.21, (what's on d.o) the first works fine, the 2nd ends up doing a non-JS "preview" style attach with a new page load, the 3rd works directly via AHAH, the 4th reloads the page, etc. See #377460: AHAH enhanced comment_upload_js works for 1 attachment for more on the currently clumsy (but still functioning) behavior. However, revision 1.7.2.22 is a regression from that, since you can no longer attach more than one file at all. That's the only reason I mention it here...
Also, I notice that revision 1.7.2.22 includes a new feature -- the setting to prepend the node id to uploaded file names. Can you please not change multiple things in the same commit? ;) Always run "cvs diff" before you commit, so you see exactly what you're committing. Can we leave the code for #88911: Append node ID to project issue file attachments out of fixing this bug and deal with that in its own issue?
Thanks!
-Derek
Comment #14
netaustin CreditAttribution: netaustin commentedFixed in 200394. The node ID bit has been removed.
Comment #15
dwwStill not working in local testing... Multiple attach sometimes hangs indefinitely. Sometimes I get a page full of JSON output at /comment-upload/js while trying to attach. Etc.
@netaustin: can I make a few requests:
A) Please post a patch for review before committing it.
B) Please get changes tested and vetted before tagging another official release.
Thanks! ;)
Comment #16
netaustin CreditAttribution: netaustin commentedRefer to this issue and patch for the hanging attachments. The issue description is no longer discarded after clicking attach, so marking this as fixed.
Comment #18
Anonymous (not verified) CreditAttribution: Anonymous commentedThis is still an issue. It has happened to me two times today while posting files (with descriptions) in the issue queues. Another report of it appeared here: http://drupal.org/node/377596#comment-1564986
Comment #19
dwwThe fix just needs to be deployed on d.o, as expressed via the needs drupal.org deployment tag.
However, we can't do that until we figure out WTF is going on with #441216: upload periodically hanging in Safari, Opera and perhaps others
Comment #20
netaustin CreditAttribution: netaustin commentedSince #441216 is fixed, this is fixed also, as of 221560.
Comment #21
dww6.x-1.0-alpha5 is now deployed on d.o.
http://drupal.org/node/445748#comment-1675652 proves this is fixed: I added the "Thanks, Gabor, for committing!" part after attaching that file, didn't hit preview, and lo, it's still there. ;)
Yay!