caelum / vraptor4

A web MVC action-based framework, on top of CDI, for fast and maintainable Java development.
http://vraptor.org
Apache License 2.0
350 stars 331 forks source link

Java 8 problems to get parameter names #608

Open garcia-jj opened 10 years ago

garcia-jj commented 10 years ago

If some controller have Java 8 code (like lambdas), Paranamer can't recognize parameter names.

Java 8 users must use this code to use JEP118 instead Paranamer: https://gist.github.com/garcia-jj/1bbfd89a66fed84c60a8

garcia-jj commented 10 years ago

I think to create a produces that check Java version. If is 8 then uses reflection parameter, or else uses Paranamer.

Turini commented 10 years ago

Great! Let me know when it's done to the first release.

garcia-jj commented 10 years ago

What you think to push a RC1? So users can help us to test in real apps before stable release.

felipeweb commented 10 years ago

Good Idea!

Felipe Oliveira Em 07/07/2014 23:11, "Otávio Garcia" notifications@github.com escreveu:

What you think to push a RC1? So users can help us to test in real apps before stable release.

— Reply to this email directly or view it on GitHub https://github.com/caelum/vraptor4/issues/608#issuecomment-48264474.

Turini commented 10 years ago

I like it! Can I release it now? (or there are things to do before it?)

garcia-jj commented 10 years ago

Yes. Release it.

Turini commented 10 years ago

done! In a few hours it will be available on maven repo :)

felipeweb commented 10 years ago

the release is already on maven?

Turini commented 10 years ago

yes, http://search.maven.org/#artifactdetails|br.com.caelum.vraptor|vraptor-java8|4.0.0-RC1|jar

garcia-jj commented 9 years ago

I did a commit some weeks ago fixing this issue on Paranamer. Lambdas are not recognize without adding some constants into BytecodeReadingParanamer.

paul-hammant/paranamer#20

I'll watch when a new Paranamer release are ready to use into Maven central. So I'll send a pull request with the fix.

Thank you.

garcia-jj commented 9 years ago

Waiting paul-hammant/paranamer#22