Re: svn commit: r867 - trunk: entity/src/org/subethamail/entity/i frontend/content

Actions

From Jon Scott Stevens Nov 9, 2006 12:04 PM

The UI shows needed.toString(), not needed.getPretty(). That's what  
I'm talking about. It's ugly for users to see VIEW_ARCHIVES vs. "View  
Archives".

It's not a hack, it's quite important as a lot of people see that  
page as the first thing they see when they click a link in an email  
(especially a link related to 'leave attachments on server). We need  
to make that page as self explanatory as possible.

jon

On Nov 9, 2006, at 11:30 AM, Jeff Schnitzer wrote:

> The UI does show a pretty version of the message, look in
> error_permission.jsp.  No point in adding it to the exception message.
> The message is printed on that page too just as a quick hack to show
> which permission is the problem.
>
> Jeff
>
> Jon Scott Stevens wrote:
>> On Nov 9, 2006, at 1:16 AM, lhoriman@tigris.org wrote:
>>
>>
>>> -		super("Requires permission '" + needed.getPretty() + "'. " +
>>> +		super("Requires permission " + needed);
>>>
>>
>> I think we should at least have the pretty version of the message.
>> What is the point of showing the user an internal enum value?
>>
>> jon

Thread History