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=22a1d0701a7654b3cb04133fffbaad4491eabc0d;hpb=19f0f822616e0e8c160392dda4452a11b17444d6;p=scpubgit%2Fstemmaweb.git

diff --git a/lib/stemmaweb/Controller/Relation.pm b/lib/stemmaweb/Controller/Relation.pm
index 22a1d07..24d1de3 100644
--- a/lib/stemmaweb/Controller/Relation.pm
+++ b/lib/stemmaweb/Controller/Relation.pm
@@ -1,7 +1,10 @@
 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' }
@@ -30,22 +33,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,19 +43,52 @@ 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" );
         $tradition->collation->_set_tradition( $tradition );
         $c->model('Directory')->save( $tradition );
     }
+    # Check permissions. Will return 403 if denied, otherwise will
+    # put the appropriate value in the stash.
+    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 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;
@@ -86,46 +106,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
@@ -172,6 +195,8 @@ success or 403 on error.
 sub relationships :Chained('text') :PathPart :Args(0) {
 	my( $self, $c ) = @_;
 	my $tradition = delete $c->stash->{'tradition'};
+	my $ok = _check_permission( $c, $tradition );
+	return unless $ok;
 	my $collation = $tradition->collation;
 	my $m = $c->model('Directory');
 	if( $c->request->method eq 'GET' ) {
@@ -181,43 +206,84 @@ 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 );
 		}
 		$c->stash->{'result'} = \@all_relations;
-	} 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 = { 'type' => $relation,
-					 'scope' => $scope };
-		$opts->{'annotation'} = $note if $note;
-		
-		try {
-			my @vectors = $collation->add_relationship( $node, $target, $opts );
-			$c->stash->{'result'} = \@vectors;
-			$m->save( $tradition );
-		} catch( Text::Tradition::Error $e ) {
+	} else {
+		# Check write permissions first of all
+		if( $c->stash->{'permission'} ne 'full' ) {
 			$c->response->status( '403' );
-			$c->stash->{'result'} = { 'error' => $e->message };
+			$c->stash->{'result'} = { 
+				'error' => 'You do not have permission to modify this tradition.' };
+			$c->detach( 'View::JSON' );
+		} elsif( $c->request->method eq 'POST' ) {
+			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;
+				}
+			}
+		
+			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 );
+				$c->stash->{'result'} = \@vectors;
+				$m->save( $tradition );
+			} 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" };
+			}
+		} 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, $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" };
+			}
 		}
-	} elsif( $c->request->method eq 'DELETE' ) {
-		my $node = $c->request->param('source_id');
-		my $target = $c->request->param('target_id');
-	
-		try {
-			my @vectors = $collation->del_relationship( $node, $target );
-			$m->save( $tradition );
-			$c->stash->{'result'} = \@vectors;
-		} catch( Text::Tradition::Error $e ) {
-			$c->response->status( '403' );
-			$c->stash->{'result'} = { 'error' => $e->message };
-		}	
 	}
 	$c->forward('View::JSON');
 }
@@ -244,25 +310,25 @@ 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;
 }
 
 sub readings :Chained('text') :PathPart :Args(0) {
 	my( $self, $c ) = @_;
 	my $tradition = delete $c->stash->{'tradition'};
+	my $ok = _check_permission( $c, $tradition );
+	return unless $ok;
 	my $collation = $tradition->collation;
 	my $m = $c->model('Directory');
 	if( $c->request->method eq 'GET' ) {
@@ -298,45 +364,59 @@ sub reading :Chained('text') :PathPart :Args(1) {
 		$c->stash->{'result'} = $rdg ? _reading_struct( $rdg )
 			: { 'error' => "No reading with ID $reading_id" };
 	} elsif ( $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;
-		# 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 $midx = $1;
-					$c->log->debug( "Fetching lexeme $midx" );
-					my $lx = $rdg->lexeme( $midx );
-					my $strrep = $rdg->language . ' // ' 
-						. $c->request->param( $p );
-					my $idx = $lx->has_form( $strrep );
-					unless( defined $idx ) {
-						# Make the word form and add it to the lexeme.
-						$c->log->debug("Adding new form for $strrep");
-						try {
-							$idx = $lx->add_matching_form( $strrep ) - 1;
-						} catch( Text::Tradition::Error $e ) {
-							$c->response->status( '403' );
-							$errmsg = $e->message;
+		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 );
 
@@ -345,9 +425,216 @@ 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;
+    # Does this user have access?
+    if( $user ) {
+   		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 ) {
+    	$c->stash->{'permission'} = 'readonly';
+    	return 1;
+    } 
+	# Forbidden!
+	$c->response->status( 403 );
+	$c->response->body( 'You do not have permission to view this tradition.' );
+	$c->detach( 'View::Plain' );
+	return 0;
+}
+
 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';
 	}