Merge 'trunk' into 'DBIx-Class-current'
Matt S Trout [Tue, 21 Mar 2006 15:57:51 +0000 (15:57 +0000)]
r9049@obrien (orig r1275):  jguenther | 2006-03-21 12:03:55 +0000
changed Foo/Bar in docs to more meaningful names
r9050@obrien (orig r1276):  matthewt | 2006-03-21 14:10:01 +0000
Docs tweaks from bluefeet, correction to ResultSet POD update by jguenther
r9051@obrien (orig r1277):  matthewt | 2006-03-21 14:26:22 +0000
Corrected columns_info_for indent level - 2 space indent everyone please, you may not like it but the rest of the code is already like that
r9052@obrien (orig r1278):  matthewt | 2006-03-21 15:57:16 +0000
Tweaked columns_info_for test to get around SQLite inconsistencies

lib/DBIx/Class/Positioned.pm [new file with mode: 0644]
lib/DBIx/Class/Tree/AdjacencyList.pm [new file with mode: 0644]
t/helperrels/26positioned.t [new file with mode: 0644]
t/lib/DBICTest/Schema/Artist.pm
t/lib/sqlite.sql
t/run/04db.tl
t/run/145db2.tl
t/run/26positioned.tl [new file with mode: 0644]

diff --git a/lib/DBIx/Class/Positioned.pm b/lib/DBIx/Class/Positioned.pm
new file mode 100644 (file)
index 0000000..a8daf69
--- /dev/null
@@ -0,0 +1,348 @@
+# vim: ts=8:sw=4:sts=4:et
+package DBIx::Class::Positioned;
+use strict;
+use warnings;
+use base qw( DBIx::Class );
+
+=head1 NAME
+
+DBIx::Class::Positioned - Modify the position of objects in an ordered list.
+
+=head1 SYNOPSIS
+
+Create a table for your positionable data.
+
+  CREATE TABLE employees (
+    employee_id INTEGER PRIMARY KEY AUTOINCREMENT,
+    name TEXT NOT NULL,
+    position INTEGER NOT NULL
+  );
+
+In your Schema or DB class add Positioned to the top 
+of the component list.
+
+  __PACKAGE__->load_components(qw( Positioned ... ));
+
+Specify the column that stores the position number for 
+each row.
+
+  package My::Employee;
+  __PACKAGE__->position_column('position');
+
+Thats it, now you can change the position of your objects.
+
+  #!/use/bin/perl
+  use My::Employee;
+  
+  my $employee = My::Employee->create({ name=>'Matt S. Trout' });
+  
+  my $rs = $employee->siblings();
+  my @siblings = $employee->siblings();
+  
+  my $sibling;
+  $sibling = $employee->first_sibling();
+  $sibling = $employee->last_sibling();
+  $sibling = $employee->previous_sibling();
+  $sibling = $employee->next_sibling();
+  
+  $employee->move_previous();
+  $employee->move_next();
+  $employee->move_first();
+  $employee->move_last();
+  $employee->move_to( $position );
+
+=head1 DESCRIPTION
+
+This module provides a simple interface for modifying the position 
+of DBIx::Class objects.
+
+=head1 METHODS
+
+=head2 position_column
+
+  __PACKAGE__->position_column('position');
+
+Sets and retrieves the name of the column that stores the 
+positional value of each record.  Default to "position".
+
+=cut
+
+__PACKAGE__->mk_classdata( 'position_column' => 'position' );
+
+=head2 siblings
+
+  my $rs = $employee->siblings();
+  my @siblings = $employee->siblings();
+
+Returns either a result set or an array of all other objects 
+excluding the one you called it on.
+
+=cut
+
+sub siblings {
+    my( $self ) = @_;
+    my $position_column = $self->position_column;
+    my $rs = $self->search(
+        {
+            $position_column => { '!=' => $self->get_column($position_column) },
+            $self->_parent_clause(),
+        },
+        { order_by => $self->position_column },
+    );
+    return $rs->all() if (wantarray());
+    return $rs;
+}
+
+=head2 first_sibling
+
+  my $sibling = $employee->first_sibling();
+
+Returns the first sibling object.
+
+=cut
+
+sub first_sibling {
+    my( $self ) = @_;
+    return ($self->search(
+        { $self->_parent_clause() },
+        { rows=>1, order_by => $self->position_column },
+    )->all())[0];
+}
+
+=head2 last_sibling
+
+  my $sibling = $employee->last_sibling();
+
+Return the last sibling.
+
+=cut
+
+sub last_sibling {
+    my( $self ) = @_;
+    return ($self->search(
+        { $self->_parent_clause() },
+        { rows=>1, order_by => $self->position_column.' DESC' },
+    )->all())[0];
+}
+
+=head2 previous_sibling
+
+  my $sibling = $employee->previous_sibling();
+
+Returns the sibling that resides one position higher.  Undef 
+is returned if the current object is the first one.
+
+=cut
+
+sub previous_sibling {
+    my( $self ) = @_;
+    my $position_column = $self->position_column;
+    return ($self->search(
+        {
+            $position_column => { '<' => $self->get_column($position_column) },
+            $self->_parent_clause(),
+        },
+        { rows=>1, order_by => $position_column.' DESC' },
+    )->all())[0];
+}
+
+=head2 next_sibling
+
+  my $sibling = $employee->next_sibling();
+
+Returns the sibling that resides one position lower.  Undef 
+is returned if the current object is the last one.
+
+=cut
+
+sub next_sibling {
+    my( $self ) = @_;
+    my $position_column = $self->position_column;
+    return ($self->search(
+        {
+            $position_column => { '>' => $self->get_column($position_column) },
+            $self->_parent_clause(),
+        },
+        { rows=>1, order_by => $position_column },
+    )->all())[0];
+}
+
+=head2 move_previous
+
+  $employee->move_previous();
+
+Swaps position with the sibling on position previous in the list.  
+1 is returned on success, and 0 is returned if the objects is already 
+the first one.
+
+=cut
+
+sub move_previous {
+    my( $self ) = @_;
+    my $previous = $self->previous_sibling();
+    return undef if (!$previous);
+    my $position_column = $self->position_column;
+    my $self_position = $self->get_column( $position_column );
+    $self->set_column( $position_column, $previous->get_column($position_column) );
+    $previous->set_column( $position_column, $self_position );
+    $self->update();
+    $previous->update();
+    return 1;
+}
+
+=head2 move_next
+
+  $employee->move_next();
+
+Swaps position with the sibling in the next position.  1 is returned on 
+success, and 0 is returned if the object is already the last in the list.
+
+=cut
+
+sub move_next {
+    my( $self ) = @_;
+    my $next = $self->next_sibling();
+    return undef if (!$next);
+    my $position_column = $self->position_column;
+    my $self_position = $self->get_column( $position_column );
+    $self->set_column( $position_column, $next->get_column($position_column) );
+    $next->set_column( $position_column, $self_position );
+    $self->update();
+    $next->update();
+    return 1;
+}
+
+=head2 move_first
+
+  $employee->move_first();
+
+Moves the object to the first position.  1 is returned on 
+success, and 0 is returned if the object is already the first.
+
+=cut
+
+sub move_first {
+    my( $self ) = @_;
+    return $self->move_to( 1 );
+}
+
+=head2 move_last
+
+  $employee->move_last();
+
+Moves the object to the very last position.  1 is returned on 
+success, and 0 is returned if the object is already the last one.
+
+=cut
+
+sub move_last {
+    my( $self ) = @_;
+    my $count = $self->search({$self->_parent_clause()})->count();
+    return $self->move_to( $count );
+}
+
+=head2 move_to
+
+  $employee->move_to( $position );
+
+Moves the object to the specified position.  1 is returned on 
+success, and 0 is returned if the object is already at the 
+specified position.
+
+=cut
+
+sub move_to {
+    my( $self, $to_position ) = @_;
+    my $position_column = $self->position_column;
+    my $from_position = $self->get_column( $position_column );
+    return undef if ( $from_position==$to_position );
+    my $rs = $self->search({
+        -and => [
+            $position_column => { ($from_position>$to_position?'<':'>') => $from_position },
+            $position_column => { ($from_position>$to_position?'>=':'<=') => $to_position },
+        ],
+        $self->_parent_clause(),
+    });
+    my $op = ($from_position>$to_position) ? '+' : '-';
+    $rs->update({
+        $position_column => \"$position_column $op 1",
+    });
+    $self->set_column( $position_column => $to_position );
+    $self->update();
+    return 1;
+}
+
+=head2 insert
+
+Overrides the DBIC insert() method by providing a default 
+position number.  The default will be the number of rows in 
+the table +1, thus positioning the new record at the last position.
+
+=cut
+
+sub insert {
+    my $self = shift;
+    my $position_column = $self->position_column;
+    $self->set_column( $position_column => $self->search( {$self->_parent_clause()} )->count()+1 ) 
+        if (!$self->get_column($position_column));
+    $self->next::method( @_ );
+}
+
+=head2 delete
+
+Overrides the DBIC delete() method by first moving the object 
+to the last position, then deleting it, thus ensuring the 
+integrity of the positions.
+
+=cut
+
+sub delete {
+    my $self = shift;
+    $self->move_last;
+    $self->next::method( @_ );
+}
+
+=head1 PRIVATE METHODS
+
+These methods are used internally.  You should never have the 
+need to use them.
+
+=head2 _parent_clause
+
+  sub _parent_clause {
+    my( $self ) = @_;
+    return ( parent_id => $self->parent_id );
+  }
+
+This method is a placeholder for you, or another component, to 
+provide additional limits for all the various queries in this 
+module.  This allows for more than one positionable list within 
+the same table since any move_* method will adhere to the clause 
+that you specify.
+
+=cut
+
+sub _parent_clause {
+    return ();
+}
+
+1;
+__END__
+
+=head1 BUGS
+
+If a position is not specified for an insert than a position 
+will be chosen based on COUNT(*)+1.  But, it first selects the 
+count then inserts the record.  The space of time between select 
+and insert introduces a race condition.  To fix this we need the 
+ability to lock tables in DBIC.  I've added an entry in the TODO 
+about this.
+
+=head1 AUTHOR
+
+Aran Deltac <bluefeet@cpan.org>
+
+=head1 LICENSE
+
+You may distribute this code under the same terms as Perl itself.
+
diff --git a/lib/DBIx/Class/Tree/AdjacencyList.pm b/lib/DBIx/Class/Tree/AdjacencyList.pm
new file mode 100644 (file)
index 0000000..4e9f28e
--- /dev/null
@@ -0,0 +1,182 @@
+# vim: ts=8:sw=4:sts=4:et
+package DBIx::Class::Tree::AdjacencyList;
+use strict;
+use warnings;
+use base qw( DBIx::Class );
+
+=head1 NAME
+
+DBIx::Class::Tree::AdjacencyList - Manage a tree of data using the common adjacency list model.
+
+=head1 SYNOPSIS
+
+Create a table for your tree data.
+
+  CREATE TABLE employees (
+    employee_id INTEGER PRIMARY KEY AUTOINCREMENT,
+    parent_id INTEGER NOT NULL,
+    name TEXT NOT NULL
+  );
+
+In your Schema or DB class add Tree::AdjacencyList to the top 
+of the component list.
+
+  __PACKAGE__->load_components(qw( Tree::AdjacencyList ... ));
+  # If you want positionable data make sure this 
+  # module comes first, as in:
+  __PACKAGE__->load_components(qw( Tree::AdjacencyList Positioned ... ));
+
+Specify the column that contains the parent ID each row.
+
+  package My::Employee;
+  __PACKAGE__->parent_column('parent_id');
+
+Thats it, now you can modify and analyze the tree.
+
+  #!/use/bin/perl
+  use My::Employee;
+  
+  my $employee = My::Employee->create({ name=>'Matt S. Trout' });
+  
+  my $rs = $employee->children();
+  my @siblings = $employee->children();
+  
+  my $parent = $employee->parent();
+  $employee->parent( 7 );
+
+=head1 DESCRIPTION
+
+This module provides methods for working with adjacency lists.  The 
+adjacency list model is a very common way of representing a tree structure.  
+In this model each row in a table has a prent ID column that references the 
+primary key of another row in the same table.  Because of this the primary 
+key must only be one column and is usually some sort of integer.  The row 
+with a parent ID of 0 is the root row and is usually the parent of all 
+other rows.
+
+=head1 METHODS
+
+=head2 parent_column
+
+  __PACKAGE__->parent_column('parent_id');
+
+Declares the name of the column that contains the self-referential 
+ID which defines the parent row.  Defaults to "parent_id".
+
+=cut
+
+__PACKAGE__->mk_classdata( 'parent_column' => 'parent_id' );
+
+=head2 parent
+
+  my $parent = $employee->parent();
+  $employee->parent( $parent_obj );
+  $employee->parent( $parent_id );
+
+Retrieves the object's parent ID, or sets the object's 
+parent ID.  If setting the parent ID then 0 will be returned 
+if the object already has the specified parent, and 1 on 
+success.
+
+If you are using the Positioned component this 
+module will first move the object to the last position of 
+the list, change the parent ID, then move the object to the 
+last position of the new list.  This ensures the intergrity 
+of the positions.
+
+=cut
+
+sub parent {
+    my( $self, $new_parent ) = @_;
+    my $parent_column = $self->parent_column();
+    if ($new_parent) {
+        if (ref($new_parent)) {
+            $new_parent = $new_parent->id();
+        }
+        return 0 if ($new_parent == $self->get_column($parent_column));
+        my $positioned = $self->can('position_column');
+        $self->move_last if ($positioned);
+        $self->set_column( $parent_column => $new_parent );
+        if ($positioned) {
+            $self->set_column(
+                $self->position_column() => $self->search( {$self->_parent_clause()} )->count() + 1
+            );
+        }
+        $self->update();
+        return 1;
+    }
+    else {
+        return $self->find( $self->get_column( $parent_column ) );
+    }
+}
+
+=head2 children
+
+  my $children_rs = $employee->children();
+  my @children = $employee->children();
+
+Returns a list or record set, depending on context, of all 
+the objects one level below the current one.
+
+If you are using the Positioned component then this method 
+will return the children sorted by the position column.
+
+=cut
+
+sub children {
+    my( $self ) = @_;
+    my $rs = $self->search(
+        { $self->parent_column()=>$self->id() },
+        ( $self->can('position_column') ? {order_by=>$self->position_column()} : () )
+    );
+    return $rs->all() if (wantarray());
+    return $rs;
+}
+
+=head2 descendents
+
+Same as children.  Declared so that this module is 
+compatible with the Tree::NestedSet module.
+
+=cut
+
+#*descendants = \&children;
+
+=head1 PRIVATE METHODS
+
+These methods are used internally.  You should never have the 
+need to use them.
+
+=head2 _parent_clause
+
+This method is provided as an override of the method in 
+DBIC::Positioned.  This way Positioned and Tree::AdjacencyList 
+may be used together without conflict.  Make sure that in 
+you component list that you load Tree::AdjacencyList before you 
+load Positioned.
+
+This method assumes a parent ID of 0 if none is defined.  This 
+usually comes in to play if you are just createing the object 
+and it has not yet been assigned a parent.
+
+=cut
+
+sub _parent_clause {
+    my( $self ) = @_;
+    return (
+        $self->parent_column() =>
+        $self->get_column($self->parent_column()) || 0
+    );
+}
+
+1;
+__END__
+
+=head1 AUTHOR
+
+Aran Deltac <bluefeet@cpan.org>
+
+=head1 LICENSE
+
+You may distribute this code under the same terms as Perl itself.
+
diff --git a/t/helperrels/26positioned.t b/t/helperrels/26positioned.t
new file mode 100644 (file)
index 0000000..8e42bbd
--- /dev/null
@@ -0,0 +1,7 @@
+use Test::More;
+use lib qw(t/lib);
+use DBICTest;
+use DBICTest::HelperRels;
+
+require "t/run/26positioned.tl";
+run_tests(DBICTest->schema);
index f4c6706..996f53c 100644 (file)
@@ -3,7 +3,7 @@ package # hide from PAUSE
 
 use base 'DBIx::Class::Core';
 
