cleanup all_filters name generation
[MojoFacets.git] / lib / MojoFacets / Data.pm
index b8d7547..8a52a85 100644 (file)
@@ -39,11 +39,12 @@ sub index {
                        $file =~ s/$data_dir\/*//;
                        push @files, $file;
                } else {
-                       warn "IGNORE: $file\n";
+                       #warn "IGNORE: $file\n";
                }
        }, $data_dir);
 
        @files = sort { lc $a cmp lc $b } @files;
+       unshift @files, grep { defined $loaded->{$_}->{generated} } keys %$loaded;
        my $size;
        $size->{$_} = -s "$data_dir/$_" foreach @files;
 
@@ -130,7 +131,7 @@ sub __stats {
                                }
                                if ( $unique ) {
                                        $stats->{$n}->{unique} = 1;
-                                       warn "# $n unique ",dump( $unique );
+                                       #warn "# $n unique ",dump( $unique );
                                }
                        }
                }
@@ -157,6 +158,8 @@ sub stats {
 sub _load_path {
        my ( $self, $path ) = @_;
 
+       return if defined $loaded->{$path}->{'generated'};
+
        my $full_path = $self->app->home->rel_file( 'data/' . $path );
        die "$full_path $!" unless -r $full_path;
 
@@ -217,16 +220,23 @@ sub _load_path {
 sub load {
        my $self = shift;
 
+       my $path = $self->param('path') || $self->redirect_to( '/data/index' );
+
        my @paths = $self->param('paths');
        warn "# paths ", dump @paths;
+
+       foreach my $p ( keys %$loaded ) {
+               next if grep { /^\Q$p\E$/ } @paths;
+               warn "remove $p from memory\n";
+               delete $loaded->{$p};
+       }
+
        $self->_load_path( $_ ) foreach @paths;
 
-       my $path = $self->param('path') || $self->redirect_to( '/data/index' );
        warn "# path $path\n";
        $self->_load_path( $path );
 
        $self->session( 'path' => $path );
-       $self->session( 'modified' => $loaded->{$path}->{modified} );
 
        my $redirect_to = '/data/items';
 
@@ -254,22 +264,32 @@ sub _loaded {
        my $path = $self->session('path') || $self->param('path');
        $self->redirect_to('/data/index') unless $path;
 
-       if ( $loaded->{$path}->{modified} > 1 ) {
-               warn "rebuild stats for $path forced by modified\n";
-               $loaded->{$path}->{stats} = __stats( $loaded->{$path}->{data}->{items} );
-               $loaded->{$path}->{modified} = 1;
+       if ( defined $loaded->{$path}->{modified} && $loaded->{$path}->{modified} > 1 ) {
+               my $caller = (caller(1))[3];
+               if ( $caller =~ m/::edit/ ) {
+                       warn "rebuild stats for $path ignored caller $caller\n";
+               } else {
+                       warn "rebuild stats for $path FORCED by modified caller $caller\n";
+                       $loaded->{$path}->{stats} = __stats( $loaded->{$path}->{data}->{items} );
+                       $loaded->{$path}->{modified} = 1;
+               }
        }
 
        if ( ! defined $loaded->{$path}->{$name} ) {
                warn "$path $name isn't loaded\n";
                $self->_load_path( $path );
-               $self->redirect_to('/data/index')
-                       unless defined $loaded->{$path}->{$name};
                if ( ! defined $loaded->{$path}->{stats} ) {
                        warn "rebuild stats for $path\n";
                        $loaded->{$path}->{stats} = __stats( $loaded->{$path}->{data}->{items} );
                }
+               if ( ! defined $loaded->{$path}->{$name} ) {
+                       warn "MISSING $name for $path\n";
+                       $self->redirect_to('/data/index')
+               }
        }
+
+       $self->session( 'modified' => $loaded->{$path}->{modified} );
+
        return $loaded->{$path}->{$name};
 }
 
@@ -290,7 +310,11 @@ sub _permanent_path {
 
 sub _export_path {
        my $self = shift;
-       my $path = $self->_param_or_session('path') || die "no path";
+       my $path = $self->_param_or_session('path');
+       if ( ! $path ) {
+               warn "no path in param or session";
+               return;
+       }
        my $dir = $self->app->home->rel_dir('public') . "/export/$path";
        mkpath $dir unless -e $dir;
        $dir . '/' . join('.', @_);
@@ -359,7 +383,7 @@ sub _param_scalar {
                $self->session($name => $scalar);
        }
 
-       warn "# _perm_scalar $name ",dump $scalar;
+       warn "# _param_scalar $name ",dump $scalar;
        return $scalar;
 }
 
@@ -410,7 +434,9 @@ sub _filter_on_data {
        foreach my $i ( 0 .. $#$items ) {
 
                if ( defined $items->[$i]->{$name} ) {
-                       foreach my $v ( @{ $items->[$i]->{$name} } ) {
+                       my $row = $items->[$i]->{$name};
+                       $row = [ $row ] unless ref $row eq 'ARRAY'; # FIXME probably wrong place
+                       foreach my $v ( @$row ) {
                                if ( defined $filter_hash->{ $v } ) {
                                        $filtered_items->{$i}++;
                                }
@@ -426,58 +452,19 @@ sub _filter_on_data {
        warn "filter $name with ", scalar keys %$filtered_items, " items created\n";
 }
 
-sub _data_items {
-       my ( $self, $all ) = @_;
-       my $data = $self->_loaded( 'data' );
-
-       return @{ $data->{items} } if $all == 1;
-
-       my $filters = $self->_current_filters;
-       my $filter_value;
-       foreach my $f ( keys %$filters ) {
-               foreach my $n ( @{ $filters->{$f} } ) {
-                       $filter_value->{$f}->{$n} = 1;
-               }
-       }
-       my @items = @{ $data->{items} };
-       @items = grep {
-               my $i = $_;
-               my $pass = 1;
-               foreach my $n ( keys %$filter_value ) {
-                       if ( ! exists $i->{$n} ) {
-                               if ( defined $filter_value->{$n}->{_missing} ) {
-                                       $pass = 1;
-                                       next;
-                               } else {
-                                       $pass = 0;
-                                       last;
-                               }
-                       }
-                       # and match any of values in element
-                       my $have_values = 0;
-                       foreach my $v ( @{ $i->{$n} } ) { # FIXME not array?
-                               $have_values ||= 1 if defined $filter_value->{$n}->{$v};
-                       }
-                       if ( ! $have_values ) {
-                               $pass = 0;
-                               last;
-                       }
-               }
-               $pass;
-       } @items if $filter_value;
-       return @items;
-}
-
 
 sub _current_filters {
        my $self = shift;
        my $current_filters;
+       my $columns = $self->_loaded('header');
+       if ( my $sc = $self->session('columns') ) {
+               $columns = $sc;
+       }
+
        $current_filters->{ $_ } = $filters->{ $_ }
-               foreach (
-                       grep { defined $filters->{ $_ } }
-                       @{ $self->_loaded('header') }
-               );
-       #warn "# current_filters ",dump($current_filters);
+               foreach ( grep { defined $filters->{ $_ } } @$columns )
+       ;
+       warn "# _current_filters ",dump($columns);
        return $current_filters;
 }
 
@@ -517,11 +504,20 @@ sub _data_sorted_by {
 }
 
 
+sub __all_filters {
+       my $order = pop @_;
+       join(',', sort(@_), 'order', $order);
+}
+
 sub items {
        my $self = shift;
 
+       if ( my $show = $self->param('id') ) {
+               $self->param('show', $show);
+               warn "show $show\n";
+       }
+
        my $path = $self->session('path');
-       $self->redirect_to('/data/index') unless defined $loaded->{ $path };
 
        my @columns = $self->_param_array('columns');
        $self->redirect_to('/data/columns') unless @columns;
@@ -534,6 +530,10 @@ sub items {
        # fix offset when changing limit
        $offset = int( $offset / $limit ) * $limit;
 
+       if ( ! grep { /^\Q$order\E$/ } @columns ) {
+               $order = $columns[0];
+               $self->session( order => $order );
+       }
        my $sorted = $self->_data_sorted_by( $order );
 
        my @filter_names;
@@ -552,7 +552,7 @@ sub items {
                }
        }
 
-       my $all_filters = join(' ',sort @filter_names,'order:',$order);
+       my $all_filters = __all_filters( @filter_names,$order );
 
 #      warn "# all_filters $all_filters ", dump( $loaded->{$path}->{filtered}->{$all_filters} );
 
@@ -588,21 +588,123 @@ sub items {
 
        warn "all_filters $all_filters produced ", $#$filtered + 1, " items\n" if $filtered;
 
-       my $sorted_items;
        my $data = $self->_loaded('data');
+
+       my $code = $self->_param_scalar('code','');
+       $code =~ s{[\r\n]+$}{}s;
+
+       my $commit = $self->param('commit');
+       my $test = $self->param('test');
+
+       my $cols_changed;
+
+       if ( $code && ( $test || $commit ) ) {
+               # XXX find columns used in code snippet and show them to user
+               foreach my $column ( $code =~ m/\$row->{(.+?)}/g ) {
+                       if ( $column =~ s/^(['"])// ) {
+                               $column =~ s/$1$//;
+                       }
+                       $cols_changed->{$column}++;
+                       next if grep { /$column/ } @columns;
+                       $cols_changed->{$column}++;
+                       unshift @columns, $column;
+                       if ( $commit ) {
+                               $self->session('columns', [ @columns ]);
+                               $loaded->{$path}->{columns} = [ @columns ];
+                               __path_modified( $path, 2 );
+                       }
+               }
+       }
+
+       my $code_path = $self->app->home->rel_dir('public') . "/code";
+       if ( $commit ) {
+               warn "# commit on ", $#$filtered + 1, " items:\n$code\n";
+               my $out;
+               foreach ( 0 .. $#$filtered ) {
+                       my $i = $filtered->[$_];
+                       my $row = $data->{items}->[$i];
+                       eval $code;
+               }
+               if ( my $description = $self->param('code_description') ) {
+                       my $depends = $self->param('code_depends') || die "no code_depends?";
+                       my $path = "$code_path/$depends.$description.pl";
+                       if ( -e $path && ! $self->param('overwrite') ) {
+                               warn "# code $path not saved\n";
+                       } else {
+                               write_file $path, $code;
+                               warn "code $path ", -s $path, " bytes saved\n";
+                       }
+               }
+               $code = '';
+               if ( $out ) {
+                       my $commit_dataset = join('.'
+                               , $self->param('code_depends')
+                               , $self->param('code_description')
+                       );
+                       my $key = $self->param('code_depends');
+                       $key =~ s/,.+$//;
+                       $key ||= 'key';
+                       my $items;
+                       foreach my $n ( keys %$out ) {
+                               my $i = { $key => [ $n ] };
+                               $i->{$_} = [ $out->{$n}->{$_} ] foreach keys %{ $out->{$n} };
+                               push @$items, $i;
+                       };
+                       undef $out;
+                       my $stats = __stats( $items );
+                       my @columns = grep { ! m/^\Q$key\E$/ } sort keys %$stats;
+                       unshift @columns, $key;
+
+                       $loaded->{$commit_dataset} = {
+                               header => [ @columns ],
+                               columns => [ @columns ],
+                               mtime => time(),
+                               data => { items => $items },
+                               stats => $stats,
+                               generated => 1,
+                       };
+                       warn "# loaded out ", dump( $loaded->{$commit_dataset} );
+                       $self->session('path', $commit_dataset);
+                       $self->session('columns', [ @columns ]);
+                       $self->session('order', $key);
+                       $self->redirect_to('/data/items');
+               }
+       }
+
+       my $sorted_items;
        my $from_end = $sort eq 'd' ? $#$filtered : 0;
+       my $out;
        foreach ( 0 .. $limit ) {
                my $i = $_ + $offset;
                last unless defined $filtered->[$i];
                $i = $from_end - $i if $from_end;
                my $id = $filtered->[$i];
-               push @$sorted_items,
-               my $item = $data->{items}->[ $id ];
-               $item->{_row_id} ||= $id;
+               my $row = $data->{items}->[ $id ];
+               if ( $code && $test ) {
+                       $row = Storable::dclone $row;
+                       eval $code;
+                       if ( $@ ) {
+                               warn "ERROR evaling\n$code\n$@";
+                               $self->stash('eval_error', $@) if $@;
+                       } else {
+                               warn "EVAL ",dump($row);
+                       }
+               }
+               $row->{_row_id} ||= $id;
+               push @$sorted_items, $row;
        }
 
        warn "# sorted_items ", $#$sorted_items + 1, " offset $offset limit $limit order $sort";
 
+       my $code_depends = $self->param('code_depends')||
+       join(',', grep { defined $cols_changed->{$_} && $cols_changed->{$_} == 1 } @columns );
+       my $code_description = $self->param('code_description') ||
+       join(',', grep { defined $cols_changed->{$_} && $cols_changed->{$_} == 2 } @columns );
+
+       $code_depends ||= $code_description; # self-modifing
+
+       warn "# cols_changed ",dump( $cols_changed, $code_depends, $code_description );
+
        $self->render(
                order => $order,
                offset => $offset,
@@ -611,7 +713,14 @@ sub items {
                columns => [ @columns ],
                rows => $#$filtered + 1,
                numeric => { map { $_, $self->_is_numeric($_) } @columns },
+               unique  => { map { $_, $self->_is_unique( $_) } @columns },
                filters => $self->_current_filters,
+               code => $code,
+               cols_changed => $cols_changed,
+               code_depends => $code_depends,
+               code_description => $code_description,
+               code_path => $code_path,
+               out => $out,
        );
 
 }
@@ -636,6 +745,12 @@ sub _is_numeric {
                $stats->{$name}->{numeric} > $count / 2;
 }
 
+sub _is_unique {
+       my ( $self, $name ) = @_;
+       my $stats = $self->_loaded( 'stats' );
+       defined $stats->{$name}->{unique};
+}
+
 sub _remove_filter {
        my ($self,$name) = @_;
        warn "_remove_filter $name\n";
@@ -647,7 +762,7 @@ sub _remove_filter {
        warn "filters left: ", keys %{ $loaded->{$path}->{filters} };
 
        foreach (
-                       grep { /\b$name\b/ }
+                       grep { /\Q$name\E/ }
                        keys %{ $loaded->{$path}->{filtered} }
        ) {
                delete $loaded->{$path}->{filtered}->{$_};
@@ -672,7 +787,7 @@ sub facet {
        my $data = $self->_loaded('data');
 
        my $filters = $self->_current_filters;
-       my $all_filters = join(' ',sort keys %$filters,'order:',$self->session('order'));
+       my $all_filters = __all_filters( keys %$filters,$self->session('order') );
        my $filtered = $loaded->{$path}->{filtered}->{$all_filters}
                if defined $loaded->{$path}->{filtered}->{$all_filters};
 
@@ -731,6 +846,30 @@ sub facet {
        );
 }
 
+
+sub __invalidate_path_column {
+       my ( $path, $name ) = @_;
+
+       if ( defined $loaded->{$path}->{sorted}->{$name} ) {
+               delete $loaded->{$path}->{sorted}->{$name};
+               warn "# invalidate $path sorted $name\n";
+       }
+
+       foreach ( grep { m/\Q$name\E/ } keys %{ $loaded->{$path}->{filtered} } ) {
+               delete $loaded->{$path}->{filtered}->{$_};
+               warn "# invalidate $path filtered $_\n";
+       }
+}
+
+sub __path_modified {
+       my ( $path, $value ) = @_;
+       $value = 1 unless defined $value;
+       
+       $loaded->{$path}->{modified}  = $value;
+
+       warn "# __path_modified $path $value\n";
+}
+
 sub edit {
        my $self = shift;
        my $new_content = $self->param('new_content');
@@ -783,20 +922,11 @@ sub edit {
                        warn "# change $path $i $old -> $new\n";
                        $loaded->{$path}->{data}->{items}->[$i]->{$name} = $v;
 
-                       if ( defined $loaded->{$path}->{sorted}->{$name} ) {
-                           delete $loaded->{$path}->{sorted}->{$name};
-                               warn "# invalidate $path sorted $name\n";
-                       }
-
-                       foreach ( grep { m/$name/ } keys %{ $loaded->{$path}->{filtered} } ) {
-                           delete $loaded->{$path}->{filtered}->{$_};
-                               warn "# invalidate $path filtered $_\n";
-                       }
+                       __invalidate_path_column( $path, $name );
 
                        $status = 201; # created
                        # modified = 2 -- force rebuild of stats
-                       $loaded->{$path}->{modified}  = 2;
-                       $self->session( 'modified' => 2 );
+                       __path_modified( $path, 2 );
        
                        $new_content = join("\xB6",@$v);
 
@@ -822,8 +952,7 @@ sub save {
        my $self = shift;
        my $path = $self->_param_or_session('path');
        my $dump_path = $self->_save( $path );
-       $loaded->{$path}->{modified} = 0;
-       $self->session( 'modified' => 0 );
+       __path_modified( $path, 0 );
 
        $self->redirect_to( '/data/items' );
 }