Details
-
Type:
Feature Request
-
Status: Closed (View Workflow)
-
Priority:
Minor
-
Resolution:
Won't Fix
-
Affects Version/s: 1.0.0.Beta1
-
Fix Version/s: None
-
Component/s: Builtin Plugins, Java EE
-
Labels:None
Gliffy Diagrams
Activity
- All
- Comments
- Work Log
- History
- Activity
- Links Hierarchy
Ah, I think I am thinking about something else.
You would:
@Inject UserQuery query;
|
then call:
User user = query.getUserByName("blah");
|
But the code to actually instantiate and execute the query would be abstracted away. It gives you a typesafe way of referencing queries.
Your approach has benefits as well, since you could just:
@Inject @All List<User> users;
|
So I get that, too. I think both compliment each other. The query I defined would not @Produce anything, though.
Show
Lincoln Baxter III
added a comment - Ah, I think I am thinking about something else.
You would:
@Inject UserQuery query;
then call:
User user = query.getUserByName("blah");
But the code to actually instantiate and execute the query would be abstracted away. It gives you a typesafe way of referencing queries.
Your approach has benefits as well, since you could just:
@Inject @All List<User> users;
So I get that, too. I think both compliment each other. The query I defined would not @Produce anything, though.
Lincoln Baxter III
added a comment - Ah, I think I am thinking about something else.
You would:
@Inject UserQuery query;
then call:
User user = query.getUserByName("blah");
But the code to actually instantiate and execute the query would be abstracted away. It gives you a typesafe way of referencing queries.
Your approach has benefits as well, since you could just:
@Inject @All List<User> users;
So I get that, too. I think both compliment each other. The query I defined would not @Produce anything, though. Yes, this approach works very well if you just want to inject the query. I was wondering how this would work from something like EL. But probably with method parameters it's not a problem, assuming you can cache the result of the query in a context.
This would be horrible in EL without caching 
Show
Lincoln Baxter III
added a comment - This would be horrible in EL without caching
Lincoln Baxter III
added a comment - This would be horrible in EL without caching This should belong to a specific framework addon
Show
George Gastaldi
added a comment - This should belong to a specific framework addon
George Gastaldi
added a comment - This should belong to a specific framework addon 
https://github.com/seam/seam-example-confbuzz/blob/develop/src/main/java/seam/example/confbuzz/TodaysConferencesQuery.java
https://github.com/seam/seam-example-confbuzz/blob/develop/src/main/java/seam/example/confbuzz/AllConferenceQuery.java