You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Dave Brondsema <da...@brondsema.net> on 2015/10/14 18:44:45 UTC
[allura:tickets] #8003 Bugs in attachments to comments
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** open
**Milestone:** unreleased
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Wed Oct 14, 2015 04:44 PM UTC
**Owner:** nobody
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Dave Brondsema <da...@brondsema.net>.
- **labels**: 42cc --> 42cc, sf-2, sf-current
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** in-progress
**Milestone:** unreleased
**Labels:** 42cc sf-2 sf-current
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Fri Dec 11, 2015 04:56 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Dave Brondsema <da...@brondsema.net>.
- **labels**: 42cc, sf-2, sf-current --> 42cc, sf-2
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** closed
**Milestone:** unreleased
**Labels:** 42cc sf-2
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Tue Jan 05, 2016 11:14 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Dave Brondsema <da...@brondsema.net>.
- **status**: review --> closed
- **Reviewer**: Dave Brondsema
- **Comment**:
I think this is a good enough improvement for now to merge it. Cache busting the thumbnails can be separate.
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** closed
**Milestone:** unreleased
**Labels:** 42cc sf-2 sf-current
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Fri Dec 25, 2015 09:50 AM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Igor Bondarenko <je...@gmail.com>.
- **labels**: --> 42cc
- **status**: open --> in-progress
- **assigned_to**: Igor Bondarenko
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** in-progress
**Milestone:** unreleased
**Labels:** 42cc
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Wed Oct 14, 2015 04:44 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Igor Bondarenko <je...@gmail.com>.
- **status**: in-progress --> review
- **Comment**:
Closed #877.
`ib/8003`
Note that if you're replacing image attachment thumbnail will not change rightaway. It will change only if you reload a page manually. That's because of browser caching. At least in Chrome, after adding the attachment and redirecting back to the, let's say, tickets page, browser will not issue another request for thumbnail, but will use one in cache. To avoid that we'll need to change thumb/attach url whenever image is changed (e.g. append `?<last-changed-date>` to the url). We're setting `ETag`, but it's not sufficient because browser doesn't make request to check if image changed in some cases as I described above.
I didn't change url right now, since there's quite a few places that need to be changed. E.g. thumb url always constructed manually from `attach.url()` in templates, so changing `attach.url()` to append `?<last-changed-date>` will break those and probably the right way will be to add separate method to `Attachment` class for `thumb_url` and use that everywhere.
Anyway, wanted to consult you guys before doing that. Check out the new code and let me know what do you think. Thanks
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** review
**Milestone:** unreleased
**Labels:** 42cc sf-2 sf-current
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Thu Dec 24, 2015 03:55 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] Re: #8003 Bugs in attachments to comments
Posted by Dave Brondsema <da...@brondsema.net>.
I think replacing it would be good. It's easier, and its often the expected behavior, for example if you are attaching some script or a screenshot, then upload with a new name you probably mean to update it and don't need anyone to access the previous version.
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** in-progress
**Milestone:** unreleased
**Labels:** 42cc
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Fri Dec 11, 2015 04:56 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Igor Bondarenko <je...@gmail.com>.
Seems like somethig changed since this ticket was created. Now you can delete attachments for comments just fine.
We need to decide regarding second point though. What behavior do we want?
In fact, I just tested and top-level artifact editing also duplicates a file but not replaces it (tested on tickets). And it does it in a weird way. The file stays the same as the **first** uploaded version, but you have `N` links to it (which are the same) in the UI.
So, should we make all attachments replace each other if name matches, or should we fix duplication to generate different links and save all version of the files?
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** in-progress
**Milestone:** unreleased
**Labels:** 42cc
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Wed Dec 09, 2015 02:14 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
[allura:tickets] #8003 Bugs in attachments to comments
Posted by Igor Bondarenko <je...@gmail.com>.
Regarding attachments removal, there is actually a bug. You can't delete attachments to other user's comment even if you're a moderator and can edit the comment itself and add/rewrite new attachments. Fix is underway.
---
** [tickets:#8003] Bugs in attachments to comments**
**Status:** in-progress
**Milestone:** unreleased
**Labels:** 42cc sf-2 sf-current
**Created:** Wed Oct 14, 2015 04:44 PM UTC by Dave Brondsema
**Last Updated:** Mon Dec 14, 2015 05:16 PM UTC
**Owner:** Igor Bondarenko
Editing a comment to modify its attachments has some bugs, compared to what you can do for top-level artifact attachments.
* no way to delete an attachment
* uploading with the same name duplicates the file, rather than replace it (see [#4350] which could be considered a feature)
---
Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.