LP#1468422 Remove deprecated open-ils.storage remote_update
authorBill Erickson <berickxx@gmail.com>
Tue, 24 Nov 2015 17:18:16 +0000 (12:18 -0500)
committerBill Erickson <berickxx@gmail.com>
Fri, 26 Feb 2016 15:07:42 +0000 (10:07 -0500)
User update in Actor.pm was the only remaining code that leveraged
the open-ils.storage remote_update API.  With that code moving to
open-ils.cstore, save some RAM by no longer auto-loading/publishing
remote_update methods.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher.pm

index a10d1d3..db78f30 100644 (file)
@@ -332,25 +332,6 @@ sub mass_delete {
     return $success;
 }
 
-sub remote_update_node {
-    my $self = shift;
-    my $client = shift;
-    my $keys = shift;
-    my $vals = shift;
-
-    local $OpenILS::Application::Storage::WRITE = 1;
-
-    my $cdbi = $self->{cdbi};
-
-    my $success = 1;
-    try {
-        $success = $cdbi->remote_update($keys,$vals);
-    } catch Error with {
-        $success = 0;
-    };
-    return $success;
-}
-
 sub merge_node {
     my $self = shift;
     my $client = shift;
@@ -718,29 +699,6 @@ for my $fmclass ( (Fieldmapper->classes) ) {
             );
         }
 
-        # Create the remote_update method
-        unless ( __PACKAGE__->is_registered( $api_prefix.'.remote_update' ) ) {
-            __PACKAGE__->register_method(
-                api_name    => $api_prefix.'.remote_update',
-                method      => 'remote_update_node',
-                api_level   => 1,
-                cdbi        => $cdbi,
-                argc        => 1,
-            );
-        }
-
-        # Create the batch remote_update method
-        unless ( __PACKAGE__->is_registered( $api_prefix.'.batch.remote_update' ) ) {
-            __PACKAGE__->register_method(
-                api_name    => $api_prefix.'.batch.remote_update',
-                method      => 'batch_call',
-                api_level   => 1,
-                unwrap      => 1,
-                cdbi        => $cdbi,
-                argc        => 1,
-            );
-        }
-
         # Create the search-based mass delete method
         unless ( __PACKAGE__->is_registered( $api_prefix.'.mass_delete' ) ) {
             __PACKAGE__->register_method(