[Jifty-commit] jifty branch, master, updated. a96d64ac387814ae563705b58c8710e6634f9571

Jifty commits jifty-commit at lists.jifty.org
Tue Apr 6 16:21:04 EDT 2010


The branch, master has been updated
       via  a96d64ac387814ae563705b58c8710e6634f9571 (commit)
      from  7635d9bb4ba0f8904bbaa991073c6457b3331da8 (commit)

Summary of changes:
 lib/Jifty/Plugin/REST/Dispatcher.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit a96d64ac387814ae563705b58c8710e6634f9571
Author: Shawn M Moore <sartak at bestpractical.com>
Date:   Tue Apr 6 16:21:02 2010 -0400

    Revert "Do not forbid virtual columns from the REST dispatcher"
    
    This reverts commit 7635d9bb4ba0f8904bbaa991073c6457b3331da8.
    
    This could totally cause things to show up that shouldn't, such as
    foo_id for foo columns

diff --git a/lib/Jifty/Plugin/REST/Dispatcher.pm b/lib/Jifty/Plugin/REST/Dispatcher.pm
index c52b713..95f855d 100644
--- a/lib/Jifty/Plugin/REST/Dispatcher.pm
+++ b/lib/Jifty/Plugin/REST/Dispatcher.pm
@@ -524,7 +524,7 @@ qw( name
 
 sub valid_column {
     my ( $model, $column ) = @_;
-    return scalar grep { $_->name eq $column and not $_->private } $model->new->columns;
+    return scalar grep { $_->name eq $column and not $_->virtual and not $_->private } $model->new->columns;
 }
 
 =head2 list_model_columns

-----------------------------------------------------------------------


More information about the Jifty-commit mailing list