Re: empty wicket:message

2007-11-05 Thread Sebastiaan van Erk
BTW, I just wanted to say how much I love Wicket! (Which you can tell by the minor issues that I'm mailing about on this list, which underlines the fact that I can't find any major ones...) :-) The resulting code is so short and so clean, Wicket it is so easy to work with, and everything

Re: empty wicket:message

2007-11-05 Thread Johan Compagner
On 11/5/07, Sebastiaan van Erk [EMAIL PROTECTED] wrote: BTW, I just wanted to say how much I love Wicket! As long as you don't give all your Love to Wicket, you have to keep some for Morena! (and her beautiful wicket site http://www.denherdervarga.com/ ) johan

Re: empty wicket:message

2007-11-05 Thread Sebastiaan van Erk
Johan Compagner wrote: On 11/5/07, Sebastiaan van Erk [EMAIL PROTECTED] wrote: BTW, I just wanted to say how much I love Wicket! As long as you don't give all your Love to Wicket, you have to keep some for Morena! (and her beautiful wicket site http://www.denherdervarga.com/ ) Don't worry

Re: empty wicket:message

2007-11-04 Thread Johan Compagner
thats better. I think that throw exceptions on missing resources shouldn't happen in deployment mode anyway i think there is no much use for it a good log statement in the server log should be enough. i dont find it very dangerous that it shows the body IF it has a body because that you see it on

Re: empty wicket:message

2007-11-04 Thread Sebastiaan van Erk
Johan Compagner wrote: thats better. I think that throw exceptions on missing resources shouldn't happen in deployment mode anyway i think there is no much use for it a good log statement in the server log should be enough. i dont find it very dangerous that it shows the body IF it has a body

empty wicket:message

2007-11-03 Thread Sebastiaan van Erk
Hi, I was wondering if it is possible to configure wicket to make wicket:message output the key in braces when the key is not found (at least in development mode), because that would make it a lot easier to spot missing labels... That is, what I'd like to do is: wicket:message

Re: empty wicket:message

2007-11-03 Thread Igor Vaynberg
im afraid we cannot do that. because the contract is that when a resource is not found we output the body, which is supposed to be the default text. to do what you want we need to change that contract. i am not opposed to that per se, but there needs to be a discussion followed by a vote. -igor

Re: empty wicket:message

2007-11-03 Thread Al Maw
Sebastiaan van Erk wrote: I was wondering if it is possible to configure wicket to make wicket:message output the key in braces when the key is not found (at least in development mode), because that would make it a lot easier to spot missing labels... That is, what I'd like to do is:

Re: empty wicket:message

2007-11-03 Thread Sebastiaan van Erk
Thanks for the reply, I was afraid I was going to get this answer. :-) I can understand it though. Igor Vaynberg wrote: im afraid we cannot do that. because the contract is that when a resource is not found we output the body, which is supposed to be the default text. Am I pushing my luck

Re: empty wicket:message

2007-11-03 Thread Eelco Hillenius
Mmmm. I must say I agree with this. I'd actually prefer it to throw an exception. ;-) I'm surprised we don't do this already! I would have expected that if IResourceSettings#getThrowExceptionOnMissingResource returns true, an exception would be thrown here as well. I think we should fix it.

Re: empty wicket:message

2007-11-03 Thread Igor Vaynberg
then you are violating the contract of wicket:message. if we do this then there is little point to allowing body inside wicket:message -igor On 11/3/07, Eelco Hillenius [EMAIL PROTECTED] wrote: Mmmm. I must say I agree with this. I'd actually prefer it to throw an exception. ;-) I'm

Re: empty wicket:message

2007-11-03 Thread Eelco Hillenius
Btw, I would mind a decision about this in this weekend, as I'm finishing the chapter on localization right now :-) FWIW, I don't like the silent failure we have now, and believe that using the body as a default isn't very helpful; e.g. if you have the default body the same as what you intended

Re: empty wicket:message

2007-11-03 Thread Sebastiaan van Erk
Eelco Hillenius wrote: Btw, I would mind a decision about this in this weekend, as I'm finishing the chapter on localization right now :-) FWIW, I don't like the silent failure we have now, and believe that using the body as a default isn't very helpful; e.g. if you have the default body the

Re: empty wicket:message

2007-11-03 Thread Eelco Hillenius
The easiest solution would be this: Index: /Users/eelcohillenius/Documents/workspace_wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java === ---

Re: empty wicket:message

2007-11-03 Thread Johan Compagner
Yes we could do that. Just remove that default value. But what i would like to have IF you have a body specified then we don't throw anything This way we keep old behavior and we have the new one Because if i specifiy a body then thats the default value for me. And if not then i still can see very

Re: empty wicket:message

2007-11-03 Thread Eelco Hillenius
On 11/3/07, Johan Compagner [EMAIL PROTECTED] wrote: Yes we could do that. Just remove that default value. But what i would like to have IF you have a body specified then we don't throw anything This way we keep old behavior and we have the new one Because if i specifiy a body then thats the

Re: empty wicket:message

2007-11-03 Thread Al Maw
Johan Compagner wrote: Yes we could do that. Just remove that default value. But what i would like to have IF you have a body specified then we don't throw anything This way we keep old behavior and we have the new one FWIW, I entirely agree with this. If we just change it for tags that have

Re: empty wicket:message

2007-11-03 Thread Eelco Hillenius
On 11/3/07, Al Maw [EMAIL PROTECTED] wrote: Johan Compagner wrote: Yes we could do that. Just remove that default value. But what i would like to have IF you have a body specified then we don't throw anything This way we keep old behavior and we have the new one FWIW, I entirely agree