[open-ils-commits] [GIT] Evergreen ILS branch rel_2_11 updated. 24edd83b6c7ea4961aeeb5b1d2bf4a0becc1b142

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_11 has been updated
       via  24edd83b6c7ea4961aeeb5b1d2bf4a0becc1b142 (commit)
      from  244e76a10c47cbd1a7d8a932be635eb31d0d54e7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 24edd83b6c7ea4961aeeb5b1d2bf4a0becc1b142
Author: Jason Etheridge <jason at EquinoxInitiative.org>
Date:   Tue Jan 17 16:39:58 2017 -0500

    Fix Can't locate object method "workstation" error
    
    When trying to utilize the Recall functionality, Evergreen would fail to update
    targeted circulations (truncate the due date, etc.)  In the logs, we would have
    an error like this:
    
    [] open-ils.storage [ERR ::action.pm:1597:] Processing of hold failed:
    Can't locate object method "workstation" via package "action::circulation" at
    /usr/local/share/perl/5.14.2/OpenILS/Application/Storage/CDBI.pm line 181.
    
    We first tried adding the workstation column to Storage/CDBI/action.pm, but a
    similar message would then appear instead: Can't locate object method
    "checkin_workstation"
    
    This sidesteps the problem entirely.
    
    Thanks to Mike for the solution!
    
    Signed-off-by: Jason Etheridge <jason at EquinoxInitiative.org>
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI.pm
index d8391b8..d164251 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI.pm
@@ -178,7 +178,7 @@ sub to_fieldmapper {
 
     if (ref($obj)) {
         for my $field ( $fm->real_fields ) {
-            $fm->$field( ''.$obj->$field ) if defined $obj->$field;
+            $fm->$field( ''.$obj->$field ) if $obj->find_column($field) && defined $obj->$field;
         }
     }
 

-----------------------------------------------------------------------

Summary of changes:
 .../lib/OpenILS/Application/Storage/CDBI.pm        |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS