Handle text direction when merging
[scpubgit/stemmaweb.git] / lib / stemmaweb / Controller / Relation.pm
index 357b9ca..a8e63d0 100644 (file)
@@ -1,12 +1,23 @@
 package stemmaweb::Controller::Relation;
+use JSON qw/ to_json from_json /;
 use Moose;
+use Moose::Util::TypeConstraints qw/ find_type_constraint /;
 use Module::Load;
 use namespace::autoclean;
+use Text::Tradition::Datatypes;
 use TryCatch;
 
 BEGIN { extends 'Catalyst::Controller' }
 
 
+sub throw {
+       Text::Tradition::Error->throw(
+               'ident' => 'Collation error',
+               'message' => $_[0],
+               );
+}
+
+
 =head1 NAME
 
 stemmaweb::Controller::Relation - Controller for the relationship mapper
@@ -30,22 +41,6 @@ sub index :Path :Args(0) {
        $c->stash->{'template'} = 'relate.tt';
 }
 
-=head2 definitions
-
- GET relation/definitions
-Returns a data structure giving the valid types and scopes for a relationship.
-
-=cut
-
-sub definitions :Local :Args(0) {
-       my( $self, $c ) = @_;
-       my $valid_relationships = [ qw/ spelling orthographic grammatical lexical transposition / ];
-       my $valid_scopes = [ qw/ local global / ];
-       $c->stash->{'result'} = { 'types' => $valid_relationships, 'scopes' => $valid_scopes };
-       $c->forward('View::JSON');
-}
-
 =head2 text
 
  GET relation/$textid/
