Re: Behavior#onTag() doesn't have the Component

2016-11-25 Thread Sven Meier
Hi, the behavior is added to a single component only and readily available anyway in scope: component.add(onTag(tag -> tag.put(key, component.getId())); What do we gain when the same component is given as argument? component.add(onTag(component, tag -> tag.put(key, component.getId(

[GitHub] wicket issue #196: WICKET-6289 fix autolinker so it does not generate an onc...

2016-11-25 Thread martin-g
Github user martin-g commented on the issue: https://github.com/apache/wicket/pull/196 Looks good! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if th

Behavior#onTag() doesn't have the Component

2016-11-25 Thread Martin Grigorov
Hi, At Martijn's slides from ApacheCon [1] I've noticed that critiques he mention is that org.apache.wicket.behavior.Behavior#onTag() uses SerializableConsumer and ignores the Component parameter. I agree that having the component would be handy! Any good reasons why it should be like this ? 1.

[GitHub] wicket issue #199: WICKET-6289 fix autolinker so it does not generate an onc...

2016-11-25 Thread martin-g
Github user martin-g commented on the issue: https://github.com/apache/wicket/pull/199 Is it really needed for 1.5.x ? This branch is for security fixes only. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pr

[GitHub] wicket pull request #198: WICKET-6289 fix autolinker so it does not generate...

2016-11-25 Thread duesenklipper
GitHub user duesenklipper opened a pull request: https://github.com/apache/wicket/pull/198 WICKET-6289 fix autolinker so it does not generate an onclick attribu… …te for img tags You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] wicket pull request #199: WICKET-6289 fix autolinker so it does not generate...

2016-11-25 Thread duesenklipper
GitHub user duesenklipper opened a pull request: https://github.com/apache/wicket/pull/199 WICKET-6289 fix autolinker so it does not generate an onclick attribu… …te for img tags You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] wicket pull request #197: WICKET-6289 fix autolinker so it does not generate...

2016-11-25 Thread duesenklipper
GitHub user duesenklipper opened a pull request: https://github.com/apache/wicket/pull/197 WICKET-6289 fix autolinker so it does not generate an onclick attribu… …te for img tags You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] wicket pull request #196: WICKET-6289 fix autolinker so it does not generate...

2016-11-25 Thread duesenklipper
GitHub user duesenklipper opened a pull request: https://github.com/apache/wicket/pull/196 WICKET-6289 fix autolinker so it does not generate an onclick attribu… …te for img tags You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] wicket issue #194: WICKET-6287 Switch from json.org to open-json - 6.x

2016-11-25 Thread klopfdreh
Github user klopfdreh commented on the issue: https://github.com/apache/wicket/pull/194 fyi @martin-g --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or i

[GitHub] wicket issue #195: WICKET-6287 Switch from json.org to open-json - 7.x

2016-11-25 Thread klopfdreh
Github user klopfdreh commented on the issue: https://github.com/apache/wicket/pull/195 fyi @martin-g --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or i

[GitHub] wicket pull request #195: WICKET-6287 Switch from json.org to open-json - 7....

2016-11-25 Thread klopfdreh
GitHub user klopfdreh opened a pull request: https://github.com/apache/wicket/pull/195 WICKET-6287 Switch from json.org to open-json - 7.x There are clirr errors, because of some classes missing in open-json You can merge this pull request into a Git repository by running: $ gi

[GitHub] wicket pull request #194: WICKET-6287 Switch from json.org to open-json - 6....

2016-11-25 Thread klopfdreh
GitHub user klopfdreh opened a pull request: https://github.com/apache/wicket/pull/194 WICKET-6287 Switch from json.org to open-json - 6.x There are clirr errors, because of some classes missing in open-json You can merge this pull request into a Git repository by running: $ gi

Re: using lambdas fro Args?

2016-11-25 Thread Ernesto Reinaldo Barreiro
ok. On Fri, Nov 25, 2016 at 1:16 PM, Martin Grigorov wrote: > What would be the benefit ? > > The logging libraries provide such API but there one want to avoid > calculating the log message unless really needed. > In our case the "name" is not something expensive to calculate. It is a > static

Re: using lambdas fro Args?

2016-11-25 Thread Martin Grigorov
What would be the benefit ? The logging libraries provide such API but there one want to avoid calculating the log message unless really needed. In our case the "name" is not something expensive to calculate. It is a static string (always?!). What would bring value is using the Java 8 goodies to

Re: using lambdas fro Args?

2016-11-25 Thread Ernesto Reinaldo Barreiro
*public class *Args { *public static * T notNull(T argument, Supplier name) { *if*(argument == *null*) { *throw new *IllegalArgumentException(name.get() + *" may not be null"*); } *else *{ *return *argument; } } *public void *test() {

Re: using lambdas fro Args?

2016-11-25 Thread Martin Grigorov
How the new usage would look like ? Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On Fri, Nov 25, 2016 at 12:08 PM, Ernesto Reinaldo Barreiro < reier...@gmail.com> wrote: > Hi, > > Does it make sense to use lambdas for things like > > https://github.com/apache/wic

using lambdas fro Args?

2016-11-25 Thread Ernesto Reinaldo Barreiro
Hi, Does it make sense to use lambdas for things like https://github.com/apache/wicket/blob/master/wicket-util/src/main/java/org/apache/wicket/util/lang/Args.java#L37 ? -- Regards - Ernesto Reinaldo Barreiro

Re: One more Apache project uses Wicket - AsterixDb

2016-11-25 Thread Andrea Del Bene
Very good! I will add it to the list this weekend. Andrea. On 24/11/2016 21:51, Martin Grigorov wrote: https://github.com/apache/asterixdb/search?l=Maven+POM&q=wicket&utf8=%E2%9C%93 we should add it to http://wicket.apache.org/apache/! Martin Grigorov Wicket Training and Consulting https://