yes, it was introduced in revision 3860 by the following change:

        void ISqlStringVisitor.Parameter()
        {
            string name = formatter.GetParameterName(parameterIndex);
            parameterIndex++;
            result.Append(name);
        }

which was changed to this:

        void ISqlStringVisitor.Parameter(Parameter parameter)
        {
            string name = formatter.GetParameterName(
                parameter.OriginalPositionInQuery ?? parameterIndex);
            parameterIndex++;
            result.Append(name);
        }

the change was necessary to get some really complex paging with order by
queries working, but it did unfortunately also break paging parameters in
certain multicriteria situations

On Mon, Dec 15, 2008 at 10:04 PM, Tuna Toksöz <[email protected]> wrote:

> Davy do you know if this issue existed before new 2005 dialect proposal?
>
>
> On Mon, Dec 15, 2008 at 10:32 PM, Davy Brion <[email protected]> wrote:
>
>> well not sure if it's exactly the same... in my issue, the parameters are
>> also ordered incorrectly, but in the other direction and only when using
>> MultiCriteria (and probably MultiQuery as well)
>>
>> i have a possible fix in mind, which i'm going to try soon, but it would
>> be somewhat specific to the MultiCriteria/MultiQuery situation
>>
>>
>> On Mon, Dec 15, 2008 at 9:30 PM, Tuna Toksöz <[email protected]> wrote:
>>
>>> probably. So moving previous as a subtask of your issue.
>>>
>>>
>>> On Mon, Dec 15, 2008 at 10:29 PM, Davy Brion <[email protected]>wrote:
>>>
>>>> hmm, this also appears to be related to an issue i created earlier
>>>> today:
>>>> http://jira.nhibernate.org/browse/NH-1609
>>>>
>>>>
>>>> On Mon, Dec 15, 2008 at 9:17 PM, Tuna Toksöz <[email protected]> wrote:
>>>>
>>>>> http://jira.nhibernate.org/browse/NH-1610
>>>>>
>>>>> Here is the issue, add it to your watch list so you can track.
>>>>>
>>>>>
>>>>> On Mon, Dec 15, 2008 at 10:13 PM, Diego Jancic <[email protected]>wrote:
>>>>>
>>>>>>  Sure I will (not at work now, but I'm gonna do it tomorrow)
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> *From:* [email protected] [mailto:
>>>>>> [email protected]] *On Behalf Of *Tuna Toksöz
>>>>>> *Sent:* Monday, December 15, 2008 18:07
>>>>>> *To:* [email protected]
>>>>>> *Subject:* [nhibernate-development] Re: Parameters Bug
>>>>>>
>>>>>>
>>>>>>
>>>>>> Yes that was it! I am going to reopen the issue, btw eventhough it
>>>>>> maynot be related can you update to revision 3945 and see what happens?
>>>>>>
>>>>>> On Mon, Dec 15, 2008 at 10:02 PM, Diego Jancic <[email protected]>
>>>>>> wrote:
>>>>>>
>>>>>>
>>>>>> René André suggested a changed (in the same issue), which was never
>>>>>> applied.
>>>>>> File: src\NHibernate\SqlCommand\SqlString.cs, line: 27
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Tuna Toksöz
>>>>>> http://tunatoksoz.com
>>>>>>
>>>>>> Typos included to enhance the readers attention!
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Tuna Toksöz
>>>>> http://tunatoksoz.com
>>>>>
>>>>> Typos included to enhance the readers attention!
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Davy Brion
>>>> http://davybrion.com
>>>>
>>>
>>>
>>>
>>> --
>>> Tuna Toksöz
>>> http://tunatoksoz.com
>>>
>>> Typos included to enhance the readers attention!
>>>
>>>
>>
>>
>> --
>> Davy Brion
>> http://davybrion.com
>>
>
>
>
> --
> Tuna Toksöz
> http://tunatoksoz.com
>
> Typos included to enhance the readers attention!
>
>


-- 
Davy Brion
http://davybrion.com

Reply via email to