Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, deshani, dill0wn, and 10 others

#5666 Attachments break when tickets are moved

Milestone: v1.0.0

Status: closed

Owner: nobody

Labels: 42cc (432)

Component: General

Reviewer: Dave Brondsema

Updated: 2015-08-20

Created: 2013-01-24

Creator: Dave Brondsema

Private: No

Example https://sourceforge.net/p/test-brondsem/features/1/ was moved and the attachment (as part of a comment) doesn't work.

Check attachments at the top-level (ticket) and for attachements.

Related

Tickets: ~~#5666~~
Tickets: ~~#5730~~

Discussion

Igor Bondarenko - 2013-01-25

Created #261: [#5666] Move attachments along with the ticket (2cp)

Related

Tickets: ~~#5666~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-01-25

status: open --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-01

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-01

Closed #261. allura:je/42cc_5666

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-01

qa: Dave Brondsema
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-01

status: code-review --> closed

milestone: forge-backlog --> forge-feb-08
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-01

Everything worked fine on this branch, but when I rebased it onto the latest master then forgetracker.tests.functional.test_root:TestFunctionalController.test_move_attachment started to fail. There were no significant conflicts (just some adjacent tests) so I'm not sure what happened.

I also think [#5730] will affect the logic. I can re-review this ticket and 5730 at the same time, if you can provide a branch that has all the changes merged together

Related

Tickets: ~~#5730~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-01

status: closed --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2013-02-04

Ok, we'll fix this after [#5730]

Related

Tickets: ~~#5730~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2013-02-05

status: in-progress --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.