github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-05-26 09:02:05 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Add helpers for title and content for application.html.erb and crawler.html.erb (#32290)

Browse Source 
Previously, there was only those helpers in `crawler.html.erb` as we
added it in https://github.com/discourse/discourse/pull/32259

However, to keep it consistent, we need to add it in
`application.html.erb` as well.
This commit is contained in:
Gabriel Grubba
2025-04-14 16:34:23 -03:00
committed by GitHub
parent dcef061651
commit 12f05181c7
4 changed files with 14 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/helpers/application_helper.rb
View File

@ -270,7 +270,7 @@ module ApplicationHelper
(request ? I18n.locale.to_s : SiteSetting.default_locale).sub("_", "-")
end
def crawlable_title_content
def title_content
DiscoursePluginRegistry.apply_modifier(
:meta_data_content,
content_for(:title) || SiteSetting.title,
@ -279,7 +279,7 @@ module ApplicationHelper
)
end
def crawlable_description_content
def description_content
DiscoursePluginRegistry.apply_modifier(
:meta_data_content,
@description_meta || SiteSetting.site_description,

4
app/views/layouts/application.html.erb
View File

@ -2,8 +2,8 @@
<html lang="<%= html_lang %>" class="<%= html_classes %>">
<head>
<meta charset="utf-8">
<title><%= content_for?(:title) ? yield(:title) : SiteSetting.title %></title>
<meta name="description" content="<%= @description_meta || SiteSetting.site_description %>">
<title><%= title_content %></title>
<meta name="description" content="<%= description_content %>">
<meta name="discourse_theme_id" content="<%= theme_id %>">
<meta name="discourse_current_homepage" content="<%= current_homepage %>">

4
app/views/layouts/crawler.html.erb
View File

@ -2,8 +2,8 @@
<html lang="<%= html_lang %>">
<head>
<meta charset="utf-8">
<title><%= crawlable_title_content %></title>
<meta name="description" content="<%= crawlable_description_content %>">
<title><%= title_content %></title>
<meta name="description" content="<%= description_content %>">
<%= render partial: "layouts/head" %>
<%= render partial: "common/discourse_stylesheet" %>
<%= theme_lookup("head_tag") %>

16
spec/helpers/application_helper_spec.rb
View File

@ -745,10 +745,10 @@ RSpec.describe ApplicationHelper do
end
end
describe "#crawlable_title_content" do
describe "#title_content" do
it "returns the correct title" do
SiteSetting.title = "Test Title"
result = helper.crawlable_title_content
result = helper.title_content
expect(result).to include("Test Title")
end
@ -757,16 +757,16 @@ RSpec.describe ApplicationHelper do
helper.stubs(:content_for?).with(:title).returns(true)
helper.stubs(:content_for).with(:title).returns("Custom Title")
result = helper.crawlable_title_content
result = helper.title_content
expect(result).to include("Custom Title")
end
end
describe "#crawlable_description_content" do
describe "#description_content" do
it "returns the correct description" do
SiteSetting.site_description = "Test Description"
result = helper.crawlable_description_content
result = helper.description_content
expect(result).to include("Test Description")
end
@ -774,7 +774,7 @@ RSpec.describe ApplicationHelper do
it "accepts a content argument" do
@description_meta = "Custom Description"
result = helper.crawlable_description_content
result = helper.description_content
expect(result).to include("Custom Description")
end
@ -810,7 +810,7 @@ RSpec.describe ApplicationHelper do
it "modifies the title tag" do
plugin.register_modifier(:meta_data_content, &block)
title = helper.crawlable_title_content
title = helper.title_content
expect(title).to include("BIG TITLE")
end
@ -818,7 +818,7 @@ RSpec.describe ApplicationHelper do
it "modifies the description tag" do
plugin.register_modifier(:meta_data_content, &block)
description = helper.crawlable_description_content
description = helper.description_content
expect(description).to include(" - modified by plugin")
end