on_behalf_of and sword_error functions not being used consistently.

[gobfrey]

https://github.com/eprints/eprints/blob/3.3/perl_lib/EPrints/Apache/CRUD.pm#L1004
my( $rc, $owner ) = on_behalf_of( $repo, $r, $user );
return $rc if $rc != OK;

https://github.com/eprints/eprints/blob/3.3/perl_lib/EPrints/Apache/CRUD.pm#L1732
my $on_behalf_of = on_behalf_of( $repo, $r, $user );
if( $on_behalf_of->{status} != OK )
{
return sword_error( $repo, $r, %$on_behalf_of );
}
$on_behalf_of = $on_behalf_of->{on_behalf_of};

https://github.com/eprints/eprints/blob/3.3/perl_lib/EPrints/Apache/CRUD.pm#L1972
sub sword_error
{
my( $self, %opts ) = @_;

...note self is not passed above.