X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2Fstemmaweb%2FController%2FRelation.pm;h=24d1de39ad040897c60513c71970d71f664e6a84;hb=d935aef8ba54569b02ab830e070bc486415643c0;hp=0b5d1bb17e9809ddd904e0ea40a192afe9769e48;hpb=56e3972ee78693b5fbcb530fad920ed9beb6be5a;p=scpubgit%2Fstemmaweb.git diff --git a/lib/stemmaweb/Controller/Relation.pm b/lib/stemmaweb/Controller/Relation.pm index 0b5d1bb..24d1de3 100644 --- a/lib/stemmaweb/Controller/Relation.pm +++ b/lib/stemmaweb/Controller/Relation.pm @@ -1,8 +1,10 @@ package stemmaweb::Controller::Relation; -use JSON qw/ to_json /; +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' } @@ -70,7 +72,10 @@ sub main :Chained('text') :PathPart('') :Args(0) { my $collation = $tradition->collation; # Stash the relationship definitions - $c->stash->{'relationship_scopes'} = to_json( [ qw/ local global / ] ); + $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; @@ -128,7 +133,7 @@ sub main :Chained('text') :PathPart('') :Args(0) { $c->stash->{'startseg'} = $startseg if defined $startseg; $c->stash->{'svg_string'} = $svg_str; $c->stash->{'text_title'} = $tradition->name; - if( $tradition->can('language') ) { + if( $tradition->can('language') && $tradition->language ) { $c->stash->{'text_lang'} = $tradition->language; $c->stash->{'can_morphologize'} = 1; } else { @@ -201,8 +206,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 ); } @@ -215,15 +227,34 @@ sub relationships :Chained('text') :PathPart :Args(0) { 'error' => 'You do not have permission to modify this tradition.' }; $c->detach( 'View::JSON' ); } elsif( $c->request->method eq 'POST' ) { - 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'); + 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' ); + } + + # 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, 'propagate' => 1 }; - $opts->{'scope'} = $scope if $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 ); @@ -231,20 +262,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'); @@ -280,11 +318,9 @@ sub _reading_struct { 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; } @@ -331,7 +367,7 @@ 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; } @@ -366,7 +402,7 @@ sub reading :Chained('text') :PathPart :Args(1) { $errmsg = $e->message; } catch { # Something else went wrong, probably a Moose error - $c->response->status( '403' ); + $c->response->status( '500' ); $errmsg = 'Something went wrong with the request'; } } @@ -389,6 +425,190 @@ sub reading :Chained('text') :PathPart :Args(1) { } +=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; @@ -413,8 +633,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'; }