-
Notifications
You must be signed in to change notification settings - Fork 33
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
setter for Navigator is not available in ContextSupport Class #10
Comments
Can you elaborate? I.e. what is the problem you're trying to solve here? |
The problem is we cannot use ContextSupport in JEE environment to let container manage the object creation via @Inject and set the attributes later in the code. |
OK, that's plausible. I'm going to need to dig a little deeper and think about this. Adding mutability is not something we can take lightly. |
I wonder if there's anywhere we use the no-args ContextSupport constructor? It doesn't seem very useful without this setter. |
yes, no-arg constructor is not useful without this setter. |
Adding a setter for navigator should not impact mutability of this class as that class already holds three setters. |
Regarding unit test, I dont see any value for a setter. Anyway, if you insist I can write. |
No description provided.
The text was updated successfully, but these errors were encountered: