remove LocalizeMessageID until extract can extract it

Author: nicksnyder

v2/i18n/localizer.go

@@ -112,12 +112,13 @@ func (l *Localizer) LocalizeMessage(msg *Message) (string, error) {
 	})
 }
 
+// TODO: uncomment this (and the test) when extract has been updated to extract these call sites too.
 // Localize returns a localized message.
-func (l *Localizer) LocalizeMessageID(messageID string) (string, error) {
-	return l.Localize(&LocalizeConfig{
-		MessageID: messageID,
-	})
-}
+// func (l *Localizer) LocalizeMessageID(messageID string) (string, error) {
+// 	return l.Localize(&LocalizeConfig{
+// 		MessageID: messageID,
+// 	})
+// }
 
 // LocalizeWithTag returns a localized message and the language tag.
 func (l *Localizer) LocalizeWithTag(lc *LocalizeConfig) (string, language.Tag, error) {
@@ -158,6 +159,15 @@ func (l *Localizer) LocalizeWithTag(lc *LocalizeConfig) (string, language.Tag, e
 	return msg, tag, nil
 }
 
+			MessageID:    messageID,
+			PluralCount:  kv["PluralCount"],
+			TemplateData: args,
+		}
+
+		return l.Localize(lc)
+	}
+}
+
 func (l *Localizer) getTemplate(id string, defaultMessage *Message) (language.Tag, *MessageTemplate) {
 	// Fast path.
 	// Optimistically assume this message id is defined in each language.

v2/i18n/localizer_test.go

@@ -567,9 +567,9 @@ func TestLocalizer_Localize(t *testing.T) {
 				check(localizer.LocalizeMessage(test.conf.DefaultMessage))
 			}
 
-			if test.conf.MessageID != "" && reflect.DeepEqual(test.conf, &LocalizeConfig{MessageID: test.conf.MessageID}) {
-				check(localizer.LocalizeMessageID(test.conf.MessageID))
-			}
+			// if test.conf.MessageID != "" && reflect.DeepEqual(test.conf, &LocalizeConfig{MessageID: test.conf.MessageID}) {
+			// 	check(localizer.LocalizeMessageID(test.conf.MessageID))
+			// }
 		})
 	}
 }