[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wink-dev
Subject:    [jira] Updated: (WINK-297) serviceability improvement; externalize
From:       "Mike Rheinheimer (JIRA)" <jira () apache ! org>
Date:       2010-06-30 19:51:50
Message-ID: 2657280.140361277927510348.JavaMail.jira () thor
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/WINK-297?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Mike Rheinheimer updated WINK-297:
----------------------------------

    Attachment: WINK-297.patch

> serviceability improvement; externalize strings, confirm formatting
> -------------------------------------------------------------------
> 
> Key: WINK-297
> URL: https://issues.apache.org/jira/browse/WINK-297
> Project: Wink
> Issue Type: Improvement
> Components: Client, Common, Server
> Affects Versions: 1.1.2
> Reporter: Mike Rheinheimer
> Assignee: Mike Rheinheimer
> Fix For: 1.1.2
> 
> Attachments: WINK-297.patch
> 
> 
> We've needed serviceability improvements for a while.  Here's the first pass.  I've \
> externalized all strings that should be externalized, marked the rest with \
> //$NON-NLS-#$ via Eclipse's tool, and here's the cool part:  wrote a unittest to do \
> the following: 1)  checks that the strings that should be externalized are \
> externalized (only debug messages do not need to be externalized) 2)  checks that \
> all keys referred to by Messages.getMessage actually exist 3)  checks that there \
> are no unused keys in the resource.properties file 4)  checks that the number of \
> params matches up with the number of braces {} in a formatted log string, both \
> externalized and non-externalized See attached patch.  The test is under \
> org.apache.wink.common.internal.i18n.MessagesTest. I'll wait to commit this until \
> the 1.1.1 release vote and process is done.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic