starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-25 13:05:53 +03:00

[GH-ISSUE #323] 🐛 [Compose] Detaching & Reattaching of files does not work #277

New issue

Closed

opened 2026-02-25 21:34:38 +03:00 by kerem · 7 comments

kerem commented

2026-02-25 21:34:38 +03:00

Owner

Originally created by @ulfgebhardt on GitHub (Mar 6, 2019).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/323

Originally assigned to: @jasonmunro on GitHub.

🐛 Bugreport

[Compose] Detaching & Reattaching of files does not work

Consider following Situation:

You want to send a File and start writing the EMail and attach the File
Now you notice there is a mistake in the File while writing
You correct the Mistake, detach the file, and try to reattach it.
The File does not appear, you have the start another compose

This is probl. the case since the file name has not changed.
(I did not try with a changed filename)

Version & Environment

Rev: 70a3d129f0493d7885176040b39a69d51c422921

OS: [Debian]

Steps to reproduce

Attach File
Detach File
Reattach same File (or changed File with same name/path)
...
Profit

Originally created by @ulfgebhardt on GitHub (Mar 6, 2019). Original GitHub issue: https://github.com/cypht-org/cypht/issues/323 Originally assigned to: @jasonmunro on GitHub. ## 🐛 Bugreport  ### [Compose] Detaching & Reattaching of files does not work Consider following Situation: - You want to send a File and start writing the EMail and attach the File - Now you notice there is a mistake in the File while writing - You correct the Mistake, detach the file, and try to reattach it. - The File does not appear, you have the start another compose This is probl. the case since the file name has not changed. (I did not try with a changed filename) ### Version & Environment  Rev: [70a3d129f0493d7885176040b39a69d51c422921](https://github.com/jasonmunro/cypht/commit/70a3d129f0493d7885176040b39a69d51c422921)  OS: [Debian] ### Steps to reproduce 1. Attach File 2. Detach File 3. Reattach same File (or changed File with same name/path) 4. ... 5. Profit

kerem

2026-02-25 21:34:38 +03:00

closed this issue
added the
bug

smtp

awaiting feedback
labels

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@jasonmunro commented on GitHub (Mar 20, 2019):

This appears to be a Javascript bug as the second attempt to attach the file never triggers a server side request. I think the problem is using a change event as part of the flow. Re-uploading the same file is not a change in state so it never triggers the actual upload. I will let you know when I have a fix.

@jasonmunro commented on GitHub (Mar 20, 2019): This appears to be a Javascript bug as the second attempt to attach the file never triggers a server side request. I think the problem is using a change event as part of the flow. Re-uploading the same file is not a change in state so it never triggers the actual upload. I will let you know when I have a fix.

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@jasonmunro commented on GitHub (Mar 20, 2019):

Should be fixed in the master branch

@jasonmunro commented on GitHub (Mar 20, 2019): Should be fixed in the master branch

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@jasonmunro commented on GitHub (Jul 3, 2019):

@ulfgebhardt I believe this is fixed so I'm closing this issue. If you are still seeing issues please re-open. Thanks!

@jasonmunro commented on GitHub (Jul 3, 2019): @ulfgebhardt I believe this is fixed so I'm closing this issue. If you are still seeing issues please re-open. Thanks!

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@ulfgebhardt commented on GitHub (Jul 3, 2019):

Hey @jasonmunro i am sorry that i did not respond in a timely manner - haven't updated cypht in quite some time. <3

@ulfgebhardt commented on GitHub (Jul 3, 2019): Hey @jasonmunro i am sorry that i did not respond in a timely manner - haven't updated cypht in quite some time. <3

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@marclaporte commented on GitHub (Mar 17, 2021):

@marclaporte commented on GitHub (Mar 17, 2021): Related: https://github.com/jasonmunro/cypht/issues/421

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@marclaporte commented on GitHub (Mar 17, 2021):

@ulfgebhardt Please retest with latest code.

Thanks!

@marclaporte commented on GitHub (Mar 17, 2021): @ulfgebhardt Please retest with latest code. Thanks!

kerem commented

2026-02-25 21:34:38 +03:00

Author

Owner

@ulfgebhardt commented on GitHub (Mar 17, 2021):

@marclaporte I will in time, I still work with quite the old version of cypht. Will update some day I guess <3

@ulfgebhardt commented on GitHub (Mar 17, 2021): @marclaporte I will in time, I still work with quite the old version of cypht. Will update some day I guess <3

kerem referenced this issue

2026-02-25 21:36:12 +03:00

[PR #277] [MERGED] Adding username of created user to output for easier debugging #778