Project

General

Profile

Bug #21637

wrong <title> on http://sat630.example.com/apidoc/v2.<some langs>.html - contains html tag

Added by Martin Bacovsky almost 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Low
Category:
API
Target version:
Difficulty:
Triaged:
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1510783

Description of problem:
There is wrong <title> on http://sat630.example.com/apidoc/v2.&lt;lang&gt;.html - contains html tag

Version-Release number of selected component (if applicable):
satellite-6.3.0-21.0.beta.el7sat.noarch (snap 24)

How reproducible:
always

Steps to Reproduce:
1. Go to http://sat630.example.com/apidoc/ and check page title ("<html><head><title>..." tag content).
2. Now click at "v2" in the header and check title again:
http://sat630.example.com/apidoc/v2.html
3. Now click at "en_GB" and check title again:
http://sat630.example.com/apidoc/v2.en_GB.html
or:
http://sat630.example.com/apidoc/v2.fr.html

Actual results:
In step 1. and 2., title is "<title>API documentation</title>"
In step 3., title is "<title><span class="translation_missing" title="translation missing: en-GB.apipie.api_documentation">Api Documentation</span></title>" which is incorrect - looks broken in browsers heading

Expected results:
There should be no HTML tags

Associated revisions

Revision 8d54948a (diff)
Added by Ivan Necas almost 3 years ago

Fixes #21637 - fallback to default locale with I18n

This makes the I18n behavior closer to the gettext one, and prevents
some ugly 'missing_translation' messages in the UI (for example in
apipie, that uses the I18n backend for it's strings).

History

#1 Updated by Martin Bacovsky almost 3 years ago

  • Target version set to 233
  • Priority changed from Normal to Low
  • Assignee set to Martin Bacovsky
  • Status changed from New to Assigned
  • Category changed from 19 to API

#2 Updated by Martin Bacovsky almost 3 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Apipie/apipie-rails/pull/571 added

#3 Updated by The Foreman Bot almost 3 years ago

  • Pull request https://github.com/theforeman/foreman/pull/5034 added

#4 Updated by Daniel Lobato Garcia almost 3 years ago

  • Legacy Backlogs Release (now unused) set to 296

#5 Updated by Ivan Necas almost 3 years ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF