IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-25 10:20:58 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: Correct user profile URLs in /about crawler view

Browse Source
This commit is contained in:
David Taylor 2020-07-14 16:09:27 +01:00
parent 9917b9cb21
commit cc6d722de1
No known key found for this signature in database
GPG Key ID: 46904C18B1D3F434
2 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/views/about/index.html.erb
View File

@ -14,8 +14,8 @@
<div class='admins-list' itemscope itemtype='http://schema.org/ItemList'>
<% @about.admins.each do |user| %>
<div itemprop='itemListElement' itemscope itemtype='http://schema.org/ListItem'>
<meta itemprop='url' content='<%= user_path(user) %>'>
<a href='<%= user_path(user) %>' itemprop='item'>
<meta itemprop='url' content='<%= user_path(user.username_lower) %>'>
<a href='<%= user_path(user.username_lower) %>' itemprop='item'>
<span itemprop='image'>
<img width="45" height="45" class="avatar" src="<%= user.small_avatar_url %>">
</span>
@ -35,8 +35,8 @@
<div class='moderators-list' itemscope itemtype='http://schema.org/ItemList'>
<% @about.moderators.each do |user| %>
<div itemprop='itemListElement' itemscope itemtype='http://schema.org/ListItem'>
<meta itemprop='url' content='<%= user_path(user) %>'>
<a href='<%= user_path(user) %>' itemprop='item'>
<meta itemprop='url' content='<%= user_path(user.username_lower) %>'>
<a href='<%= user_path(user.username_lower) %>' itemprop='item'>
<span itemprop='image'>
<img width="45" height="45" class="avatar" src="<%= user.small_avatar_url %>">
</span>

7
spec/requests/about_controller_spec.rb
View File

@ -35,6 +35,13 @@ describe AboutController do
expect(response.status).to eq(200)
expect(response.body).to include("<title>About - Discourse</title>")
end
it "should include correct user URLs" do
Fabricate(:admin, username: "anAdminUser")
get '/about', headers: { 'HTTP_USER_AGENT' => 'Googlebot' }
expect(response.status).to eq(200)
expect(response.body).to include("/u/anadminuser")
end
end
it "serializes stats when 'Guardian#can_see_about_stats?' is true" do