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

[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  c5980d3a22e04b15e08406bffc77c38e9ac4a31b (commit)
      from  d0604dbdddfe41faa8444aac66900edf34f4af32 (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 c5980d3a22e04b15e08406bffc77c38e9ac4a31b
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Jul 11 14:50:49 2013 -0400

    Link checker: verification review UI needs more columns about redirects
    
    Link checker results can contain information about redirects when URLs
    in a bib record happen to point to systems that return 3XX repsonses.
    
    All this is nicely captured in the uvuv and uvu tables, but the
    user interface only offers the uvu.redirect_from column, when
    uvuv.redirect_to and uvu.id are also necessary to understanding redirect
    relationships among the results.
    
    Reported by Erica Rohlfs and Bill Erickson.
    
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
    Signed-off-by: Bill Erickson <berick at esilibrary.com>

diff --git a/Open-ILS/src/templates/url_verify/review_attempt.tt2 b/Open-ILS/src/templates/url_verify/review_attempt.tt2
index fa4f3b5..91aca89 100644
--- a/Open-ILS/src/templates/url_verify/review_attempt.tt2
+++ b/Open-ILS/src/templates/url_verify/review_attempt.tt2
@@ -53,11 +53,13 @@
         fmClass="'uvuv'">
         <thead>
             <tr>
-                <th field="redirect_from" fpath="url.redirect_from"></th><!-- From here through fragment, these are fields from uvu.  We could have just used autoFieldFields on the FlattenerGrid, but by naming them explicitly we use the same field names that we use in the URL selection interface, allowing the two interfaces to use common saved filter sets. -->
+                <th field="redirect_from" fpath="url.redirect_from"></th>
+                <th field="redirect_to" fpath="redirect_to"></th><!-- From here through fragment, these are fields from uvu.  We could have just used autoFieldFields on the FlattenerGrid, but by naming them explicitly we use the same field names that we use in the URL selection interface, allowing the two interfaces to use common saved filter sets. XXX Not very important, but is this comment even true anymore? -->
+                <th field="url_id" fpath="url.id" ffilter="true"></th>
+                <th field="full_url" fpath="url.full_url" ffilter="true"></th>
                 <th field="tag" fpath="url.tag" ffilter="true"></th>
                 <th field="subfield" fpath="url.subfield" ffilter="true"></th>
                 <th field="ord" fpath="url.ord" ffilter="true"></th>
-                <th field="full_url" fpath="url.full_url" ffilter="true"></th>
                 <th field="scheme" fpath="url.scheme" _visible="false" ffilter="true"></th>
                 <th field="host" fpath="url.host" _visible="false" ffilter="true"></th>
                 <th field="domain" fpath="url.domain" _visible="false" ffilter="true"></th>

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

Summary of changes:
 .../src/templates/url_verify/review_attempt.tt2    |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS