fix(open_graph): territory must be in uppercase

curbengh · Oct 28, 2019 · 28949da · 28949da
1 parent 7704dcb
commit 28949da
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/lib/plugins/helper/open_graph.js b/lib/plugins/helper/open_graph.js
@@ -84,9 +84,13 @@ function openGraphHelper(options = {}) {
   }
 
   if (language) {
-    language = language.replace('-', '_');
+    if (language.length === 5) {
+      const territory = language.slice(-2);
 
-    if (language.length === 5) result += og('og:locale', language);
+      language = language.replace('-', '_').replace(territory, territory.toUpperCase());
+
+      result += og('og:locale', language);
+    }
   }
 
   images = images.map(path => {

diff --git a/test/scripts/helpers/open_graph.js b/test/scripts/helpers/open_graph.js
@@ -613,7 +613,7 @@ describe('open_graph', () => {
       is_post: isPost
     }, {language: 'es-cr'});
 
-    result.should.contain(meta({property: 'og:locale', content: 'es_cr'}));
+    result.should.contain(meta({property: 'og:locale', content: 'es_CR'}));
   });
 
   it('og:locale - page.lang', () => {
@@ -623,7 +623,7 @@ describe('open_graph', () => {
       is_post: isPost
     });
 
-    result.should.contain(meta({property: 'og:locale', content: 'es_mx'}));
+    result.should.contain(meta({property: 'og:locale', content: 'es_MX'}));
   });
 
   it('og:locale - page.language', () => {
@@ -633,7 +633,7 @@ describe('open_graph', () => {
       is_post: isPost
     });
 
-    result.should.contain(meta({property: 'og:locale', content: 'es_gt'}));
+    result.should.contain(meta({property: 'og:locale', content: 'es_GT'}));
   });
 
   it('og:locale - config.language', () => {
@@ -645,7 +645,7 @@ describe('open_graph', () => {
       is_post: isPost
     });
 
-    result.should.contain(meta({property: 'og:locale', content: 'es_pa'}));
+    result.should.contain(meta({property: 'og:locale', content: 'es_PA'}));
   });
 
   it('og:locale - no language set', () => {