@@ -56,8 +51,14 @@ sub definitions :Local :Args(0) {
 
 sub text :Chained('/') :PathPart('relation') :CaptureArgs(1) {
        my( $self, $c, $textid ) = @_;
-       # If the tradition has more than 500 ranks or so, split it up.
        my $tradition = $c->model('Directory')->tradition( $textid );
+       unless( $tradition ) {
+               $c->response->status('404');
+               $c->response->body("No such tradition with ID $textid");
+               $c->detach('View::Plain');
+               return;
+       }
+       
     # Account for a bad interaction between FastCGI and KiokuDB
     unless( $tradition->collation->tradition ) {
         $c->log->warn( "Fixing broken tradition link" );
@@ -69,11 +70,36 @@ sub text :Chained('/') :PathPart('relation') :CaptureArgs(1) {
     my $ok = _check_permission( $c, $tradition );
     return unless $ok;
 
+       $c->stash->{'textid'} = $textid;
+       $c->stash->{'tradition'} = $tradition;
+}
+
+sub main :Chained('text') :PathPart('') :Args(0) {
+       my( $self, $c ) = @_;
+       my $tradition = delete $c->stash->{'tradition'};
+       my $collation = $tradition->collation;
+
+       # Stash text direction to use in JS.
+       $c->stash->{'direction'} = $collation->direction;
+
+       # Stash the relationship definitions
+       $c->stash->{'relationship_scopes'} = 
+               to_json( find_type_constraint( 'RelationshipScope' )->values );
+       $c->stash->{'ternary_values'} = 
+               to_json( find_type_constraint( 'Ternary' )->values );
+       my @reltypeinfo;
+       foreach my $type ( sort { _typesort( $a, $b ) } $collation->relations->types ) {
+               next if $type->is_weak;
+               my $struct = { name => $type->name, description => $type->description };
+               push( @reltypeinfo, $struct );
+       }
+       $c->stash->{'relationship_types'} = to_json( \@reltypeinfo );
+       
        # See how big the tradition is. Edges are more important than nodes
        # when it comes to rendering difficulty.
-       my $numnodes = scalar $tradition->collation->readings;
-       my $numedges = scalar $tradition->collation->paths;
-       my $length = $tradition->collation->end->rank;
+       my $numnodes = scalar $collation->readings;
+       my $numedges = scalar $collation->paths;
+       my $length = $collation->end->rank;
        # We should display no more than roughly 500 nodes, or roughly 700
        # edges, at a time.
        my $segments = $numnodes / 500;
@@ -91,46 +117,49 @@ sub text :Chained('/') :PathPart('relation') :CaptureArgs(1) {
                        $r += $segsize;
                }
                $c->stash->{'textsegments'} = [];
-               $c->stash->{'segsize'} = $segsize;
-               $c->stash->{'margin'} = $margin;
                foreach my $i ( 0..$#divs ) {
                        my $seg = { 'start' => $divs[$i] };
                        $seg->{'display'} = "Segment " . ($i+1);
                        push( @{$c->stash->{'textsegments'}}, $seg );
                }
        }
-       $c->stash->{'textid'} = $textid;
-       $c->stash->{'tradition'} = $tradition;
-}
-
-sub main :Chained('text') :PathPart('') :Args(0) {
-       my( $self, $c ) = @_;
        my $startseg = $c->req->param('start');
-       my $tradition = delete $c->stash->{'tradition'};
-       my $collation = $tradition->collation;
        my $svgopts;
        if( $startseg ) {
                # Only render the subgraph from startseg to endseg or to END,
                # whichever is less.
-               my $endseg = $startseg + $c->stash->{'segsize'} + $c->stash->{'margin'};
+               my $endseg = $startseg + $segsize + $margin;
                $svgopts = { 'from' => $startseg };
                $svgopts->{'to'} = $endseg if $endseg < $collation->end->rank;
        } elsif( exists $c->stash->{'textsegments'} ) {
                # This is the unqualified load of a long tradition. We implicitly start 
-               # at zero, but go only as far as 550.
-               my $endseg = $c->stash->{'segsize'} + $c->stash->{'margin'};
+               # at zero, but go only as far as our segment size.
+               my $endseg = $segsize + $margin;
                $startseg = 0;
                $svgopts = { 'to' => $endseg };
        }
+       # Spit out the SVG
        my $svg_str = $collation->as_svg( $svgopts );
        $svg_str =~ s/\n//gs;
        $c->stash->{'startseg'} = $startseg if defined $startseg;
        $c->stash->{'svg_string'} = $svg_str;
        $c->stash->{'text_title'} = $tradition->name;
-       $c->stash->{'text_lang'} = $tradition->language;
+       if( $tradition->can('language') && $tradition->language ) {
+               $c->stash->{'text_lang'} = $tradition->language;
+               $c->stash->{'can_morphologize'} = 1;
+       } else {
+               $c->stash->{'text_lang'} = 'Default';
+       }
        $c->stash->{'template'} = 'relate.tt';
 }
 
+sub _typesort {
+       my( $a, $b ) = @_;
+       my $blsort = $a->bindlevel <=> $b->bindlevel;
+       return $blsort if $blsort;
+       return $a->name cmp $b->name;
+}
+
 =head2 help
 
  GET relation/help/$language
@@ -188,8 +217,15 @@ sub relationships :Chained('text') :PathPart :Args(0) {
                        my $relobj = $collation->relations->get_relationship( @$p );
                        next if $relobj->type eq 'collated'; # Don't show these
                        next if $p->[0] eq $p->[1]; # HACK until bugfix
-                       my $relhash = { source => $p->[0], target => $p->[1], 
-                                 type => $relobj->type, scope => $relobj->scope };
+                       my $relhash = { source_id => $p->[0], target_id => $p->[1], 
+                                 source_text => $collation->reading( $p->[0] )->text,
+                                 target_text => $collation->reading( $p->[1] )->text,
+                                 type => $relobj->type, scope => $relobj->scope,
+                                 a_derivable_from_b => $relobj->a_derivable_from_b,
+                                 b_derivable_from_a => $relobj->b_derivable_from_a,
+                                 non_independent => $relobj->non_independent,
+                                 is_significant => $relobj->is_significant
+                                 };
                        $relhash->{'note'} = $relobj->annotation if $relobj->has_annotation;
                        push( @all_relations, $relhash );
                }
@@ -199,23 +235,37 @@ sub relationships :Chained('text') :PathPart :Args(0) {
                if( $c->stash->{'permission'} ne 'full' ) {
                        $c->response->status( '403' );
                        $c->stash->{'result'} = { 
-                               'error' => 'You do not have permission to view this tradition.' };
+                               'error' => 'You do not have permission to modify this tradition.' };
+                       $c->detach( 'View::JSON' );
                } elsif( $c->request->method eq 'POST' ) {
-                       unless( $c->stash->{'permission'} eq 'full' ) {
-                               $c->response->status( '403' );
-                               $c->stash->{'result'} = { 
-                                       'error' => 'You do not have permission to view this tradition.' };
+                       my $opts = $c->request->params; 
+                       
+                       # Retrieve the source / target from the options
+                       my $node = delete $opts->{source_id};
+                       my $target = delete $opts->{target_id};
+                       
+                       # Make sure we didn't send a blank or invalid relationship type
+                       my $relation = $opts->{type};
+                       unless( $collation->get_relationship_type( $relation ) ) {
+                               my $errmsg = $relation ? "No such relationship type $relation" :
+                                       "You must specify a relationship type";
+                               $c->stash->{'result'} = { error => $errmsg };
+                               $c->response->status( '400' );
                                $c->detach( 'View::JSON' );
-                       }       
-                       my $node = $c->request->param('source_id');
-                       my $target = $c->request->param('target_id');
-                       my $relation = $c->request->param('rel_type');
-                       my $note = $c->request->param('note');
-                       my $scope = $c->request->param('scope');
+                       }
+                       
+                       # Keep the data clean
+                       my @booleans = qw/ a_derivable_from_b b_derivable_from_a non_independent /;
+                       foreach my $k ( keys %$opts ) {
+                               if( $opts->{$k} && grep { $_ eq $k } @booleans ) {
+                                       $opts->{$k} = 1;
+                               }
+                       }
                
-                       my $opts = { 'type' => $relation,
-                                                'scope' => $scope };
-                       $opts->{'annotation'} = $note if $note;
+                       delete $opts->{scope} unless $opts->{scope};
+                       delete $opts->{annotation} unless $opts->{annotation};
+                       delete $opts->{is_significant} unless $opts->{is_significant};
+                       $opts->{propagate} = 1;
                        
                        try {
                                my @vectors = $collation->add_relationship( $node, $target, $opts );
@@ -223,20 +273,27 @@ sub relationships :Chained('text') :PathPart :Args(0) {
                                $m->save( $tradition );
                        } catch( Text::Tradition::Error $e ) {
                                $c->response->status( '403' );
-                               $c->stash->{'result'} = { 'error' => $e->message };
+                               $c->stash->{'result'} = { error => $e->message };
+                       } catch {
+                               $c->response->status( '500' );
+                               $c->stash->{'result'} = { error => "Something went wrong with the request" };
                        }
                } elsif( $c->request->method eq 'DELETE' ) {
                        my $node = $c->request->param('source_id');
                        my $target = $c->request->param('target_id');
-               
+                       my $scopewide = $c->request->param('scopewide') 
+                               && $c->request->param('scopewide') eq 'true';
                        try {
-                               my @vectors = $collation->del_relationship( $node, $target );
+                               my @vectors = $collation->del_relationship( $node, $target, $scopewide );
                                $m->save( $tradition );
                                $c->stash->{'result'} = \@vectors;
                        } catch( Text::Tradition::Error $e ) {
                                $c->response->status( '403' );
                                $c->stash->{'result'} = { 'error' => $e->message };
-                       }       
+                       } catch {
+                               $c->response->status( '500' );
+                               $c->stash->{'result'} = { error => "Something went wrong with the request" };
+                       }
                }
        }
        $c->forward('View::JSON');
@@ -264,19 +321,17 @@ sub _reading_struct {
        # Return a JSONable struct of the useful keys.  Keys meant to be writable
        # have a true value; read-only keys have a false value.
        my $struct = {};
-       map { $struct->{$_} = $reading->$_ } keys( %read_write_keys );
+       map { $struct->{$_} = $reading->$_ if $reading->can( $_ ) } keys( %read_write_keys );
        # Special case
-       $struct->{'lexemes'} = [ $reading->lexemes ];
+       $struct->{'lexemes'} = $reading->can( 'lexemes' ) ? [ $reading->lexemes ] : [];
        # Look up any words related via spelling or orthography
        my $sameword = sub { 
                my $t = $_[0]->type;
                return $t eq 'spelling' || $t eq 'orthographic';
        };
-       my @variants;
-       foreach my $sr ( $reading->related_readings( $sameword ) ) {
-               push( @variants, $sr->text );
-       }
-       $struct->{'variants'} = \@variants;
+       # Now add the list data
+       $struct->{'variants'} = [ map { $_->text } $reading->related_readings( $sameword ) ];
+       $struct->{'witnesses'} = [ $reading->witnesses ];
        return $struct;
 }
 
@@ -323,51 +378,56 @@ sub reading :Chained('text') :PathPart :Args(1) {
                if( $c->stash->{'permission'} ne 'full' ) {
                        $c->response->status( '403' );
                        $c->stash->{'result'} = { 
-                               'error' => 'You do not have permission to view this tradition.' };
+                               'error' => 'You do not have permission to modify this tradition.' };
                        $c->detach('View::JSON');
+                       return;
                }
                my $errmsg;
-               # Are we re-lemmatizing?
-               if( $c->request->param('relemmatize') ) {
-                       my $nf = $c->request->param('normal_form');
-                       # TODO throw error unless $nf
-                       $rdg->normal_form( $nf );
-                       # TODO throw error if lemmatization fails
-                       # TODO skip this if normal form hasn't changed
-                       $rdg->lemmatize();
-               } else {
-                       # Set all the values that we have for the reading.
-                       # TODO error handling
-                       foreach my $p ( keys %{$c->request->params} ) {
-                               if( $p =~ /^morphology_(\d+)$/ ) {
-                                       # Set the form on the correct lexeme
-                                       my $morphval = $c->request->param( $p );
-                                       next unless $morphval;
-                                       my $midx = $1;
-                                       my $lx = $rdg->lexeme( $midx );
-                                       my $strrep = $rdg->language . ' // ' . $morphval;
-                                       my $idx = $lx->has_form( $strrep );
-                                       unless( defined $idx ) {
-                                               # Make the word form and add it to the lexeme.
-                                               try {
-                                                       $idx = $lx->add_matching_form( $strrep ) - 1;
-                                               } catch( Text::Tradition::Error $e ) {
-                                                       $c->response->status( '403' );
-                                                       $errmsg = $e->message;
-                                               } catch {
-                                                       # Something else went wrong, probably a Moose error
-                                                       $c->response->status( '403' );
-                                                       $errmsg = 'Something went wrong with the request';      
+               if( $rdg && $rdg->does('Text::Tradition::Morphology') ) {
+                       # Are we re-lemmatizing?
+                       if( $c->request->param('relemmatize') ) {
+                               my $nf = $c->request->param('normal_form');
+                               # TODO throw error unless $nf
+                               $rdg->normal_form( $nf );
+                               # TODO throw error if lemmatization fails
+                               # TODO skip this if normal form hasn't changed
+                               $rdg->lemmatize();
+                       } else {
+                               # Set all the values that we have for the reading.
+                               # TODO error handling
+                               foreach my $p ( keys %{$c->request->params} ) {
+                                       if( $p =~ /^morphology_(\d+)$/ ) {
+                                               # Set the form on the correct lexeme
+                                               my $morphval = $c->request->param( $p );
+                                               next unless $morphval;
+                                               my $midx = $1;
+                                               my $lx = $rdg->lexeme( $midx );
+                                               my $strrep = $rdg->language . ' // ' . $morphval;
+                                               my $idx = $lx->has_form( $strrep );
+                                               unless( defined $idx ) {
+                                                       # Make the word form and add it to the lexeme.
+                                                       try {
+                                                               $idx = $lx->add_matching_form( $strrep ) - 1;
+                                                       } catch( Text::Tradition::Error $e ) {
+                                                               $c->response->status( '403' );
+                                                               $errmsg = $e->message;
+                                                       } catch {
+                                                               # Something else went wrong, probably a Moose error
+                                                               $c->response->status( '500' );
+                                                               $errmsg = 'Something went wrong with the request';      
+                                                       }
                                                }
+                                               $lx->disambiguate( $idx ) if defined $idx;
+                                       } elsif( $read_write_keys{$p} ) {
+                                               my $val = _clean_booleans( $rdg, $p, $c->request->param( $p ) );
+                                               $rdg->$p( $val );
                                        }
-                                       $lx->disambiguate( $idx ) if defined $idx;
-                               } elsif( $read_write_keys{$p} ) {
-                                       my $val = _clean_booleans( $rdg, $p, $c->request->param( $p ) );
-                                       $rdg->$p( $val );
-                               }
-                       }               
+                               }               
+                       }
+                       $m->save( $rdg );
+               } else {
+                       $errmsg = "Reading does not exist or cannot be morphologized";
                }
-               $m->save( $rdg );
                $c->stash->{'result'} = $errmsg ? { 'error' => $errmsg }
                        : _reading_struct( $rdg );
 
@@ -376,13 +436,308 @@ sub reading :Chained('text') :PathPart :Args(1) {
 
 }
 
+sub compress :Chained('text') :PathPart :Args(0) {
+       my( $self, $c ) = @_;
+       my $tradition = delete $c->stash->{'tradition'};
+       my $collation = $tradition->collation;
+       my $m = $c->model('Directory');
+
+       my @rids = $c->request->param('readings[]');
+       my @readings;
+
+       foreach my $rid (@rids) {
+               my $rdg = $collation->reading( $rid );
+
+               push @readings, $rdg;
+       }
+
+       my $len = scalar @readings;
+
+       if( $c->request->method eq 'POST' ) {
+               if( $c->stash->{'permission'} ne 'full' ) {
+                       $c->response->status( '403' );
+                       $c->stash->{'result'} = { 
+                               'error' => 'You do not have permission to modify this tradition.' };
+                       $c->detach('View::JSON');
+                       return;
+               }
+
+               # Sanity check: first save the original text of each witness.
+               my %origtext;
+               foreach my $wit ( $tradition->witnesses ) {
+                       $origtext{$wit->sigil} = $collation->path_text( $wit->sigil );
+                       if( $wit->is_layered ) {
+                               my $acsig = $wit->sigil . $collation->ac_label;
+                               $origtext{$acsig} = $collation->path_text( $acsig );
+                       }
+               }
+
+               my $first = 0;
+
+               for (my $i = 0; $i < $len; $i++) {
+                       my $rdg = $readings[$i];
+
+                       if ($rdg->is_combinable) {
+                               $first = $i;
+                               last;
+                       }
+               }
+
+               my @nodes;
+               push @nodes, "$readings[$first]";
+
+               for (my $i = $first+1; $i < $len; $i++) {
+                       my $rdg = $readings[$first];
+                       my $next = $readings[$i];
+
+                       last unless $next->is_combinable;
+                       push @nodes, "$next";
+
+                       try {
+                               $collation->merge_readings( "$rdg", "$next", 1 );
+                       } catch ($e) {
+                               $c->stash->{result} = {
+                                       error_msg => $e->message,
+                               };
+
+                               $c->detach('View::JSON');
+                       }
+               }
+               
+               try {
+                       # Finally, make sure we haven't screwed anything up.
+                       foreach my $wit ( $tradition->witnesses ) {
+                               my $pathtext = $collation->path_text( $wit->sigil );
+                               throw( "Text differs for witness " . $wit->sigil )
+                                       unless $pathtext eq $origtext{$wit->sigil};
+                               if( $wit->is_layered ) {
+                                       my $acsig = $wit->sigil . $collation->ac_label;
+                                       $pathtext = $collation->path_text( $acsig );
+                                       throw( "Layered text differs for witness " . $wit->sigil )
+                                               unless $pathtext eq $origtext{$acsig};
+                               }
+                       }
+               } catch ($e) {
+                       $c->stash->{result} = {
+                               error_msg => $e->message,
+                       };
+
+                       $c->detach('View::JSON');
+               }
+
+
+               $collation->relations->rebuild_equivalence();
+               $collation->calculate_ranks();
+
+               $m->save($collation);
+
+               if ($collation->direction eq 'RL') {
+                       @nodes = reverse @nodes;
+               }
+
+               $c->stash->{'result'} = {
+                       success => 1,
+                       nodes   => \@nodes,
+               };
+
+               $c->forward('View::JSON');
+       }
+}
+
+=head2 merge
+
+ POST relation/$textid/merge { data }
+Merges the requested readings, combining the witnesses of both readings into
+the target reading. All non-conflicting source relationships are inherited by
+the target relationship.
+
+=cut
+
+sub merge :Chained('text') :PathPart :Args(0) {
+       my( $self, $c ) = @_;
+       my $tradition = delete $c->stash->{'tradition'};
+       my $collation = $tradition->collation;
+       my $m = $c->model('Directory');
+       if( $c->request->method eq 'POST' ) {
+               if( $c->stash->{'permission'} ne 'full' ) {
+                       $c->response->status( '403' );
+                       $c->stash->{'result'} = { 
+                               'error' => 'You do not have permission to modify this tradition.' };
+                       $c->detach('View::JSON');
+                       return;
+               }
+               my $errmsg;
+               my $response;
+               
+               my $main = $c->request->param('target_id');
+               my $second = $c->request->param('source_id');
+               # Find the common successor of these, so that we can detect other
+               # potentially identical readings.
+               my $csucc = $collation->common_successor( $main, $second );
+
+               # Try the merge if these are parallel readings.
+               if( $csucc->id eq $main || $csucc->id eq $second ) {
+                       $errmsg = "Cannot merge readings in the same path";
+               } else {
+                       try {
+                               $collation->merge_readings( $main, $second );
+                       } catch( Text::Tradition::Error $e ) {
+                               $c->response->status( '403' );
+                               $errmsg = $e->message;
+                       } catch {
+                               # Something else went wrong, probably a Moose error
+                               $c->response->status( '403' );
+                               $errmsg = 'Something went wrong with the request';      
+                       }
+               }
+               
+               # Look for readings that are now identical.
+               if( $errmsg ) {
+                       $response = { status => 'error', error => $errmsg };
+               } else {
+                       $response = { status => 'ok' };
+                       unless( $c->request->param('single') ) {
+                               my @identical = $collation->identical_readings(
+                                       start => $main, end => $csucc->id );
+                               if( @identical ) {
+                                       $response->{'checkalign'} = [ 
+                                               map { [ $_->[0]->id, $_->[1]->id ] } @identical ];
+                               }
+                       }
+                       $m->save( $collation );
+               }
+               $c->stash->{'result'} = $response;
+               $c->forward('View::JSON');                      
+       }
+}
+
+=head2 duplicate
+
+ POST relation/$textid/duplicate { data }
+Duplicates the requested readings, detaching the witnesses specified in
+the list to use the new reading(s) instead of the old. The data to be
+passed should be a JSON structure:
+
+ { readings: rid1,rid2,rid3,...
+   witnesses: [ wit1, ... ] }
+
+=cut
+
+sub duplicate :Chained('text') :PathPart :Args(0) {
+       my( $self, $c ) = @_;
+       my $tradition = delete $c->stash->{'tradition'};
+       my $collation = $tradition->collation;
+       my $m = $c->model('Directory');
+       if( $c->request->method eq 'POST' ) {
+               if( $c->stash->{'permission'} ne 'full' ) {
+                       $c->response->status( '403' );
+                       $c->stash->{'result'} = { 
+                               'error' => 'You do not have permission to modify this tradition.' };
+                       $c->detach('View::JSON');
+                       return;
+               }
+               my $errmsg;
+               my $response = {};
+               # Sort out which readings need to be duplicated from the set given, and
+               # ensure that all the given wits bear each relevant reading.
+               
+               my %wits = ();
+               map { $wits{$_} = 1 } $c->request->param('witnesses[]');
+               my %rdgranks = ();
+               foreach my $rid ( $c->request->param('readings[]') ) {
+                       my $numwits = 0;
+                       my $rdg = $collation->reading( $rid );
+                       foreach my $rwit ( $rdg->witnesses( $rid ) ) {
+                               $numwits++ if exists $wits{$rwit};
+                       }
+                       next unless $numwits; # Disregard readings with none of our witnesses
+                       if( $numwits < keys( %wits ) ) {
+                               $errmsg = "Reading $rid contains some but not all of the specified witnesses.";
+                               last;
+                       } elsif( exists $rdgranks{ $rdg->rank } ) {
+                               $errmsg = "More than one reading would be detached along with $rid at rank " . $rdg->rank;
+                               last;
+                       } else {
+                               $rdgranks{ $rdg->rank } = $rid;
+                       }
+               }
+               
+               # Now check that the readings make a single sequence.
+               unless( $errmsg ) {
+                       my $prior;
+                       foreach my $rank ( sort { $a <=> $b } keys %rdgranks ) {
+                               my $rid = $rdgranks{$rank};
+                               if( $prior ) {
+                                       # Check that there is only one path between $prior and $rdg.
+                                       foreach my $wit ( keys %wits ) {
+                                               unless( $collation->prior_reading( $rid, $wit ) eq $prior ) {
+                                                       $errmsg = "Diverging witness paths from $prior to $rid at $wit";
+                                                       last;
+                                               }
+                                       }
+                               }
+                               $prior = $rid;
+                       }
+               }
+               
+               # Abort if we've run into a problem.
+               if( $errmsg ) {
+                       $c->stash->{'result'} = { 'error' => $errmsg };
+                       $c->response->status( '403' );
+                       $c->forward('View::JSON');
+                       return;
+               }
+               
+               # Otherwise, do the dirty work.
+               my @witlist = keys %wits;
+               my @deleted_relations;
+               foreach my $rank ( sort { $a <=> $b } keys %rdgranks ) {
+                       my $newrdg;
+                       my $reading_id = $rdgranks{$rank};
+                       my @delrels;
+                       try {
+                               ( $newrdg, @delrels ) = 
+                                       $collation->duplicate_reading( $reading_id, @witlist );
+                       } catch( Text::Tradition::Error $e ) {
+                               $c->response->status( '403' );
+                               $errmsg = $e->message;
+                       } catch {
+                               # Something else went wrong, probably a Moose error
+                               $c->response->status( '500' );
+                               $errmsg = 'Something went wrong with the request';      
+                       }
+                       if( $newrdg ) {
+                               my $data = _reading_struct( $newrdg );
+                               $data->{'orig_rdg'} = $reading_id;
+                               $response->{"$newrdg"} = $data;
+                               push( @deleted_relations, @delrels );
+                       }
+               } 
+               if( $errmsg ) {
+                       $c->stash->{'result'} = { 'error' => $errmsg };
+               } else {
+                       $m->save( $collation );
+                       $response->{'DELETED'} = \@deleted_relations;
+                       $c->stash->{'result'} = $response;
+               }
+       }
+       $c->forward('View::JSON');
+}
+
+
+
 sub _check_permission {
        my( $c, $tradition ) = @_;
     my $user = $c->user_exists ? $c->user->get_object : undef;
+    # Does this user have access?
     if( $user ) {
-       $c->stash->{'permission'} = 'full'
-               if( $user->is_admin || $tradition->user->id eq $user->id );
-       return 1;
+               if( $user->is_admin || 
+                       ( $tradition->has_user && $tradition->user->id eq $user->id ) ) {
+                       $c->stash->{'permission'} = 'full';
+                       return 1;
+               }
     } 
     # Is it public?
     if( $tradition->public ) {
@@ -397,8 +752,8 @@ sub _check_permission {
 }
 
 sub _clean_booleans {
-       my( $rdg, $param, $val ) = @_;
-       if( $rdg->meta->get_attribute( $param )->type_constraint->name eq 'Bool' ) {
+       my( $obj, $param, $val ) = @_;
+       if( $obj->meta->get_attribute( $param )->type_constraint->name eq 'Bool' ) {
                $val = 1 if $val eq 'true';
                $val = undef if $val eq 'false';
        }