From c5980d3a22e04b15e08406bffc77c38e9ac4a31b Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Thu, 11 Jul 2013 14:50:49 -0400
Subject: [PATCH] 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@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
---
 Open-ILS/src/templates/url_verify/review_attempt.tt2 | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Open-ILS/src/templates/url_verify/review_attempt.tt2 b/Open-ILS/src/templates/url_verify/review_attempt.tt2
index fa4f3b52d8..91aca89820 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>
-- 
2.11.0