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>

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 (file)
--- 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>