-__PACKAGE__->load_components('PK::Auto');
+__PACKAGE__->load_components('Positioned','PK::Auto');
 
 DBICTest::Schema::Artist->table('artist');
 DBICTest::Schema::Artist->add_columns(
@@ -16,12 +16,17 @@ DBICTest::Schema::Artist->add_columns(
     size      => 100,
     is_nullable => 1,
   },
+  position => {
+    data_type => 'integer',
+  },
 );
 DBICTest::Schema::Artist->set_primary_key('artistid');
+__PACKAGE__->position_column('position');
 
 __PACKAGE__->mk_classdata('field_name_for', {
     artistid    => 'primary key',
     name        => 'artist name',
+    position    => 'list position',
 });
 
 1;
index f59aca8..e7646c4 100644 (file)
@@ -43,7 +43,8 @@ CREATE TABLE cd_to_producer (
 --
 CREATE TABLE artist (
   artistid INTEGER PRIMARY KEY NOT NULL,
-  name varchar(100)
+  name varchar,
+  position INTEGER
 );
 
 --
index daea4fe..f7d847c 100644 (file)
@@ -44,7 +44,12 @@ my $test_type_info = {
     'name' => {
         'data_type' => 'varchar',
         'is_nullable' => 0,
-    }
+    },
+    'position' => {
+        'data_type' => 'INTEGER',
+        'is_nullable' => 0,
+        'size' => undef,
+    },
 };
 is_deeply($type_info, $test_type_info, 'columns_info_for - column data types');
 
