[open-ils-commits] [GIT] Evergreen ILS branch rel_2_4 updated. 99e51b4da401106dded016ec3a4aaadb1990be18

[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_4 has been updated
       via  99e51b4da401106dded016ec3a4aaadb1990be18 (commit)
      from  5167ab0de36e765a5ab6110899ce84f0c2b2c7be (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 99e51b4da401106dded016ec3a4aaadb1990be18
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Mon Apr 15 14:06:54 2013 -0400

    Acq: Be fuzzy about case when retrieving EDI documents
    
    The edi_fetcher relies on a routine in OpenILS::Acq::EDI to, among
    other things, try to avoid fetching the same EDI document multiple times
    when many rows in acq.edi_account refer to the same host and login
    credentials.
    
    Since in practice most vendors seem to run FTP servers for EDI on
    Windows, not UNIX, and pathnames are therefore case-insensitive, that
    test for other occurrences ought also to be case-insensitive.
    
    If I were to look at this as a purist, I could argue that vendor servers
    might sometimes by run on UNIX, and that for some reason it is possible
    that different vendor-to-buyer EDI documents (order responses or
    invoices) could have pathnames that differ only in the case of some
    characters.  But that seems wildly unlikely.  If anyone does take this
    possibility seriously, perhaps acq.edi_account needs a Boolean column to
    indicate the remote host's case [in]sensitivity.
    
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/EDI.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/EDI.pm
index a84b1c3..924787a 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/EDI.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/EDI.pm
@@ -135,7 +135,7 @@ sub retrieve_core {
                             password => $account->password,
                             in_dir => $account->in_dir
                         },
-                        remote_file => $remote_file,
+                        remote_file => {ilike => $remote_file},
                         status      => {'in' => [qw/ processed /]},
                     },
                     { join => {"acqedi" => {}}, limit => 1 }

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

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


hooks/post-receive
-- 
Evergreen ILS