order_by fixes including enable_order_by and coerce_order_by

3 files changed, 21 insertions, 1 deletions

diff --git a/lib/ComponentUI/Controller/TestModel/Bar.pm b/lib/ComponentUI/Controller/TestModel/Bar.pm
index 0839506..4b592c0 100644
--- a/lib/ComponentUI/Controller/TestModel/Bar.pm
+++ b/lib/ComponentUI/Controller/TestModel/Bar.pm
@@ -8,7 +8,19 @@ __PACKAGE__->config(
   collection_name => 'Bar',
   action => {
     base => { Chained => '/base', PathPart => 'testmodel/bar' },
+    list => {
+      ViewPort => {
+        enable_order_by => [qw/name foo published_at/],
+        coerce_order_by => { foo => ['foo.last_name', 'foo.first_name'] },
+      }
+    }
   },
 );
 
+sub get_collection {
+  my ($self, $c) = @_;
+  my $collection = $self->next::method($c);
+  return $collection->where({}, { prefetch => 'foo' });
+}
+
 1;
diff --git a/lib/ComponentUI/Controller/TestModel/Baz.pm b/lib/ComponentUI/Controller/TestModel/Baz.pm
index f1d7c8b..2fece5a 100644
--- a/lib/ComponentUI/Controller/TestModel/Baz.pm
+++ b/lib/ComponentUI/Controller/TestModel/Baz.pm
@@ -6,7 +6,14 @@ use Reaction::Class;
 __PACKAGE__->config(
   model_name => 'TestModel',
   collection_name => 'Baz',
-  action => { base => { Chained => '/base', PathPart => 'testmodel/baz' } },
+  action => {
+    base => { Chained => '/base', PathPart => 'testmodel/baz' },
+    list => {
+      ViewPort => {
+        enable_order_by => [qw/id name/],
+      },
+    },
+  },
 );
 
 1;
diff --git a/lib/ComponentUI/Controller/TestModel/Foo.pm b/lib/ComponentUI/Controller/TestModel/Foo.pm
index 56839a2..917ea17 100644
--- a/lib/ComponentUI/Controller/TestModel/Foo.pm
+++ b/lib/ComponentUI/Controller/TestModel/Foo.pm
@@ -13,6 +13,7 @@ __PACKAGE__->config(
         action_prototypes => { delete_all => 'Delete all records' },
         excluded_fields => [qw/id/],
         action_order => [qw/delete_all create/],
+        enable_order_by => [qw/last_name/],
         Member => {
           action_order => [qw/view update delete/],
         },