Trailing WS crusade - got to save them bits
[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->result_source->schema->{_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     # Also we do a one-by-one if the position is part of the PK
723     # since once we update a column via scalarref we lose the
724     # ability to retrieve this column back (we do not know the
725     # id anymore)
726
727     my $rsrc = $self->result_source;
728
729     # set in case there are more cascades combined with $rs->update => $rs_update_all overrides
730     local $rsrc->schema->{_ORDERED_INTERNAL_UPDATE} = 1;
731     my @pcols = $rsrc->primary_columns;
732     my $pos_is_pk = first { $_ eq $position_column } @pcols;
733     if (
734       $pos_is_pk
735         or
736       first { $_ eq $position_column } ( map { @$_ } (values %{{ $rsrc->unique_constraints }} ) )
737     ) {
738         my $cursor = $shift_rs->search (
739           {}, { order_by => { "-$ord", $position_column }, select => [$position_column, @pcols] }
740         )->cursor;
741         my $rs = $self->result_source->resultset;
742
743         my @all_data = $cursor->all;
744         while (my $data = shift @all_data) {
745           my $pos = shift @$data;
746           my $cond;
747           for my $i (0.. $#pcols) {
748             $cond->{$pcols[$i]} = $data->[$i];
749           }
750
751           $rs->find($cond)->update ({ $position_column => $pos + ( ($op eq '+') ? 1 : -1 ) });
752         }
753     }
754     else {
755         $shift_rs->update ({ $position_column => \ "$position_column $op 1" } );
756     }
757 }
758
759 =head1 PRIVATE METHODS
760
761 These methods are used internally.  You should never have the
762 need to use them.
763
764 =head2 _group_rs
765
766 This method returns a resultset containing all members of the row
767 group (including the row itself).
768
769 =cut
770 sub _group_rs {
771     my $self = shift;
772     return $self->result_source->resultset->search({$self->_grouping_clause()});
773 }
774
775 =head2 _siblings
776
777 Returns an unordered resultset of all objects in the same group
778 excluding the object you called this method on.
779
780 =cut
781 sub _siblings {
782     my $self = shift;
783     my $position_column = $self->position_column;
784     my $pos;
785     return defined ($pos = $self->get_column($position_column))
786         ? $self->_group_rs->search(
787             { $position_column => { '!=' => $pos } },
788           )
789         : $self->_group_rs
790     ;
791 }
792
793 =head2 _position
794
795   my $num_pos = $item->_position;
796
797 Returns the B<absolute numeric position> of the current object, with the
798 first object being at position 1, its sibling at position 2 and so on.
799
800 =cut
801 sub _position {
802     my $self = shift;
803     return $self->_position_from_value ($self->get_column ($self->position_column) );
804 }
805
806 =head2 _grouping_clause
807
808 This method returns one or more name=>value pairs for limiting a search
809 by the grouping column(s).  If the grouping column is not defined then
810 this will return an empty list.
811
812 =cut
813 sub _grouping_clause {
814     my( $self ) = @_;
815     return map {  $_ => $self->get_column($_)  } $self->_grouping_columns();
816 }
817
818 =head2 _get_grouping_columns
819
820 Returns a list of the column names used for grouping, regardless of whether
821 they were specified as an arrayref or a single string, and returns ()
822 if there is no grouping.
823
824 =cut
825 sub _grouping_columns {
826     my( $self ) = @_;
827     my $col = $self->grouping_column();
828     if (ref $col eq 'ARRAY') {
829         return @$col;
830     } elsif ($col) {
831         return ( $col );
832     } else {
833         return ();
834     }
835 }
836
837 =head2 _is_in_group
838
839     $item->_is_in_group( {user => 'fred', list => 'work'} )
840
841 Returns true if the object is in the group represented by hashref $other
842
843 =cut
844 sub _is_in_group {
845     my ($self, $other) = @_;
846     my $current = {$self->_grouping_clause};
847
848     no warnings qw/uninitialized/;
849
850     return 0 if (
851         join ("\x00", sort keys %$current)
852             ne
853         join ("\x00", sort keys %$other)
854     );
855     for my $key (keys %$current) {
856         return 0 if $current->{$key} ne $other->{$key};
857     }
858     return 1;
859 }
860
861 =head2 _ordered_internal_update
862
863 This is a short-circuited method, that is used internally by this
864 module to update positioning values in isolation (i.e. without
865 triggering any of the positioning integrity code).
866
867 Some day you might get confronted by datasets that have ambiguous
868 positioning data (e.g. duplicate position values within the same group,
869 in a table without unique constraints). When manually fixing such data
870 keep in mind that you can not invoke L<DBIx::Class::Row/update> like
871 you normally would, as it will get confused by the wrong data before
872 having a chance to update the ill-defined row. If you really know what
873 you are doing use this method which bypasses any hooks introduced by
874 this module.
875
876 =cut
877
878 sub _ordered_internal_update {
879     my $self = shift;
880     local $self->result_source->schema->{_ORDERED_INTERNAL_UPDATE} = 1;
881     return $self->update (@_);
882 }
883
884 1;
885
886 __END__
887
888 =head1 CAVEATS
889
890 =head2 Resultset Methods
891
892 Note that all Insert/Create/Delete overrides are happening on
893 L<DBIx::Class::Row> methods only. If you use the
894 L<DBIx::Class::ResultSet> versions of
895 L<update|DBIx::Class::ResultSet/update> or
896 L<delete|DBIx::Class::ResultSet/delete>, all logic present in this
897 module will be bypassed entirely (possibly resulting in a broken
898 order-tree). Instead always use the
899 L<update_all|DBIx::Class::ResultSet/update_all> and
900 L<delete_all|DBIx::Class::ResultSet/delete_all> methods, which will
901 invoke the corresponding L<row|DBIx::Class::Row> method on every
902 member of the given resultset.
903
904 =head2 Race Condition on Insert
905
906 If a position is not specified for an insert, a position
907 will be chosen based either on L</_initial_position_value> or
908 L</_next_position_value>, depending if there are already some
909 items in the current group. The space of time between the
910 necessary selects and insert introduces a race condition.
911 Having unique constraints on your position/group columns,
912 and using transactions (see L<DBIx::Class::Storage/txn_do>)
913 will prevent such race conditions going undetected.
914
915 =head2 Multiple Moves
916
917 Be careful when issuing move_* methods to multiple objects.  If
918 you've pre-loaded the objects then when you move one of the objects
919 the position of the other object will not reflect their new value
920 until you reload them from the database - see
921 L<DBIx::Class::Row/discard_changes>.
922
923 There are times when you will want to move objects as groups, such
924 as changing the parent of several objects at once - this directly
925 conflicts with this problem.  One solution is for us to write a
926 ResultSet class that supports a parent() method, for example.  Another
927 solution is to somehow automagically modify the objects that exist
928 in the current object's result set to have the new position value.
929
930 =head2 Default Values
931
932 Using a database defined default_value on one of your group columns
933 could result in the position not being assigned correctly.
934
935 =head1 AUTHOR
936
937  Original code framework
938    Aran Deltac <bluefeet@cpan.org>
939
940  Constraints support and code generalisation
941    Peter Rabbitson <ribasushi@cpan.org>
942
943 =head1 LICENSE
944
945 You may distribute this code under the same terms as Perl itself.
946