[open-ils-commits] [GIT] Evergreen ILS branch rel_2_12 updated. 5461b6973cf7200afaa97300aa0a963ba9d4711c

[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_12 has been updated
       via  5461b6973cf7200afaa97300aa0a963ba9d4711c (commit)
      from  09d0f9fffe3eca50171ca410596c2f0ccde91d2f (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 5461b6973cf7200afaa97300aa0a963ba9d4711c
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