Actions
Patch #32922
closed
Reload detached attachments
Description
use #reload instead, after upgrading to Rails 5
Files
Related issues
Updated by Marius BĂLTEANU over 5 years 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.
Updated by Marius BĂLTEANU over 5 years ago
- Related to Feature #29914: Migrate to Rails 6.1 with Zeitwerk autoloading added
Updated by Pavel Rosický over 5 years ago
- Status changed from New to Resolved
sry, thanks for catching this, I was confused about the comment. Closing
Updated by Marius BĂLTEANU over 5 years ago
- Status changed from Resolved to New
Let's keep it open, maybe we can fix the issue.
Updated by Marius BĂLTEANU over 4 years 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.
Updated by Go MAEDA over 4 years ago
- Category set to Code cleanup/refactoring
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch. Thank you.
Actions