Skip to content

Commit

Permalink
fix(locales): fix incompeteMessageFallback to be a string rather than…
Browse files Browse the repository at this point in the history
… an object (#1853)

* fix(aria-allowed-role): allow role combobox on input tel, search, url, and email

* fix(locale structure): structure message correct when building translations

* fix(possible merge conflict): revert the code to negate any merge conflicts

* fix(possible merge conflict): revert the code to negate any merge conflicts

* fix: ci failing due to prior commits

* fix: reverting files due to prior commit
  • Loading branch information
michael-siek authored and straker committed Oct 18, 2019
1 parent 69340d8 commit 88677a9
Show file tree
Hide file tree
Showing 9 changed files with 12 additions and 39 deletions.
7 changes: 4 additions & 3 deletions build/tasks/add-locale.js
Expand Up @@ -62,9 +62,10 @@ module.exports = function(grunt) {
return out;
}, {}),
incompleteFallbackMessage: result.misc.reduce(function(out, misc) {
out[misc.incompleteFallbackMessage] = misc.metadata;
return out;
}, {})
return misc.incompleteFallbackMessage
? misc.incompleteFallbackMessage
: out;
}, '')
};

// update locale file if exists
Expand Down
2 changes: 1 addition & 1 deletion lib/core/base/audit.js
Expand Up @@ -291,7 +291,7 @@ Audit.prototype.applyLocale = function(locale) {
if (locale.incompleteFallbackMessage) {
this.data.incompleteFallbackMessage = mergeFallbackMessage(
this.data.incompleteFallbackMessage,
locale.incompleteFallbackMessage.undefined.failureMessage
locale.incompleteFallbackMessage
);
}

Expand Down
6 changes: 1 addition & 5 deletions locales/de.json
Expand Up @@ -619,9 +619,5 @@
"failureMessage": "Korrigiere alle der folgenden Punkte:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": ""
}
}
"incompleteFallbackMessage": ""
}
6 changes: 1 addition & 5 deletions locales/es.json
Expand Up @@ -769,9 +769,5 @@
"failureMessage": "Corregir (todas) las siguientes incidencias:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": "Corregir (todas) las siguientes incidencias:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
}
"incompleteFallbackMessage": "Corregir (todas) las siguientes incidencias:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
6 changes: 1 addition & 5 deletions locales/fr.json
Expand Up @@ -659,9 +659,5 @@
"failureMessage": "Corriger tous les éléments suivants : {{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": "Corriger tous les éléments suivants : {{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
}
"incompleteFallbackMessage": "Corriger tous les éléments suivants : {{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
6 changes: 1 addition & 5 deletions locales/ja.json
Expand Up @@ -778,9 +778,5 @@
"failureMessage": "次のすべてを修正します:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": "次のすべてを修正します:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
}
"incompleteFallbackMessage": "次のすべてを修正します:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
6 changes: 1 addition & 5 deletions locales/nl.json
Expand Up @@ -32,9 +32,5 @@
"failureMessage": "Gebruik een van de volgende oplossingen:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": "axe kon de reden niet vertellen. Tijd om de element inspecteur uit te breken!"
}
}
"incompleteFallbackMessage": "axe kon de reden niet vertellen. Tijd om de element inspecteur uit te breken!"
}
6 changes: 1 addition & 5 deletions locales/pt_BR.json
Expand Up @@ -769,9 +769,5 @@
"failureMessage": "Corrija todos os itens a seguir:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
},
"incompleteFallbackMessage": {
"undefined": {
"failureMessage": "Corrija todos os itens a seguir:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
}
"incompleteFallbackMessage": "Corrija todos os itens a seguir:{{~it:value}}\n {{=value.split('\\n').join('\\n ')}}{{~}}"
}
6 changes: 1 addition & 5 deletions test/core/public/configure.js
Expand Up @@ -393,11 +393,7 @@ describe('axe.configure', function() {
failureMessage: 'bar'
}
},
incompleteFallbackMessage: {
undefined: {
failureMessage: 'baz'
}
}
incompleteFallbackMessage: 'baz'
}
});

Expand Down

0 comments on commit 88677a9

Please sign in to comment.