[open-ils-commits] [GIT] Evergreen ILS branch master updated. b653ed1a1cb6c6e253de98d0fadcb97e85d6feca

[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, master has been updated
       via  b653ed1a1cb6c6e253de98d0fadcb97e85d6feca (commit)
      from  96371bc6d3b158f0b7179a13be708769c76f2cec (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 b653ed1a1cb6c6e253de98d0fadcb97e85d6feca
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