[jifty-devel] [Jifty-commit] r7674 - jifty/trunk/lib/Jifty/View/Declare

[Jesse Vincent]

Yves, can you tell me a bit more about this? It seems like it could
break the display of immutable columns, though I may be
misunderstanding.

Best,
Jesse


On Wed, Nov 25, 2009 at 06:40:33AM -0500, Jifty commits wrote:
> Author: yves
> Date: Wed Nov 25 06:40:31 2009
> New Revision: 7674
> 
> Modified:
>    jifty/trunk/lib/Jifty/View/Declare/CRUD.pm
> 
> Log:
> use update fields in Crud view, 
> we have the same sort order than in create fragment and no more double columns for refers_to fields
> 
> 
> Modified: jifty/trunk/lib/Jifty/View/Declare/CRUD.pm
> ==============================================================================
> --- jifty/trunk/lib/Jifty/View/Declare/CRUD.pm	(original)
> +++ jifty/trunk/lib/Jifty/View/Declare/CRUD.pm	Wed Nov 25 06:40:31 2009
> @@ -443,7 +443,7 @@
>          moniker => "update-" . Jifty->web->serial,
>      );
>  
> -    my @fields = $self->display_columns;
> +    my @fields = $self->display_columns($update);
>      for my $field (@fields) {
>          div { { class is 'crud-field view-argument-'.$field};
>              $self->render_field(
> @@ -640,10 +640,11 @@
>      my $sort_by = shift;
>      my $order = shift;
>      my $record_class = $self->record_class;
> +    my $update = $record_class->as_update_action();
>  
>      div {
>          { class is "crud-column-headers" };
> -        for my $argument ($self->display_columns) {
> +        for my $argument ($self->display_columns($update)) {
>              div {
>                  { class is 'crud-column-header' };
>                  ul { attr { class => 'crud-sort-menu', style => 'display:none;' };
> _______________________________________________
> Jifty-commit mailing list
> Jifty-commit at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-commit
> 

--