Well at some point each IProjection will need a boolean for IsSelected. It could be a local variable set via constructor instead.
On Fri, Dec 19, 2008 at 1:53 PM, Tuna Toksöz <[email protected]> wrote: > Why not just having an GroupByOnly thing, without having to implement > IsSelected? I have to take a look at the code, but doing it this way didn't > sound natural. > > > On Fri, Dec 19, 2008 at 11:51 PM, Will Shaver <[email protected]>wrote: > >> Ok, if you want to add this functionality it shouldn't be that hard. >> Here's what you'll need to do: >> 1. Modify IProjection to include a boolean for IsSelected >> 2. Implement IsSelected for all subclasses >> 3. When (IProjection p).ToSqlString() is called, it should first check >> IsSelected before outputting a string >> 4. Add a Projections.GroupOnlyProperty(string propertyname){ >> new PropertyProjection(propertyName, true) . IsSelected = false; >> >> Good luck. >> -Will >> >> >> On Fri, Dec 19, 2008 at 12:54 PM, alex <[email protected]> wrote: >> >>> >>> The reason why I don't need the "group by" column in my SELECT is >>> because I'm going to use it as a subquery (DetachedCriteria). I need a >>> single column SELECT so I could use the result of this subquery in the >>> IN statement of the main query. >>> >>> On Dec 19, 10:38 am, "Will Shaver" <[email protected]> wrote: >>> > Why do you need to be rid of the group by column in your results? It >>> should >>> > not adversely affect your query performance by a huge margin. The >>> result >>> > order can be maintained through adding order by to the query. It may be >>> > possible to hack the source so as to add a GroupByButDontSelect() >>> method of >>> > some sort, but for what end? >>> > I wrote the existing group by code for the criteria queries (last I >>> checked >>> > it still doesn't exist in Hibernate). It might not be too bad to add in >>> a >>> > boolean for actually selecting it or not, I'm not sure as it has been >>> > several months since I've reviewed that section of code. >>> > >>> > -Will >>> > >>> > On Fri, Dec 19, 2008 at 7:57 AM, Fabio Maulo <[email protected]> >>> wrote: >>> > > 2008/12/19 alex <[email protected]> >>> > >>> > >> Another reason for using Criteria API is Resharper and intellisense. >>> > >>> > > well... you have the problem and I hope you can solve it soon in your >>> way. >>> > > I did myyyyy waaaaay! (as said THE VOICE) >>> > > -- >>> > > Fabio Maulo >>> >>> >> >> >> > > > -- > Tuna Toksöz > http://tunatoksoz.com > > Typos included to enhance the readers attention! > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "nhusers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/nhusers?hl=en -~----------~----~----~----~------~----~------~--~---
