-
Notifications
You must be signed in to change notification settings - Fork 510
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.
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
minimum support for Play 2.5 #587
Conversation
f74a2f4
to
e722092
Compare
e722092
to
be4079c
Compare
👍 |
/** | ||
* The default responses sent when the invoker is not authenticated or authorized to execute a protected | ||
* action. | ||
* | ||
* @see securesocial.core.java.SecuredActionResponses | ||
*/ | ||
public class DefaultSecuredActionResponses extends Controller implements SecuredActionResponses { | ||
@Inject | ||
play.api.i18n.Messages messages = null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Variable messages, is of type Scala Messages, and later it seems not to be used. See line 48.
@kildievr
|
Which version of Play do you use? From Play 2.4, they introduced injection. Le Friday, May 27, 2016, zezuha [email protected] a écrit :
Olivier Droz |
@odroz. What do you mean? Pull Request 587 was branched from jaliss/securesocial/master in version 3.0-M4 that already has injection, is compatible with Play2.4.4 and in fact it works. I am just trying to run a "javaDemo" project on this pull request. |
+1 |
@zezuha I'm not 100% sure, but I don't think this PR works. For example, I think the following If there's anyone that successfully used this code with Play 2.5, let me how to do that. In the meantime, I created a forked version, tested it briefly, and it seems working with Play 2.5. The code is really messy, but for anyone interested, I uploaded the code and jars on the locations shown below: |
minimum support for Play 2.5