OK this seems to work for me. Master branch,
http://github.com/banane/radiant-drag-order/

On Mon, Aug 9, 2010 at 7:49 PM, Wes Gamble <we...@att.net> wrote:

>  FWIW,
>
> I've definitely verified that the "position" column on the Page model is
> being respected in the admin. page display.  When a parent page's children
> are requested using Page#children, the position value is also respected (it
> appears that if two pages have the same position value, then the primary key
> order prevails).  I can move pages around in the list "at will" by
> manipulating the "position" value directly.
>
> Sometimes, for whatever reason, the "position" columns of a set of pages at
> the same level are not being updated correctly.  I haven't been able to
> figure out if it has to do with the "direction" in which the items are being
> moved or not.
>
> I see drag/drop failing occasionally with an error of the type:
>
>   SELECT * FROM "pages" WHERE (slug LIKE ''internal-political-issue'%%') AND 
> ("pages"."parent_id" = 1)
>
> which is again some silly syntax issue where there are too many quotes in
> the WHERE clause.
>
> I suspect maybe two things are going on:
>
> 1) Whatever causes the SQL syntax error
>
> 2) When an error occurs, a proper "rollback" of the original order may not
> be done (which is how you might end up with duplicated position values)
>
> Wes
>
>
> On 8/9/10 9:21 PM, Marshal Linfoot wrote:
>
> Anna, thank you. I'll be happy to test your changes.
>
> On Mon, Aug 9, 2010 at 10:12 PM, banane <ban...@gmail.com> wrote:
>
>> I'll try to rework mine. I got a pull request a few weeks ago from someone
>> who'd merged branches.
>> anna
>>
>>  --
> marshal
>
>

Reply via email to