index aa721b1..31e3461 100644 (file)
@@ -14,10 +14,7 @@ DBICTest::Schema->compose_connection('DB2Test' => $dsn, $user, $pass);
 
 my $dbh = DB2Test->schema->storage->dbh;
 
-{
-    local $SIG{__WARN__} = sub {};
-    $dbh->do("DROP TABLE artist;");
-}
+$dbh->do("DROP TABLE artist", { RaiseError => 0, PrintError => 0 });
 
 $dbh->do("CREATE TABLE artist (artistid INTEGER GENERATED BY DEFAULT AS IDENTITY (START WITH 1, INCREMENT BY 1), name VARCHAR(255), charfield CHAR(10));");
 
diff --git a/t/run/26positioned.tl b/t/run/26positioned.tl
new file mode 100644 (file)
index 0000000..997f970
--- /dev/null
@@ -0,0 +1,63 @@
+# vim: filetype=perl
+
+sub run_tests {
+
+    my $schema = shift;
+    my $artists = $schema->resultset("Artist");
+
+    $artists->delete();
+    $artists->create({ artistid=>1, name=>"Joe" });
+    $artists->create({ artistid=>2, name=>"Bob" });
+    $artists->create({ artistid=>3, name=>"Ted" });
+    $artists->create({ artistid=>4, name=>"Ned" });
+    $artists->create({ artistid=>5, name=>"Don" });
+
+    $artists = $artists->search(undef,{order_by=>'position'});
+
+    plan tests => 230;
+
+    check_positions($schema);
+
+    my $artist;
+
+    foreach my $position (1..5) {
+
+        $artist = $artists->find({ position=>$position });
+        $artist->move_previous();
+        check_positions($schema);
+
+        $artist = $artists->find({ position=>$position });
+        $artist->move_next();
+        check_positions($schema);
+
+        $artist = $artists->find({ position=>$position });
+        $artist->move_first();
+        check_positions($schema);
+
+        $artist = $artists->find({ position=>$position });
+        $artist->move_last();
+        check_positions($schema);
+
+        foreach my $to_position (1..5) {
+
+            $artist = $artists->find({ position=>$position });
+            $artist->move_to($to_position);
+            check_positions($schema);
+
+        }
+
+    }
+
+}
+
+sub check_positions {
+    my $schema = shift;
+    my $artists = $schema->resultset("Artist")->search(undef,{order_by=>'position'});
+    my $expected_position = 0;
+    while (my $artist = $artists->next()) {
+        $expected_position ++;
+        ok( ($artist->position()==$expected_position), 'default positions set as expected' );
+    }
+}
+
+1;