[
https://issues.apache.org/jira/browse/LANG-362?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Niall Pemberton reopened LANG-362:
----------------------------------
OK I have a couple of points
- Custom sub-formats are not supported - so if I embed a "custom" format in a
choice format it throws an IllegalArgumentException. Do we want to support that
scenario or document clearly that it doesn't work?
(I did make this point in
https://issues.apache.org/jira/browse/LANG-362?focusedCommentId=12564290#action_12564290)
- Now the setFormats(), setFormat(), and setFormatByArgumentIndex() all throw
UnsupportedOperationException I don't see the need for the logic to re-create
the pattern - much simpler to just cache the pattern the EMF was created with
and return that value in toPattern() - or am I missing something?
- the test cases could (and IMO should) be simpler - currently there are four
classes to test EMF (AbstractMessageFormatTest,
ExtendedMessageFormatBaselineTest, MessageFormatExtensionTest. and
MessageFormatTest) and I don't see why we don't just have one
ExtendedMessageFormatTest). Tests are often a good way to look at how something
works - so the simpler the better both for those maintaining it and users
wanting to understand how to use it.
> Add ExtendedMessageFormat to org.apache.commons.lang.text
> ---------------------------------------------------------
>
> Key: LANG-362
> URL: https://issues.apache.org/jira/browse/LANG-362
> Project: Commons Lang
> Issue Type: New Feature
> Reporter: Matt Benson
> Assignee: Matt Benson
> Priority: Minor
> Fix For: 2.4
>
> Attachments: DateFormatFactory.java, extendedMessageFormat.patch.txt,
> extendedMessageFormat.patch.txt, ExtendedMessageFormat2.java,
> FormatFactory.java
>
>
> Discussed on dev@ (
> http://mail-archives.apache.org/mod_mbox/commons-dev/200710.mbox/[EMAIL
> PROTECTED] ); adding here for tracking purposes and in case anyone has any
> serious objections to my implementation. Patch forthcoming...
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
