Rewrite handling of "storage-side old values" both for PK/ident and for
[dbsrgits/DBIx-Class.git] / lib / DBIx / Class / Ordered.pm
1 package DBIx::Class::Ordered;
2 use strict;
3 use warnings;
4 use base qw( DBIx::Class );
5
6 use List::Util 'first';
7 use namespace::clean;
8
9 =head1 NAME
10
11 DBIx::Class::Ordered - Modify the position of objects in an ordered list.
12
13 =head1 SYNOPSIS
14
15 Create a table for your ordered data.
16
17   CREATE TABLE items (
18     item_id INTEGER PRIMARY KEY AUTOINCREMENT,
19     name TEXT NOT NULL,
20     position INTEGER NOT NULL
21   );
22
23 Optionally, add one or more columns to specify groupings, allowing you 
24 to maintain independent ordered lists within one table:
25
26   CREATE TABLE items (
27     item_id INTEGER PRIMARY KEY AUTOINCREMENT,
28     name TEXT NOT NULL,
29     position INTEGER NOT NULL,
30     group_id INTEGER NOT NULL
31   );
32
33 Or even
34
35   CREATE TABLE items (
36     item_id INTEGER PRIMARY KEY AUTOINCREMENT,
37     name TEXT NOT NULL,
38     position INTEGER NOT NULL,
39     group_id INTEGER NOT NULL,
40     other_group_id INTEGER NOT NULL
41   );
42
43 In your Schema or DB class add "Ordered" to the top 
44 of the component list.
45
46   __PACKAGE__->load_components(qw( Ordered ... ));
47
48 Specify the column that stores the position number for 
49 each row.
50
51   package My::Item;
52   __PACKAGE__->position_column('position');
53
54 If you are using one grouping column, specify it as follows:
55
56   __PACKAGE__->grouping_column('group_id');
57
58 Or if you have multiple grouping columns:
59
60   __PACKAGE__->grouping_column(['group_id', 'other_group_id']);
61
62 That's it, now you can change the position of your objects.
63
64   #!/use/bin/perl
65   use My::Item;
66
67   my $item = My::Item->create({ name=>'Matt S. Trout' });
68   # If using grouping_column:
69   my $item = My::Item->create({ name=>'Matt S. Trout', group_id=>1 });
70
71   my $rs = $item->siblings();
72   my @siblings = $item->siblings();
73
74   my $sibling;
75   $sibling = $item->first_sibling();
76   $sibling = $item->last_sibling();
77   $sibling = $item->previous_sibling();
78   $sibling = $item->next_sibling();
79
80   $item->move_previous();
81   $item->move_next();
82   $item->move_first();
83   $item->move_last();
84   $item->move_to( $position );
85   $item->move_to_group( 'groupname' );
86   $item->move_to_group( 'groupname', $position );
87   $item->move_to_group( {group_id=>'groupname', 'other_group_id=>'othergroupname'} );
88   $item->move_to_group( {group_id=>'groupname', 'other_group_id=>'othergroupname'}, $position );
89
90 =head1 DESCRIPTION
91
92 This module provides a simple interface for modifying the ordered 
93 position of DBIx::Class objects.
94
95 =head1 AUTO UPDATE
96
97 All of the move_* methods automatically update the rows involved in 
98 the query.  This is not configurable and is due to the fact that if you 
99 move a record it always causes other records in the list to be updated.
100
101 =head1 METHODS
102
103 =head2 position_column
104
105   __PACKAGE__->position_column('position');
106
107 Sets and retrieves the name of the column that stores the 
108 positional value of each record.  Defaults to "position".
109
110 =cut
111
112 __PACKAGE__->mk_classdata( 'position_column' => 'position' );
113
114 =head2 grouping_column
115
116   __PACKAGE__->grouping_column('group_id');
117
118 This method specifies a column to limit all queries in 
119 this module by.  This effectively allows you to have multiple 
120 ordered lists within the same table.
121
122 =cut
123
124 __PACKAGE__->mk_classdata( 'grouping_column' );
125
126 =head2 null_position_value
127
128   __PACKAGE__->null_position_value(undef);
129
130 This method specifies a value of L</position_column> which B<would
131 never be assigned to a row> during normal operation. When
132 a row is moved, its position is set to this value temporarily, so
133 that any unique constraints can not be violated. This value defaults
134 to 0, which should work for all cases except when your positions do
135 indeed start from 0.
136
137 =cut
138
139 __PACKAGE__->mk_classdata( 'null_position_value' => 0 );
140
141 =head2 siblings
142
143   my $rs = $item->siblings();
144   my @siblings = $item->siblings();
145
146 Returns an B<ordered> resultset of all other objects in the same
147 group excluding the one you called it on.
148
149 The ordering is a backwards-compatibility artifact - if you need
150 a resultset with no ordering applied use L</_siblings>
151
152 =cut
153 sub siblings {
154     my $self = shift;
155     return $self->_siblings->search ({}, { order_by => $self->position_column } );
156 }
157
158 =head2 previous_siblings
159
160   my $prev_rs = $item->previous_siblings();
161   my @prev_siblings = $item->previous_siblings();
162
163 Returns a resultset of all objects in the same group
164 positioned before the object on which this method was called.
165
166 =cut
167 sub previous_siblings {
168     my $self = shift;
169     my $position_column = $self->position_column;
170     my $position = $self->get_column ($position_column);
171     return ( defined $position
172         ? $self->_siblings->search ({ $position_column => { '<', $position } })
173         : $self->_siblings
174     );
175 }
176
177 =head2 next_siblings
178
179   my $next_rs = $item->next_siblings();
180   my @next_siblings = $item->next_siblings();
181
182 Returns a resultset of all objects in the same group
183 positioned after the object on which this method was called.
184
185 =cut
186 sub next_siblings {
187     my $self = shift;
188     my $position_column = $self->position_column;
189     my $position = $self->get_column ($position_column);
190     return ( defined $position
191         ? $self->_siblings->search ({ $position_column => { '>', $position } })
192         : $self->_siblings
193     );
194 }
195
196 =head2 previous_sibling
197
198   my $sibling = $item->previous_sibling();
199
200 Returns the sibling that resides one position back.  Returns 0
201 if the current object is the first one.
202
203 =cut
204
205 sub previous_sibling {
206     my $self = shift;
207     my $position_column = $self->position_column;
208
209     my $psib = $self->previous_siblings->search(
210         {},
211         { rows => 1, order_by => { '-desc' => $position_column } },
212     )->single;
213
214     return defined $psib ? $psib : 0;
215 }
216
217 =head2 first_sibling
218
219   my $sibling = $item->first_sibling();
220
221 Returns the first sibling object, or 0 if the first sibling 
222 is this sibling.
223
224 =cut
225
226 sub first_sibling {
227     my $self = shift;
228     my $position_column = $self->position_column;
229
230     my $fsib = $self->previous_siblings->search(
231         {},
232         { rows => 1, order_by => { '-asc' => $position_column } },
233     )->single;
234
235     return defined $fsib ? $fsib : 0;
236 }
237
238 =head2 next_sibling
239
240   my $sibling = $item->next_sibling();
241
242 Returns the sibling that resides one position forward. Returns 0
243 if the current object is the last one.
244
245 =cut
246
247 sub next_sibling {
248     my $self = shift;
249     my $position_column = $self->position_column;
250     my $nsib = $self->next_siblings->search(
251         {},
252         { rows => 1, order_by => { '-asc' => $position_column } },
253     )->single;
254
255     return defined $nsib ? $nsib : 0;
256 }
257
258 =head2 last_sibling
259
260   my $sibling = $item->last_sibling();
261
262 Returns the last sibling, or 0 if the last sibling is this 
263 sibling.
264
265 =cut
266
267 sub last_sibling {
268     my $self = shift;
269     my $position_column = $self->position_column;
270     my $lsib = $self->next_siblings->search(
271         {},
272         { rows => 1, order_by => { '-desc' => $position_column } },
273     )->single;
274
275     return defined $lsib ? $lsib : 0;
276 }
277
278 # an optimized method to get the last sibling position value without inflating a row object
279 sub _last_sibling_posval {
280     my $self = shift;
281     my $position_column = $self->position_column;
282
283     my $cursor = $self->next_siblings->search(
284         {},
285         { rows => 1, order_by => { '-desc' => $position_column }, select => $position_column },
286     )->cursor;
287
288     my ($pos) = $cursor->next;
289     return $pos;
290 }
291
292 =head2 move_previous
293
294   $item->move_previous();
295
296 Swaps position with the sibling in the position previous in
297 the list.  Returns 1 on success, and 0 if the object is
298 already the first one.
299
300 =cut
301
302 sub move_previous {
303     my $self = shift;
304     return $self->move_to ($self->_position - 1);
305 }
306
307 =head2 move_next
308
309   $item->move_next();
310
311 Swaps position with the sibling in the next position in the
312 list.  Returns 1 on success, and 0 if the object is already
313 the last in the list.
314
315 =cut
316
317 sub move_next {
318     my $self = shift;
319     return 0 unless defined $self->_last_sibling_posval;  # quick way to check for no more siblings
320     return $self->move_to ($self->_position + 1);
321 }
322
323 =head2 move_first
324
325   $item->move_first();
326
327 Moves the object to the first position in the list.  Returns 1
328 on success, and 0 if the object is already the first.
329
330 =cut
331
332 sub move_first {
333     return shift->move_to( 1 );
334 }
335
336 =head2 move_last
337
338   $item->move_last();
339
340 Moves the object to the last position in the list.  Returns 1
341 on success, and 0 if the object is already the last one.
342
343 =cut
344
345 sub move_last {
346     my $self = shift;
347     my $last_posval = $self->_last_sibling_posval;
348
349     return 0 unless defined $last_posval;
350
351     return $self->move_to( $self->_position_from_value ($last_posval) );
352 }
353
354 =head2 move_to
355
356   $item->move_to( $position );
357
358 Moves the object to the specified position.  Returns 1 on
359 success, and 0 if the object is already at the specified
360 position.
361
362 =cut
363
364 sub move_to {
365     my( $self, $to_position ) = @_;
366     return 0 if ( $to_position < 1 );
367
368     my $position_column = $self->position_column;
369
370     if ($self->is_column_changed ($position_column) ) {
371       # something changed our position, we need to know where we
372       # used to be - use the stashed value
373       $self->store_column($position_column, delete $self->{_column_data_in_storage}{$position_column});
374       delete $self->{_dirty_columns}{$position_column};
375     }
376
377     my $from_position = $self->_position;
378
379     if ( $from_position == $to_position ) {   # FIXME this will not work for non-numeric order
380       return 0;
381     }
382
383     my $guard = $self->result_source->schema->txn_scope_guard;
384
385     my ($direction, @between);
386     if ( $from_position < $to_position ) {
387       $direction = -1;
388       @between = map { $self->_position_value ($_) } ( $from_position + 1, $to_position );
389     }
390     else {
391       $direction = 1;
392       @between = map { $self->_position_value ($_) } ( $to_position, $from_position - 1 );
393     }
394
395     my $new_pos_val = $self->_position_value ($to_position);  # record this before the shift
396
397     # we need to null-position the moved row if the position column is part of a constraint
398     if (grep { $_ eq $position_column } ( map { @$_ } (values %{{ $self->result_source->unique_constraints }} ) ) ) {
399       $self->_ordered_internal_update({ $position_column => $self->null_position_value });
400     }
401
402     $self->_shift_siblings ($direction, @between);
403     $self->_ordered_internal_update({ $position_column => $new_pos_val });
404
405     $guard->commit;
406     return 1;
407 }
408
409 =head2 move_to_group
410
411   $item->move_to_group( $group, $position );
412
413 Moves the object to the specified position of the specified
414 group, or to the end of the group if $position is undef.
415 1 is returned on success, and 0 is returned if the object is
416 already at the specified position of the specified group.
417
418 $group may be specified as a single scalar if only one 
419 grouping column is in use, or as a hashref of column => value pairs
420 if multiple grouping columns are in use.
421
422 =cut
423
424 sub move_to_group {
425     my( $self, $to_group, $to_position ) = @_;
426
427     # if we're given a single value, turn it into a hashref
428     unless (ref $to_group eq 'HASH') {
429         my @gcols = $self->_grouping_columns;
430
431         $self->throw_exception ('Single group supplied for a multi-column group identifier') if @gcols > 1;
432         $to_group = {$gcols[0] => $to_group};
433     }
434
435     my $position_column = $self->position_column;
436
437     return 0 if ( defined($to_position) and $to_position < 1 );
438
439     # check if someone changed the _grouping_columns - this will
440     # prevent _is_in_group working, so we need to restore the
441     # original stashed values
442     for ($self->_grouping_columns) {
443       if ($self->is_column_changed ($_)) {
444         $self->store_column($_, delete $self->{_column_data_in_storage}{$_});
445         delete $self->{_dirty_columns}{$_};
446       }
447     }
448
449     if ($self->_is_in_group ($to_group) ) {
450       my $ret;
451       if (defined $to_position) {
452         $ret = $self->move_to ($to_position);
453       }
454
455       return $ret||0;
456     }
457
458     my $guard = $self->result_source->schema->txn_scope_guard;
459
460     # Move to end of current group to adjust siblings
461     $self->move_last;
462
463     $self->set_inflated_columns({ %$to_group, $position_column => undef });
464     my $new_group_last_posval = $self->_last_sibling_posval;
465     my $new_group_last_position = $self->_position_from_value (
466       $new_group_last_posval
467     );
468
469     if ( not defined($to_position) or $to_position > $new_group_last_position) {
470       $self->set_column(
471         $position_column => $new_group_last_position
472           ? $self->_next_position_value ( $new_group_last_posval )
473           : $self->_initial_position_value
474       );
475     }
476     else {
477       my $bumped_pos_val = $self->_position_value ($to_position);
478       my @between = map { $self->_position_value ($_) } ($to_position, $new_group_last_position);
479       $self->_shift_siblings (1, @between);   #shift right
480       $self->set_column( $position_column => $bumped_pos_val );
481     }
482
483     $self->_ordered_internal_update;
484
485     $guard->commit;
486
487     return 1;
488 }
489
490 =head2 insert
491
492 Overrides the DBIC insert() method by providing a default 
493 position number.  The default will be the number of rows in 
494 the table +1, thus positioning the new record at the last position.
495
496 =cut
497
498 sub insert {
499     my $self = shift;
500     my $position_column = $self->position_column;
501
502     unless ($self->get_column($position_column)) {
503         my $lsib_posval = $self->_last_sibling_posval;
504         $self->set_column(
505             $position_column => (defined $lsib_posval
506                 ? $self->_next_position_value ( $lsib_posval )
507                 : $self->_initial_position_value
508             )
509         );
510     }
511
512     return $self->next::method( @_ );
513 }
514
515 =head2 update
516
517 Overrides the DBIC update() method by checking for a change
518 to the position and/or group columns.  Movement within a
519 group or to another group is handled by repositioning
520 the appropriate siblings.  Position defaults to the end
521 of a new group if it has been changed to undef.
522
523 =cut
524
525 sub update {
526   my $self = shift;
527
528   # this is set by _ordered_internal_update()
529   return $self->next::method(@_) if $self->{_ORDERED_INTERNAL_UPDATE};
530
531   my $upd = shift;
532   $self->set_inflated_columns($upd) if $upd;
533
534   my $position_column = $self->position_column;
535   my @group_columns = $self->_grouping_columns;
536
537   # see if the order is already changed
538   my $changed_ordering_cols = { map { $_ => $self->get_column($_) } grep { $self->is_column_changed($_) } ($position_column, @group_columns) };
539
540   # nothing changed - short circuit
541   if (! keys %$changed_ordering_cols) {
542     return $self->next::method( undef, @_ );
543   }
544   elsif (defined first { exists $changed_ordering_cols->{$_} } @group_columns ) {
545     $self->move_to_group(
546       # since the columns are already re-set the _grouping_clause is correct
547       # move_to_group() knows how to get the original storage values
548       { $self->_grouping_clause },
549
550       # The FIXME bit contradicts the documentation: POD states that
551       # when changing groups without supplying explicit positions in
552       # move_to_group(), we push the item to the end of the group.
553       # However when I was rewriting this, the position from the old
554       # group was clearly passed to the new one
555       # Probably needs to go away (by ribasushi)
556       (exists $changed_ordering_cols->{$position_column}
557         ? $changed_ordering_cols->{$position_column}  # means there was a position change supplied with the update too
558         : $self->_position                            # FIXME! (replace with undef)
559       ),
560     );
561   }
562   else {
563     $self->move_to($changed_ordering_cols->{$position_column});
564   }
565
566   return $self;
567 }
568
569 =head2 delete
570
571 Overrides the DBIC delete() method by first moving the object
572 to the last position, then deleting it, thus ensuring the
573 integrity of the positions.
574
575 =cut
576
577 sub delete {
578     my $self = shift;
579
580     my $guard = $self->result_source->schema->txn_scope_guard;
581
582     $self->move_last;
583
584     my @res;
585     if (not defined wantarray) {
586         $self->next::method( @_ );
587     }
588     elsif (wantarray) {
589         @res = $self->next::method( @_ );
590     }
591     else {
592         $res[0] = $self->next::method( @_ );
593     }
594
595     $guard->commit;
596     return wantarray ? @res : $res[0];
597 }
598
599 # add the current position/group to the things we track old values for
600 sub _track_storage_value {
601   my ($self, $col) = @_;
602   return $self->next::method($col) || defined first { $_ eq $col } ($self->position_column, $self->_grouping_columns);
603 }
604
605 =head1 METHODS FOR EXTENDING ORDERED
606
607 You would want to override the methods below if you use sparse
608 (non-linear) or non-numeric position values. This can be useful
609 if you are working with preexisting non-normalised position data,
610 or if you need to work with materialized path columns.
611
612 =head2 _position_from_value
613
614   my $num_pos = $item->_position_from_value ( $pos_value )
615
616 Returns the B<absolute numeric position> of an object with a B<position
617 value> set to C<$pos_value>. By default simply returns C<$pos_value>.
618
619 =cut
620 sub _position_from_value {
621     my ($self, $val) = @_;
622
623     return 0 unless defined $val;
624
625 #    #the right way to do this
626 #    return $self -> _group_rs
627 #                 -> search({ $self->position_column => { '<=', $val } })
628 #                 -> count
629
630     return $val;
631 }
632
633 =head2 _position_value
634
635   my $pos_value = $item->_position_value ( $pos )
636
637 Returns the B<value> of L</position_column> of the object at numeric
638 position C<$pos>. By default simply returns C<$pos>.
639
640 =cut
641 sub _position_value {
642     my ($self, $pos) = @_;
643
644 #    #the right way to do this (not optimized)
645 #    my $position_column = $self->position_column;
646 #    return $self -> _group_rs
647 #                 -> search({}, { order_by => $position_column })
648 #                 -> slice ( $pos - 1)
649 #                 -> single
650 #                 -> get_column ($position_column);
651
652     return $pos;
653 }
654
655 =head2 _initial_position_value
656
657   __PACKAGE__->_initial_position_value(0);
658
659 This method specifies a B<value> of L</position_column> which is assigned
660 to the first inserted element of a group, if no value was supplied at
661 insertion time. All subsequent values are derived from this one by
662 L</_next_position_value> below. Defaults to 1.
663
664 =cut
665
666 __PACKAGE__->mk_classdata( '_initial_position_value' => 1 );
667
668 =head2 _next_position_value
669
670   my $new_value = $item->_next_position_value ( $position_value )
671
672 Returns a position B<value> that would be considered C<next> with
673 regards to C<$position_value>. Can be pretty much anything, given
674 that C<< $position_value < $new_value >> where C<< < >> is the
675 SQL comparison operator (usually works fine on strings). The
676 default method expects C<$position_value> to be numeric, and
677 returns C<$position_value + 1>
678
679 =cut
680 sub _next_position_value {
681     return $_[1] + 1;
682 }
683
684 =head2 _shift_siblings
685
686   $item->_shift_siblings ($direction, @between)
687
688 Shifts all siblings with B<positions values> in the range @between
689 (inclusive) by one position as specified by $direction (left if < 0,
690  right if > 0). By default simply increments/decrements each
691 L</position_column> value by 1, doing so in a way as to not violate
692 any existing constraints.
693
694 Note that if you override this method and have unique constraints
695 including the L</position_column> the shift is not a trivial task.
696 Refer to the implementation source of the default method for more
697 information.
698
699 =cut
700 sub _shift_siblings {
701     my ($self, $direction, @between) = @_;
702     return 0 unless $direction;
703
704     my $position_column = $self->position_column;
705
706     my ($op, $ord);
707     if ($direction < 0) {
708         $op = '-';
709         $ord = 'asc';
710     }
711     else {
712         $op = '+';
713         $ord = 'desc';
714     }
715
716     my $shift_rs = $self->_group_rs-> search ({ $position_column => { -between => \@between } });
717
718     # some databases (sqlite) are dumb and can not do a blanket
719     # increment/decrement. So what we do here is check if the
720     # position column is part of a unique constraint, and do a
721     # one-by-one update if this is the case
722
723     my $rsrc = $self->result_source;
724
725     if (grep { $_ eq $position_column } ( map { @$_ } (values %{{ $rsrc->unique_constraints }} ) ) ) {
726
727         my @pcols = $rsrc->_pri_cols;
728         my $cursor = $shift_rs->search ({}, { order_by => { "-$ord", $position_column }, columns => \@pcols } )->cursor;
729         my $rs = $self->result_source->resultset;
730
731         my @all_pks = $cursor->all;
732         while (my $pks = shift @all_pks) {
733           my $cond;
734           for my $i (0.. $#pcols) {
735             $cond->{$pcols[$i]} = $pks->[$i];
736           }
737
738           $rs->search($cond)->update ({ $position_column => \ "$position_column $op 1" } );
739         }
740     }
741     else {
742         $shift_rs->update ({ $position_column => \ "$position_column $op 1" } );
743     }
744 }
745
746 =head1 PRIVATE METHODS
747
748 These methods are used internally.  You should never have the 
749 need to use them.
750
751 =head2 _group_rs
752
753 This method returns a resultset containing all members of the row
754 group (including the row itself).
755
756 =cut
757 sub _group_rs {
758     my $self = shift;
759     return $self->result_source->resultset->search({$self->_grouping_clause()});
760 }
761
762 =head2 _siblings
763
764 Returns an unordered resultset of all objects in the same group
765 excluding the object you called this method on.
766
767 =cut
768 sub _siblings {
769     my $self = shift;
770     my $position_column = $self->position_column;
771     return defined (my $pos = $self->get_column($position_column))
772         ? $self->_group_rs->search(
773             { $position_column => { '!=' => $self->get_column($position_column) } },
774           )
775         : $self->_group_rs
776     ;
777 }
778
779 =head2 _position
780
781   my $num_pos = $item->_position;
782
783 Returns the B<absolute numeric position> of the current object, with the
784 first object being at position 1, its sibling at position 2 and so on.
785
786 =cut
787 sub _position {
788     my $self = shift;
789     return $self->_position_from_value ($self->get_column ($self->position_column) );
790 }
791
792 =head2 _grouping_clause
793
794 This method returns one or more name=>value pairs for limiting a search
795 by the grouping column(s).  If the grouping column is not defined then 
796 this will return an empty list.
797
798 =cut
799 sub _grouping_clause {
800     my( $self ) = @_;
801     return map {  $_ => $self->get_column($_)  } $self->_grouping_columns();
802 }
803
804 =head2 _get_grouping_columns
805
806 Returns a list of the column names used for grouping, regardless of whether
807 they were specified as an arrayref or a single string, and returns ()
808 if there is no grouping.
809
810 =cut
811 sub _grouping_columns {
812     my( $self ) = @_;
813     my $col = $self->grouping_column();
814     if (ref $col eq 'ARRAY') {
815         return @$col;
816     } elsif ($col) {
817         return ( $col );
818     } else {
819         return ();
820     }
821 }
822
823 =head2 _is_in_group
824
825     $item->_is_in_group( {user => 'fred', list => 'work'} )
826
827 Returns true if the object is in the group represented by hashref $other
828
829 =cut
830 sub _is_in_group {
831     my ($self, $other) = @_;
832     my $current = {$self->_grouping_clause};
833
834     no warnings qw/uninitialized/;
835
836     return 0 if (
837         join ("\x00", sort keys %$current)
838             ne
839         join ("\x00", sort keys %$other)
840     );
841     for my $key (keys %$current) {
842         return 0 if $current->{$key} ne $other->{$key};
843     }
844     return 1;
845 }
846
847 =head2 _ordered_internal_update
848
849 This is a short-circuited method, that is used internally by this
850 module to update positioning values in isolation (i.e. without
851 triggering any of the positioning integrity code).
852
853 Some day you might get confronted by datasets that have ambiguous
854 positioning data (e.g. duplicate position values within the same group,
855 in a table without unique constraints). When manually fixing such data
856 keep in mind that you can not invoke L<DBIx::Class::Row/update> like
857 you normally would, as it will get confused by the wrong data before
858 having a chance to update the ill-defined row. If you really know what
859 you are doing use this method which bypasses any hooks introduced by
860 this module.
861
862 =cut
863
864 sub _ordered_internal_update {
865     my $self = shift;
866     local $self->{_ORDERED_INTERNAL_UPDATE} = 1;
867     return $self->update (@_);
868 }
869
870 1;
871
872 __END__
873
874 =head1 CAVEATS
875
876 =head2 Resultset Methods
877
878 Note that all Insert/Create/Delete overrides are happening on
879 L<DBIx::Class::Row> methods only. If you use the
880 L<DBIx::Class::ResultSet> versions of
881 L<update|DBIx::Class::ResultSet/update> or
882 L<delete|DBIx::Class::ResultSet/delete>, all logic present in this
883 module will be bypassed entirely (possibly resulting in a broken
884 order-tree). Instead always use the
885 L<update_all|DBIx::Class::ResultSet/update_all> and
886 L<delete_all|DBIx::Class::ResultSet/delete_all> methods, which will
887 invoke the corresponding L<row|DBIx::Class::Row> method on every
888 member of the given resultset.
889
890 =head2 Race Condition on Insert
891
892 If a position is not specified for an insert, a position
893 will be chosen based either on L</_initial_position_value> or
894 L</_next_position_value>, depending if there are already some
895 items in the current group. The space of time between the
896 necessary selects and insert introduces a race condition.
897 Having unique constraints on your position/group columns,
898 and using transactions (see L<DBIx::Class::Storage/txn_do>)
899 will prevent such race conditions going undetected.
900
901 =head2 Multiple Moves
902
903 Be careful when issuing move_* methods to multiple objects.  If 
904 you've pre-loaded the objects then when you move one of the objects 
905 the position of the other object will not reflect their new value 
906 until you reload them from the database - see
907 L<DBIx::Class::Row/discard_changes>.
908
909 There are times when you will want to move objects as groups, such 
910 as changing the parent of several objects at once - this directly 
911 conflicts with this problem.  One solution is for us to write a 
912 ResultSet class that supports a parent() method, for example.  Another 
913 solution is to somehow automagically modify the objects that exist 
914 in the current object's result set to have the new position value.
915
916 =head2 Default Values
917
918 Using a database defined default_value on one of your group columns
919 could result in the position not being assigned correctly.
920
921 =head1 AUTHOR
922
923  Original code framework
924    Aran Deltac <bluefeet@cpan.org>
925
926  Constraints support and code generalisation
927    Peter Rabbitson <ribasushi@cpan.org>
928
929 =head1 LICENSE
930
931 You may distribute this code under the same terms as Perl itself.
932