Patch #32922
Reload detached attachments
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | % Done: | 0% | ||
Category: | Code cleanup/refactoring | |||
Target version: | 5.0.0 |
Description
use #reload instead, after upgrading to Rails 5
Related issues
Associated revisions
Reload detached attachments (#32922).
Patch by Pavel Rosický.
History
#1
Updated by Marius BALTEANU about 1 year ago
Thanks Pavel for posting the patches for Rails 6 update.
I've applied this patch on the current trunk and a test fails:
Failure: AttachmentsTest#test_upload_and_resubmit_after_validation_failure [/builds/redmine-org/redmine/test/integration/attachments_test.rb:113]: Expected at least 1 element matching "input[name="attachments[p0][description]"][value="My uploaded file"]", found 0.. Expected 0 to be >= 1. bin/rails test test/integration/attachments_test.rb:99
You can see the test results here.
#2
Updated by Marius BALTEANU about 1 year ago
- Related to Feature #29914: Migrate to Rails 6.1 added
#3
Updated by Pavel Rosický about 1 year ago
- Status changed from New to Resolved
sry, thanks for catching this, I was confused about the comment. Closing
#4
Updated by Marius BALTEANU about 1 year ago
- Status changed from Resolved to New
Let's keep it open, maybe we can fix the issue.
#5
Updated by Marius BALTEANU 14 days ago
- Subject changed from reload detached attachments to Reload detached attachments
This change passes all the tests now: https://gitlab.com/redmine-org/redmine/-/pipelines/280930686
The fail reported by me above was caused by another issue that was fixing as part of Rails 6.1 update.
#6
Updated by Marius BALTEANU 14 days ago
- Target version set to 5.0.0