id3fs-index -l: bare tags: don't display tags with children (eg year)
[id3fs.git] / lib / ID3FS / Path.pm
index 2d96265..406fbdf 100644 (file)
@@ -30,7 +30,7 @@ our $PATH_ALLTRACKS= "TRACKS";
 our $PATH_NOARTIST = "NOARTIST";
 our $PATH_NOALBUM  = "NOALBUM";
 
-our $ENABLE_FILTER = 1;
+our $ENABLE_FILTER = 0;
 
 sub new
 {
@@ -48,7 +48,6 @@ sub new
     $self->{path} =~ s/\/\//\//g; # drop doubled slashes
 
     $self->parse();
-#    print "STATE: ", $self->state(), "\n";
     return $self;
 }
 
@@ -91,8 +90,6 @@ sub dirents
     my @dents=();
     my @fents=();
     my $state=$self->state();
-#    print "DIRENTS: STATE: $state\n";
-#    print "DIRENTS: FILE: $self->{path}\n";
     if($state==$STATE_ALL)
     {
        @dents=($self->filter($PATH_ALLTRACKS, $PATH_NOARTIST), $self->artists());
@@ -116,7 +113,7 @@ sub dirents
     elsif($state==$STATE_BOOLEAN)
     {
        my $parent=$self->tail();
-       unless($self->is($TYPE_BOOL, $parent) && $parent->{name} eq "NOT")
+       unless($self->is($TYPE_BOOL, $parent) && $parent->name() eq "NOT")
        {
            @dents=("NOT");
        }
@@ -149,7 +146,6 @@ sub parse
     return if($self->{path} eq "/");
     @{$self->{components}}=split(/\//, $self->{path});
     shift @{$self->{components}}; # drop empty field before leading /
-#    print "PATH: $self->{path}\n";
     my @parts=@{$self->{components}};
     my($tag, $tagval);
     $self->{elements}=[];
@@ -158,16 +154,13 @@ sub parse
     my $tags_seen=0;
     while(defined(my $name=shift @parts))
     {
-#      print "NAME: $name\n";
        my $state=$self->state();
        if($state==$STATE_INVALID)
        {
-#          print "SM: INVALID: $name\n";
            return;
        }
        elsif($state==$STATE_ROOT)
        {
-#          print "SM: ROOT: $name\n";
            if($name eq "ALL")
            {
                $self->{in_all}=1;
@@ -197,10 +190,8 @@ sub parse
        elsif($state==$STATE_TAG)
        {
            my $tag=$self->tail();
-#          print "SM: TAG/TAGVAL($state): $name\n";
            if($self->is($TYPE_TAG, $tag) && $self->{db}->tag_has_values($tag->id()))
            {
-#              print "Parsing: parent: $tag->id()\n";
                my $tagval=ID3FS::Path::Node->new($self->{db}, $TYPE_TAG, $name, $tag->id());
                if(defined($tagval))
                {
@@ -246,11 +237,10 @@ sub parse
        }
        elsif($state==$STATE_BOOLEAN)
        {
-#          print "SM: BOOLEAN: $name\n";
            my $parent=$self->tail();
            my $allownot=1;
            if($self->is($TYPE_BOOL, $parent) &&
-              $parent->{name} eq "NOT")
+              $parent->name() eq "NOT")
            {
                $allownot=0;
            }
@@ -276,7 +266,6 @@ sub parse
        }
        elsif($state==$STATE_ALBUMS)
        {
-#          print "SM: ALBUM: $name\n";
            if($name eq $PATH_ALLTRACKS)
            {
                $self->state($STATE_TRACKLIST);
@@ -301,7 +290,6 @@ sub parse
        }
        elsif($state==$STATE_TRACKLIST)
        {
-#          print "SM: TRACKLIST: $name\n";
            my $track=ID3FS::Path::Node->new($self->{db}, $TYPE_FILE, $name);
            if($track)
            {
@@ -315,7 +303,6 @@ sub parse
        }
        elsif($state==$STATE_FILE)
        {
-#          print "SM: FILE: $name\n";
            # Can't have anything after a filename
            $self->state($STATE_INVALID);
        }
@@ -356,11 +343,79 @@ sub parse
     {
        pop @elements;
     }
+
+    # calculate joins and assign table numbers to nodes
+    $self->{joins}=[$self->number_joins(@elements)];
+
+    # always need at least one join
+    $self->{joins}=[qw(INNER)] unless(@{$self->{joins}});
+
     # sort elements by precedence
     @elements=$self->sort_elements(@elements);
+
+    # convert to binary tree
     $self->{tagtree}=$self->elements_to_tree(\@elements);
 }
 
+sub number_joins
+{
+    my($self, @elements)=@_;
+    my @joins=("INNER");
+    my $table=1;
+    my $nextjoin=undef;
+    my $lastop=undef;
+    return (@joins) unless(@elements);
+    while(my $node=shift @elements)
+    {
+       if($node->type() == $TYPE_BOOL)
+       {
+           my $op=$node->name();
+           if($op eq "AND")
+           {
+               $nextjoin="INNER";
+           }
+           elsif($op eq "NOT")
+           {
+               $nextjoin="LEFT";
+           }
+           elsif($op eq "OR")
+           {
+               # NOT/foo/OR needs an extra join
+               $nextjoin="INNER" if($lastop && $lastop eq "NOT");
+           }
+           $lastop=$op;
+       }
+       else
+       {
+           $node->hasvals(0);
+           if(@elements)
+           {
+               # if tag has a value, eat the tag, shifting to the value
+               if($elements[0]->type() == $TYPE_TAG)
+               {
+                   $node->hasvals(1);
+                   $node=shift(@elements);
+               }
+           }
+           elsif($self->{db}->tag_has_values($node->id()))
+           {
+               # if the expression ends in a tag that has a value
+               # (ie we have the tag and want the value)
+               # use an INNER join even if we were in a NOT
+               $nextjoin="INNER" if($nextjoin);
+           }
+           if($nextjoin)
+           {
+               $table++;
+               push(@joins, $nextjoin);
+               $nextjoin=undef;
+           }
+           $node->table($table);
+       }
+    }
+    return @joins;
+}
+
 sub state
 {
     my($self, $newstate)=@_;
@@ -378,18 +433,18 @@ sub elements_to_tree
     my($self, $elements)=@_;
     return undef unless(@$elements);
     my ($left, $right, $op)=(undef, undef, undef);
-    my $thing=pop @$elements;
-    if($self->is($TYPE_BOOL, $thing))
+    my $node=pop @$elements;
+    if($self->is($TYPE_BOOL, $node))
     {
        $right=$self->elements_to_tree($elements);
-       if($thing->{name} ne "NOT")
+       if($node->name() ne "NOT")
        {
            $left=$self->elements_to_tree($elements);
        }
-       $thing->left($left);
-       $thing->right($right);
+       $node->left($left);
+       $node->right($right);
     }
-    return $thing;
+    return $node;
 }
 
 # Dijkstra's shunting-yard algorithm
@@ -398,25 +453,25 @@ sub sort_elements
     my ($self, @input)=@_;
     my @opstack=();
     my @output=();
-    while(my $thing = shift @input)
+    while(my $node = shift @input)
     {
-       if($self->is($TYPE_TAG, $thing))
+       if($self->is($TYPE_TAG, $node))
        {
            # Handle tag values by dropping parent
            if(@input && $self->is($TYPE_TAG, $input[0]))
            {
-               $thing=shift @input;
+               $node=shift @input;
            }
-           push(@output, $thing);
+           push(@output, $node);
        }
-       elsif($self->is($TYPE_BOOL, $thing))
+       elsif($self->is($TYPE_BOOL, $node))
        {
            while(@opstack &&
-                 ($priorities{$thing->{name}} <= $priorities{$opstack[$#opstack]->{name}}))
+                 ($priorities{$node->name()} <= $priorities{$opstack[$#opstack]->name()}))
            {
                push(@output, pop(@opstack));
            }
-           push(@opstack, $thing);
+           push(@opstack, $node);
        }
     }
     while(@opstack)
@@ -473,10 +528,10 @@ sub tail
 
 sub is
 {
-    my($self, $type, $thing)=@_;
-    return 0 unless($thing);
-    return 0 unless($thing->type());
-    return 1 if($type == $thing->type());
+    my($self, $type, $node)=@_;
+    return 0 unless($node);
+    return 0 unless($node->type());
+    return 1 if($type == $node->type());
     return 0;
 }
 
@@ -497,11 +552,8 @@ sub tags
        my $sql="SELECT DISTINCT name FROM tags WHERE parents_id='';";
        return($self->{db}->cmd_firstcol($sql));
     }
-    my $hasvals=$self->expecting_values();
-    my $parent=$self->trailing_tag_parent();
-    my @ids=();
     my $sql="SELECT tags.name FROM ";
-    if($self->in_or())
+    if($self->want_all_tags())
     {
        $sql .= "files_x_tags\n";
     }
@@ -517,14 +569,7 @@ sub tags
     my $id=$self->trailing_tag_id();
 
     my $parentclause= "tags.parents_id='";
-    if($hasvals)
-    {
-       $parentclause .= $id;
-    }
-    elsif($parent)
-    {
-       $parentclause .= $parent;
-    }
+    $parentclause .= $id if($self->expecting_values());
     $parentclause .= "'";
     push(@andclauses, $parentclause);
 
@@ -553,7 +598,6 @@ sub artists
        my $sql="SELECT DISTINCT name FROM artists WHERE name!='';";
        return($self->{db}->cmd_firstcol($sql));
     }
-    my @ids=();
     my $sql=$self->sql_start("artists.name");
     $sql .= ("INNER JOIN artists ON files.artists_id=artists.id\n" .
             "WHERE artists.name != ''\n" .
@@ -567,7 +611,6 @@ sub artists
 sub albums
 {
     my($self)=@_;
-    my @ids=();
     my $tail=$self->tail();
     if($self->is($TYPE_ARTIST, $tail))
     {
@@ -679,12 +722,12 @@ sub filename
 sub tags_subselect
 {
     my($self)=@_;
-    my $hasvals=$self->expecting_values();
     my $tree=$self->{tagtree};
-    my ($sqlclause, @joins)=$tree->to_sql($hasvals) if($tree);
+    my ($sqlclause, @joins)=$tree->to_sql() if($tree);
     my $sql="\tSELECT fxt1.files_id FROM tags t1";
     my @crosses=();
     my @inners=();
+    @joins=@{$self->{joins}};
     for(my $i=0; $i <= $#joins; $i++)
     {
        my $cnt=$i+1;
@@ -750,19 +793,27 @@ sub empty
     return 1;
 }
 
-# if path is .../OR/ or .../OR/NOT
-sub in_or
+# if path is .../OR/ or .../OR/NOT or .../AND/NOT
+sub want_all_tags
 {
     my($self)=@_;
     my $tail=$self->tail();
     return 0 unless($tail);
-    return 0 unless($tail->type() == $TYPE_BOOL);
-    return 1 if($tail->name() eq "OR");
-    return 0 unless($tail->name() eq "NOT");
     my $parent=$self->tail_parent();
-    return 0 unless($parent);
-    return 0 unless($parent->type() == $TYPE_BOOL);
-    return 1 if($parent->name() eq "OR");
+    my $parent_valid = ($parent && $parent->type() == $TYPE_BOOL);
+    if($tail->type() == $TYPE_BOOL)
+    {
+       return 1 if($tail->name() eq "OR");
+       return 0 unless($tail->name() eq "NOT");
+       return 0 unless($parent_valid);
+       return 1 if($parent->name() eq "OR");
+       return 1 if($parent->name() eq "AND");
+    }
+    elsif($tail->type() == $TYPE_TAG)
+    {
+       return 0 unless($parent_valid);
+       return 1 if($parent->name() eq "NOT");
+    }
     return 0;
 }