Merge pull request #45 from yahoo/selectordinal

Properly pretty-print `selectordinal` arguments

Author: ericf

src/print-icu-message.js

@@ -30,7 +30,7 @@ function printICUMessage(ast) {
             return `{${node.id}}`;
         }
 
-        switch (getArgumentType(node.format.type)) {
+        switch (getArgumentType(node.format)) {
         case 'number':
         case 'date':
         case 'time':
@@ -46,23 +46,30 @@ function printICUMessage(ast) {
     return printedNodes.join('');
 }
 
-function getArgumentType(astType) {
-    return astType.replace(/Format$/, '').toLowerCase();
+function getArgumentType(format) {
+    const {type, ordinal} = format;
+
+    // Special-case ordinal plurals to use `selectordinal` instead of `plural`.
+    if (type === 'pluralFormat' && ordinal) {
+        return 'selectordinal';
+    }
+
+    return type.replace(/Format$/, '').toLowerCase();
 }
 
 function printMessageTextASTNode({value}) {
     return value.replace(ESAPE_CHARS_REGEXP, (char) => ESCAPED_CHARS[char]);
 }
 
 function printSimpleFormatASTNode({id, format}) {
-    let argumentType = getArgumentType(format.type);
+    let argumentType = getArgumentType(format);
     let style = format.style ? `, ${format.style}` : '';
 
     return `{${id}, ${argumentType}${style}}`;
 }
 
 function printOptionalFormatASTNode({id, format}) {
-    let argumentType = getArgumentType(format.type);
+    let argumentType = getArgumentType(format);
     let offset = format.offset ? `, offset:${format.offset}` : '';
 
     let options = format.options.map((option) => {