New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[#1403] 🐛 Fix Groovy compilation unit that tries to write in applicat… #1406

Merged

xael-fry merged 2 commits into playframework:master from holajsh:1403_groovy_compilation_fixes

Apr 23, 2022

Commits on Apr 21, 2022

[playframework#1403 ] 🐛 Fix Groovy compilation unit that tries to writ…
```
…e in application root folder

* Restore the old reflection code that completely replace the OUTPUT phase of Groovy compilation unit
* Fix the compilation exception handling that was producing cast error as not checking properly exception type
```
holajsh committed Apr 21, 2022
Configuration menu
View commit details

Copy full SHA for 5d057d4

Browse repository at this point
Copy the full SHA

5d057d4 View commit details

Browse the repository at this point in the history
[playframework#1403 ] 🐛 Add a comment to explain why reflection is use…
```
…d instead of Groovy native methods
```
holajsh committed Apr 21, 2022
Configuration menu
View commit details

Copy full SHA for 1dbaf5c

Browse repository at this point
Copy the full SHA

1dbaf5c View commit details

Browse the repository at this point in the history