Defect #37626
closed
Diff of a javascript file in repository module is not displayed with layout
0%
Description
At repository, when viewing differences of a javascript file, the revision page is showed with no styling as the image below
By the log content, no errors occurred. But I think request format JS is a mistake.
About my application's environment Redmine 5 Ruby version ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux] RubyGems version 3.3.7 Rails version 6.1.6 Active Record version 6.1.6 Action Pack version 6.1.6 Action Mailer version 6.1.6 Active Support version 6.1.6 Application root /opt/redmine Environment production Database adapter mysql Database schema version 5.5.41
Files
Related issues
Updated by Trang Tran Thi Quynh about 2 years ago
This bug was reported at https://www.redmine.org/issues/32449 and fixed. But it occurred again after fixed for Rails 6.1 Rendering actions with '.' in the name is deprecated #34984.
/app/controllers/repositories_controller.rb - render :diff, :formats => :html, :layout => 'base.html.erb' + render :diff, :formats => :html, :layout => 'base'
Updated by Go MAEDA about 2 years ago
- Category changed from Rails support to SCM
- Status changed from New to Confirmed
Updated by Go MAEDA about 2 years ago
- Related to Defect #32449: Diff view for .js files in repositories is broken added
Updated by Go MAEDA about 2 years ago
- Related to Defect #34984: Rails 6.1 Rendering actions with '.' in the name is deprecated added
Updated by Mizuki ISHIKAWA about 2 years ago
Rewriting to base.html.erb as before should solve the problem.
When displaying a diff of the js file, it appears to look for base.js if I don't explicitly write it.
diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb
index ff1230c883..56be9d526a 100644
--- a/app/controllers/repositories_controller.rb
+++ b/app/controllers/repositories_controller.rb
@@ -286,7 +286,7 @@ class RepositoriesController < ApplicationController
@changeset = @repository.find_changeset_by_name(@rev)
@changeset_to = @rev_to ? @repository.find_changeset_by_name(@rev_to) : nil
@diff_format_revisions = @repository.diff_format_revisions(@changeset, @changeset_to)
- render :diff, :formats => :html, :layout => 'base'
+ render :diff, :formats => :html, :layout => 'base.html.erb'
end
end
Updated by Go MAEDA about 2 years ago
Mizuki ISHIKAWA wrote:
Rewriting to base.html.erb as before should solve the problem.
When displaying a diff of the js file, it appears to look for base.js if I don't explicitly write it.[...]
The patch fixes the issue but causes deprecation warnings.
DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: layouts/base.html.erb (called from block in find_all at /Users/maeda/redmines/redmine-trunk/config/initializers/10-patches.rb:60) Rendering layout layouts/base.html.erb Rendering repositories/diff.html.erb within layouts/base.html.erb
Updated by Go MAEDA about 2 years ago
- Target version set to Candidate for next minor release
Go MAEDA wrote:
The patch fixes the issue but causes deprecation warnings.
But I think we should prioritize fixing the problem when viewing JavaScript files than fixing deprecation warnings.
Updated by Marius BĂLTEANU about 2 years ago
- Target version changed from Candidate for next minor release to 5.0.3
Updated by Marius BĂLTEANU about 2 years ago
- Subject changed from At repository, viewing differences of a javascript file is lost style to Diff of a javascript file in repository module is not displayed with layout
- Assignee set to Marius BĂLTEANU
Updated by Marius BĂLTEANU about 2 years ago
- Status changed from Confirmed to Resolved
- Resolution set to Fixed
I tried to add a functional test for this case, but without success, I think because the request made by minitest is different by the one made by the browser.
In the meantime, I've merged the proposed fix to 5.0-stable and I've opened a new issue (#37732) to properly fix this issue.
Updated by Marius BĂLTEANU about 2 years ago
- Status changed from Resolved to Closed
Updated by Go MAEDA about 2 years ago
- Related to Defect #37732: Fix "DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated" in RepositoriesController added
Updated by Marius BĂLTEANU 12 months ago
- Copied to Defect #39747: Diff of a javascript file in repository module is not displayed with layout added