Great. As about the OrderBy I think there should be a reference to OrderByAlias by <see/> tag. The description should then distinguish between standard OrderBy and OrderByAlias method name. Great, great think would be putting some sample code in the <code/> tag. But this would be I guess hard to maintenance... regards
On 14 Sty, 17:45, "Richard Brown \(gmail\)" <[email protected]> wrote: > Good idea. I'm open to suggestions on ways to expand on the current > descriptions. The current descriptions are: > > /// <summary> > /// Add order expressed as a lambda expression > /// </summary> > /// <param name="path">Lambda expression</param> > /// <returns>criteria instance</returns> > IQueryOverOrderBuilder<TRoot,TSubType> OrderBy(Expression<Func<TSubType, > object>> path); > > /// <summary> > /// Add order expressed as a lambda expression > /// </summary> > /// <param name="path">Lambda expression</param> > /// <returns>criteria instance</returns> > IQueryOverOrderBuilder<TRoot,TSubType> OrderBy(Expression<Func<object>> > path); > > /// <summary> > /// Add order for an aliased projection expressed as a lambda expression > /// </summary> > /// <param name="path">Lambda expression</param> > /// <returns>criteria instance</returns> > IQueryOverOrderBuilder<TRoot,TSubType> OrderByAlias(Expression<Func<object>> > path); > > Let me know your thoughts. > > Cheers, > Richard > > -----Original Message----- > From: kamil.kliczbor > Sent: Thursday, January 13, 2011 7:20 PM > To: nhusers > Subject: [nhusers] Re: QueryOver OrderByAlias doesn't sort as expected > > Hi, > thank you for clarification. But there should be some hint how to use > it. For instance in the method summary or something. > > regards > Kamil Kliczbor > > On 12 Sty, 17:40, "Richard Brown \(gmail\)" <[email protected]> > wrote: > > Hi Kamil, > > > Sorry for the delay ... I started to fix this then I realised the current > > functionality is OK (if a little confusingly named). > > > To order by an alias simply use it directly (the OrderByAlias() method is > > purely for aliased projections, and not required for regular aliases). So > > all you need is: > > > this.Session.QueryOver<Product>() > > .JoinAlias(x => x.Category, () => categoryAlias) > > .OrderBy(x => x.Name).Desc > > .ThenBy(() => categoryAlias.Name).Asc > > .List<Product>(); > > > Sorry for the confusion. > > > Richard-----Original Message----- > > From: kamil.kliczbor > > Sent: Thursday, January 06, 2011 8:42 PM > > To: nhusers > > Subject: [nhusers] Re: QueryOver OrderByAlias doesn't sort as expected > > > Thx Richard, > > here is:http://216.121.112.228/browse/NH-2493 > > > On 6 Sty, 12:18, "Richard Brown \(gmail\)" <[email protected]> > > wrote: > > > Hi Kamil, > > > > That looks like a bug to me ... could you raise a JIRA? > > > > Cheers, > > > Richard > > > > -----Original Message----- > > > From: kamil.kliczbor > > > Sent: Wednesday, January 05, 2011 10:51 PM > > > To: nhusers > > > Subject: [nhusers] QueryOver OrderByAlias doesn't sort as expected > > > > Hello everyone, > > > > I cannot use the OrderByAlias / ThenByAlias method in QueryOver query. > > > When I use > > > this.Session.QueryOver<Product>() > > > .OrderBy(x => x.Name).Desc > > > .JoinQueryOver(x => x.Category, () => categoryAlias) > > > .OrderBy(x => x.Name).Asc > > > .List<Product>(); > > > > It produces the right SQL: > > > SELECT this_.ProductId as ProductId6_1_, > > > this_.ProductName as ProductN2_6_1_, > > > ... > > > categoryal1_.Description as Descript3_5_0_ > > > ... > > > FROM Products this_ > > > inner join Categories categoryal1_ > > > on this_.CategoryId = categoryal1_.CategoryId > > > ORDER BY this_.ProductName desc, > > > categoryal1_.CategoryName asc > > > > But when I try to write this query in other way - like this: > > > this.Session.QueryOver<Product>() > > > .JoinAlias(x => x.Category, () => categoryAlias) > > > .OrderBy(x => x.Name).Desc > > > .ThenByAlias(() => categoryAlias.Name).Asc > > > //.OrderByAlias(() => categoryAlias.Name).Asc > > > .List<Product>(); > > > > it produces wrong SQL (insted of category name it orders ascending by > > > product name ): > > > SELECT ... FROM Products this_ inner join Categories categoryal1_ on > > > this_.CategoryId=categoryal1_.CategoryId ORDER BY this_.ProductName > > > desc, this_.ProductName asc > > > > Did I missed something ? > > > > -- > > > 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 > > > athttp://groups.google.com/group/nhusers?hl=en. > > > -- > > 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 > > athttp://groups.google.com/group/nhusers?hl=en. > > -- > 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 > athttp://groups.google.com/group/nhusers?hl=en. -- 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.
