my $today = DateTime->today_local;
my $date_xtor = sub {
- $_[0]->can('transdate_as_date') ? $_[0]->transdate_as_date
- : $_[0]->can('itime_as_date') ? $_[0]->itime_as_date
+ $_[0]->can('transdate_as_date') ? $_[0]->transdate
+ : $_[0]->can('itime_as_date') ? $_[0]->itime->clone->truncate(to => 'day')
: $today;
};
my $date_comparator = sub {
SL::DB::Helper::LinkedRecords - Mixin for retrieving linked records via the table C<record_links>
+SYNOPSIS
+
+ # In SL::DB::<Object>
+ use SL::DB::Helper::LinkedRecords;
+
+ # later in consumer code
+ # retrieve all links
+ my @linked_objects = $order->linked_records(
+ direction => 'both',
+ );
+
+ # only links to Invoices
+ my @linked_objects = $order->linked_records(
+ direction => 'to',
+ to => 'Invoice',
+ );
+
+ # more than one target
+ my @linked_objects = $order->linked_records(
+ direction => 'to',
+ to => [ 'Invoice', 'Order' ],
+ );
+
+ # more than one direction
+ my @linked_objects = $order->linked_records(
+ direction => 'both',
+ both => 'Invoice',
+ );
+
+ # more than one direction and different targets
+ my @linked_objects = $order->linked_records(
+ direction => 'both',
+ to => 'Invoice',
+ from => 'Order',
+ );
+
+ # transitive over known classes
+ my @linked_objects = $order->linked_records(
+ direction => 'to',
+ to => 'Invoice',
+ via => 'DeliveryOrder',
+ );
+
+ # add a new link
+ $order->link_to_record($invoice);
+ $order->link_to_record($purchase_order, bidirectional => 1);
+
+
=head1 FUNCTIONS
=over 4
=item C<linked_records %params>
-Retrieves records linked from or to C<$self> via the table
-C<record_links>. The mandatory parameter C<direction> (either C<from>,
-C<to> or C<both>) determines whether the function retrieves records
-that link to C<$self> (for C<direction> = C<to>) or that are linked
-from C<$self> (for C<direction> = C<from>). For C<direction = both>
-all records linked from or to C<$self> are returned.
-
-The optional parameter C<from> or C<to> (same as C<direction>)
-contains the package names of Rose models for table limitation (the
-prefix C<SL::DB::> is optional). It can be a single model name as a
-single scalar or multiple model names in an array reference in which
-case all links matching any of the model names will be returned.
-
-The optional parameter C<via> can be used to retrieve all documents
-that may have intermediate documents inbetween. It is an array
-reference of Rose package names for the models that may be
-intermediate link targets. One example is retrieving all invoices for
-a given quotation no matter whether or not orders and delivery orders
-have been created. If C<via> is given then C<from> or C<to> (depending
-on C<direction>) must be given as well, and it must then not be an
+Retrieves records linked from or to C<$self> via the table C<record_links>. The
+mandatory parameter C<direction> (either C<from>, C<to> or C<both>) determines
+whether the function retrieves records that link to C<$self> (for C<direction>
+= C<to>) or that are linked from C<$self> (for C<direction> = C<from>). For
+C<direction = both> all records linked from or to C<$self> are returned.
+
+The optional parameter C<from> or C<to> (same as C<direction>) contains the
+package names of Rose models for table limitation (the prefix C<SL::DB::> is
+optional). It can be a single model name as a single scalar or multiple model
+names in an array reference in which case all links matching any of the model
+names will be returned.
+
+The optional parameter C<via> can be used to retrieve all documents that may
+have intermediate documents inbetween. It is an array reference of Rose package
+names for the models that may be intermediate link targets. One example is
+retrieving all invoices for a given quotation no matter whether or not orders
+and delivery orders have been created. If C<via> is given then C<from> or C<to>
+(depending on C<direction>) must be given as well, and it must then not be an
array reference.
Examples:
If you only need invoices created directly from an order C<$order> (no
delivery orders inbetween) then the call could look like this:
- my $invoices = $order->linked_records(direction => 'to',
- to => 'Invoice');
+ my $invoices = $order->linked_records(
+ direction => 'to',
+ to => 'Invoice',
+ );
Retrieving all invoices from a quotation no matter whether or not
orders or delivery orders where created:
- my $invoices = $quotation->linked_records(direction => 'to',
- to => 'Invoice',
- via => [ 'Order', 'DeliveryOrder' ]);
+ my $invoices = $quotation->linked_records(
+ direction => 'to',
+ to => 'Invoice',
+ via => [ 'Order', 'DeliveryOrder' ],
+ );
The optional parameter C<query> can be used to limit the records
returned. The following call limits the earlier example to invoices
created today:
- my $invoices = $order->linked_records(direction => 'to',
- to => 'Invoice',
- query => [ transdate => DateTime->today_local ]);
+ my $invoices = $order->linked_records(
+ direction => 'to',
+ to => 'Invoice',
+ query => [ transdate => DateTime->today_local ],
+ );
The optional parameters C<$params{sort_by}> and C<$params{sort_dir}>
can be used in order to sort the result. If C<$params{sort_by}> is
created with the roles of C<from> and C<to> reversed. This link will
also only be created if it doesn't exist already.
-In scalar contenxt returns either the existing link or the newly
+In scalar context returns either the existing link or the newly
created one as an instance of C<SL::DB::RecordLink>. In array context
it returns an array of links (one entry if C<$params{bidirectional}>
is falsish and two entries if it is trueish).
=item * C<date>
-Sort by the date the record was created or applies to.
+Sort by the transdate of the record was created or applies to.
-=back
+Note: If the latter has a default setting it will always mask the creation time.
-Returns a hash reference.
+=back
-Can be called both as a class or as an instance function.
+Returns an array reference.
-This function is not exported.
+Can only be called both as a class function since it is noe exported.
=back
--- /dev/null
+use Test::More;
+
+use strict;
+
+use lib 't';
+use utf8;
+
+use Carp;
+use Data::Dumper;
+use Support::TestSetup;
+use Test::Exception;
+
+use SL::DB::Buchungsgruppe;
+use SL::DB::Currency;
+use SL::DB::Customer;
+use SL::DB::Employee;
+use SL::DB::Invoice;
+use SL::DB::Order;
+use SL::DB::DeliveryOrder;
+use SL::DB::Part;
+use SL::DB::Unit;
+
+my ($customer, $currency_id, $buchungsgruppe, $employee, $vendor);
+my ($link, $links, $o1, $o2, $d, $i);
+
+sub reset_state {
+ my %params = @_;
+
+ $params{$_} ||= {} for qw(buchungsgruppe unit customer part tax);
+
+ SL::DB::Manager::DeliveryOrder->delete_all(all => 1);
+ SL::DB::Manager::Order->delete_all(all => 1);
+ SL::DB::Manager::Invoice->delete_all(all => 1);
+ SL::DB::Manager::Customer->delete_all(all => 1);
+ SL::DB::Manager::Vendor->delete_all(all => 1);
+
+ $buchungsgruppe = SL::DB::Manager::Buchungsgruppe->find_by(description => 'Standard 19%', %{ $params{buchungsgruppe} }) || croak "No accounting group";
+ $employee = SL::DB::Manager::Employee->current || croak "No employee";
+
+ $currency_id = $::instance_conf->get_currency_id;
+
+ $customer = SL::DB::Customer->new(
+ name => 'Test Customer',
+ currency_id => $currency_id,
+ %{ $params{customer} }
+ )->save;
+
+ $vendor = SL::DB::Vendor->new(
+ name => 'Test Vendor',
+ currency_id => $currency_id,
+ %{ $params{vendor} }
+ )->save;
+}
+
+sub new_order {
+ my %params = @_;
+
+ return SL::DB::Order->new(
+ customer_id => $customer->id,
+ currency_id => $currency_id,
+ employee_id => $employee->id,
+ salesman_id => $employee->id,
+ taxzone_id => 0,
+ quotation => 0,
+ %params,
+ )->save;
+}
+
+sub new_delivery_order {
+ my %params = @_;
+
+ return SL::DB::DeliveryOrder->new(
+ customer_id => $customer->id,
+ currency_id => $currency_id,
+ employee_id => $employee->id,
+ salesman_id => $employee->id,
+ taxzone_id => 0,
+ %params,
+ )->save;
+}
+
+sub new_invoice {
+ my %params = @_;
+
+ return SL::DB::Invoice->new(
+ customer_id => $customer->id,
+ currency_id => $currency_id,
+ employee_id => $employee->id,
+ salesman_id => $employee->id,
+ gldate => DateTime->today_local->to_kivitendo,
+ taxzone_id => 0,
+ invoice => 1,
+ type => 'invoice',
+ %params,
+ )->save;
+}
+
+Support::TestSetup::login();
+
+reset_state();
+
+
+$o1 = new_order();
+$i = new_invoice();
+
+$link = $o1->link_to_record($i);
+
+# try to add a link
+is ref $link, 'SL::DB::RecordLink', 'link_to_record returns new link';
+is $link->from_table, 'oe', 'from_table';
+is $link->from_id, $o1->id, 'from_id';
+is $link->to_table, 'ar', 'to_table';
+is $link->to_id, $i->id, 'to_id';
+
+# retrieve link
+$links = $o1->linked_records(direction => 'to', to => 'Invoice');
+is $links->[0]->id, $i->id, 'direct retrieve 1';
+
+$links = $o1->linked_records(direction => 'to', to => 'SL::DB::Invoice');
+is $links->[0]->id, $i->id, 'direct retrieve 2 (with SL::DB::)';
+
+$links = $o1->linked_records(direction => 'to', to => [ 'Invoice', 'Order' ]);
+is $links->[0]->id, $i->id, 'direct retrieve 3 (array target)';
+
+$links = $o1->linked_records(direction => 'both', both => 'Invoice');
+is $links->[0]->id, $i->id, 'direct retrieve 4 (direction both)';
+
+$links = $i->linked_records(direction => 'from', from => 'Order');
+is $links->[0]->id, $o1->id, 'direct retrieve 4 (direction from)';
+
+# what happens if we delete a linked record?
+$o1->delete;
+
+$links = $i->linked_records(direction => 'from', from => 'Order');
+is @$links, 0, 'no dangling link after delete';
+
+# can we distinguish between types?
+$o1 = new_order(quotation => 1);
+$o2 = new_order();
+$o1->link_to_record($o2);
+
+$links = $o2->linked_records(direction => 'from', from => 'Order', query => [ quotation => 1 ]);
+is $links->[0]->id, $o1->id, 'query restricted retrieve 1';
+
+$links = $o2->linked_records(direction => 'from', from => 'Order', query => [ quotation => 0 ]);
+is @$links, 0, 'query restricted retrieve 2';
+
+# try bidirectional linking
+$o1 = new_order();
+$o2 = new_order();
+$o1->link_to_record($o2, bidirectional => 1);
+
+$links = $o1->linked_records(direction => 'to', to => 'Order');
+is $links->[0]->id, $o2->id, 'bidi 1';
+$links = $o1->linked_records(direction => 'from', from => 'Order');
+is $links->[0]->id, $o2->id, 'bidi 2';
+$links = $o1->linked_records(direction => 'both', both => 'Order');
+is $links->[0]->id, $o2->id, 'bidi 3';
+
+# funky stuff with both
+#
+$d = new_delivery_order();
+$i = new_invoice();
+
+$o2->link_to_record($d);
+$d->link_to_record($i);
+
+
+$links = $d->linked_records(direction => 'both', to => 'Invoice', from => 'Order', sort_by => 'customer_id', sort_dir => 1);
+is $links->[0]->id, $o2->id, 'both with different from/to 1';
+is $links->[1]->id, $i->id, 'both with different from/to 2';
+
+# what happens if we double link?
+#
+$o2->link_to_record($d);
+
+$links = $o2->linked_records(direction => 'to', to => 'DeliveryOrder');
+is @$links, 1, 'double link is only added once 1';
+
+$d->link_to_record($o2, bidirectional => 1);
+
+$links = $o2->linked_records(direction => 'to', to => 'DeliveryOrder');
+is @$links, 1, 'double link is only added once 2';
+
+# doc states that to/from ae optional. test that
+$links = $o2->linked_records(direction => 'both');
+is @$links, 2, 'links without from/to get all';
+
+# doc says there will be special values set... lets see
+$links = $o1->linked_records(direction => 'to', to => 'Order');
+is $links->[0]->{_record_link_direction}, 'to', '_record_link_direction to';
+is $links->[0]->{_record_link}->to_id, $o2->id, '_record_link to';
+
+$links = $o1->linked_records(direction => 'from', from => 'Order');
+is $links->[0]->{_record_link_direction}, 'from', '_record_link_direction from';
+is $links->[0]->{_record_link}->to_id, $o1->id, '_record_link from';
+
+# check if bidi returns an array of links
+{ local $TODO = 'does not work as advertised';
+my @links = $d->link_to_record($o2, bidirectional => 1);
+is @links, 2, 'bidi returns array of links in array context';
+}
+
+# via
+$links = $o2->linked_records(direction => 'to', to => 'Invoice', via => 'DeliveryOrder');
+is $links->[0]->id, $i->id, 'simple case via links (string)';
+
+$links = $o2->linked_records(direction => 'to', to => 'Invoice', via => [ 'DeliveryOrder' ]);
+is $links->[0]->id, $i->id, 'simple case via links (arrayref)';
+
+$links = $o1->linked_records(direction => 'to', to => 'Invoice', via => [ 'Order', 'DeliveryOrder' ]);
+is $links->[0]->id, $i->id, 'simple case via links (2 hops)';
+
+# multiple links in the same direction from one object
+$o1->link_to_record($d);
+$links = $o2->linked_records(direction => 'to', to => 'Invoice', via => 'DeliveryOrder');
+is $links->[0]->id, $i->id, 'simple case via links (string)';
+
+# at this point the structure is:
+#
+# o1 <--> o2 ---> d ---> i
+# \____________,^
+#
+
+# o1 must have 2 linked records now:
+$links = $o1->linked_records(direction => 'to');
+is @$links, 2, 'more than one link';
+
+# as a special funny case, o1 via Order, Order will now yield o2, because it bounces back over itself
+{ local $TODO = 'no idea if this is desired';
+$links = $o2->linked_records(direction => 'to', to => 'Order', via => [ 'Order', 'Order' ]);
+is @$links, 2, 'via links with bidirectional hop over starting object';
+}
+
+# for sorting, get all don't bother with the links, we'll just take our records
+my @records = ($o2, $i, $o1, $d);
+my $sorted;
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('type', 1, @records);
+is_deeply $sorted, [$o1, $o2, $d, $i], 'sorting by type';
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('type', 0, @records);
+is_deeply $sorted, [$i, $d, $o2, $o1], 'sorting by type desc';
+
+$d->donumber(1);
+$o1->ordnumber(2);
+$i->invnumber(3);
+$o2->ordnumber(4);
+
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('number', 1, @records);
+is_deeply $sorted, [$d, $o1, $i, $o2], 'sorting by number';
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('number', 0, @records);
+is_deeply $sorted, [$o2, $i, $o1, $d], 'sorting by number desc';
+
+# again with natural sorting
+$d->donumber("a1");
+$o1->ordnumber("a3");
+$i->invnumber("a7");
+$o2->ordnumber("a10");
+
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('number', 1, @records);
+is_deeply $sorted, [$d, $o1, $i, $o2], 'sorting naturally by number';
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('number', 0, @records);
+is_deeply $sorted, [$o2, $i, $o1, $d], 'sorting naturally by number desc';
+
+$o2->transdate(DateTime->new(year => 2010, month => 3, day => 1));
+$i->transdate(DateTime->new(year => 2014, month => 3, day => 19));
+$o1->transdate(DateTime->new(year => 2014, month => 5, day => 1));
+$d->transdate(DateTime->new(year => 2014, month => 5, day => 2));
+
+# transdate should be used before itime
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('date', 1, @records);
+is_deeply $sorted, [$o2, $i, $o1, $d], 'sorting by transdate';
+$sorted = SL::DB::Helper::LinkedRecords->sort_linked_records('date', 0, @records);
+is_deeply $sorted, [$d, $o1, $i, $o2], 'sorting by transdate desc';
+
+done_testing();
+
+1;