fix(og:local): only convert territory to uppercase

hexojs · Nov 2, 2019 · 48d2c52 · 48d2c52
1 parent 28949da
commit 48d2c52
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/plugins/helper/open_graph.js b/lib/plugins/helper/open_graph.js
@@ -86,8 +86,9 @@ function openGraphHelper(options = {}) {
   if (language) {
     if (language.length === 5) {
       const territory = language.slice(-2);
+      const territoryRegex = new RegExp(territory.concat('$'), 'i');
 
-      language = language.replace('-', '_').replace(territory, territory.toUpperCase());
+      language = language.replace('-', '_').replace(territoryRegex, territory.toUpperCase());
 
       result += og('og:locale', language);
     }

diff --git a/test/scripts/helpers/open_graph.js b/test/scripts/helpers/open_graph.js
@@ -648,6 +648,18 @@ describe('open_graph', () => {
     result.should.contain(meta({property: 'og:locale', content: 'es_PA'}));
   });
 
+  it('og:locale - convert territory to uppercase', () => {
+    hexo.config.language = 'fr-fr';
+
+    const result = openGraph.call({
+      page: {},
+      config: hexo.config,
+      is_post: isPost
+    });
+
+    result.should.contain(meta({property: 'og:locale', content: 'fr_FR'}));
+  });
+
   it('og:locale - no language set', () => {
     const result = openGraph.call({
       page: {},