Reporter: links are broken Combined Aged and Active Circ -> Patron profle/Home library

Bug #1234932 reported by tji@sitka.bclibraries.ca
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.10
Fix Released
Medium
Unassigned
2.9
Fix Released
Medium
Unassigned

Bug Description

Patron Profile and Patron Home Library show up as links but no actual further link.

Ben Shum (bshum)
tags: added: reports
Changed in evergreen:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Codey Kolasinski (codeykolasinski) wrote :
Download full text (6.7 KiB)

We were able to fix this at C/W MARS by including the following links in the combcirc data source:

<link field="usr_home_ou" reltype="has_a" key="id" map="" class="aou"/>
<link field="usr_profile" reltype="has_a" key="id" map="" class="pgt"/>

In the end the entire fm_IDL entry looked like this:

<class id="combcirc" controller="open-ils.cstore" oils_obj:fieldmapper="action::all_circulation" oils_persist:tablename="action.all_circulation" reporter:core="true" reporter:label="Combined Aged and Active Circulations" oils_persist:readonly="true">
  <fields oils_persist:primary="id" oils_persist:sequence="money.billable_xact_id_seq">
   <field reporter:label="Check In Library" name="checkin_lib" reporter:datatype="org_unit"/>
   <field reporter:label="Check In Staff" name="checkin_staff" reporter:datatype="link"/>
   <field reporter:label="Check In Date/Time" name="checkin_time" reporter:datatype="timestamp"/>
   <field reporter:label="Circulating Library" name="circ_lib" reporter:datatype="org_unit"/>
   <field reporter:label="Circulating Staff" name="circ_staff" reporter:datatype="link"/>
   <field reporter:label="Desk Renewal" name="desk_renewal" reporter:datatype="bool"/>
   <field reporter:label="Due Date/Time" name="due_date" reporter:datatype="timestamp"/>
   <field reporter:label="Circulation Duration" name="duration" reporter:datatype="interval"/>
   <field reporter:label="Circ Duration Rule" name="duration_rule" reporter:datatype="link"/>
   <field reporter:label="Fine Interval" name="fine_interval" reporter:datatype="interval"/>
   <field reporter:label="Circ ID" name="id" reporter:datatype="id" />
   <field reporter:label="Max Fine Amount" name="max_fine" reporter:datatype="money" />
   <field reporter:label="Max Fine Rule" name="max_fine_rule" reporter:datatype="link"/>
   <field reporter:label="OPAC Renewal" name="opac_renewal" reporter:datatype="bool"/>
   <field reporter:label="Phone Renewal" name="phone_renewal" reporter:datatype="bool"/>
   <field reporter:label="Recurring Fine Amount" name="recurring_fine" reporter:datatype="money" />
   <field reporter:label="Recurring Fine Rule" name="recurring_fine_rule" reporter:datatype="link"/>
   <field reporter:label="Remaining Renewals" name="renewal_remaining" reporter:datatype="int" />
   <field reporter:label="Grace Period" name="grace_period" reporter:datatype="interval" />
   <field reporter:label="Fine Stop Reason" name="stop_fines" reporter:datatype="text"/>
   <field reporter:label="Fine Stop Date/Time" name="stop_fines_time" reporter:datatype="timestamp"/>
   <field reporter:label="Circulating Item" name="target_copy" reporter:datatype="link"/>
   <field reporter:label="Patron ZIP" name="usr_post_code" reporter:datatype="text"/>
   <field reporter:label="Transaction Finish Date/Time" name="xact_finish" reporter:datatype="timestamp" />
   <field reporter:label="Checkout Date/Time" name="xact_start" reporter:datatype="timestamp" />
   <field reporter:label="Record Creation Date/Time" name="create_time" reporter:datatype="timestamp" />
   <field reporter:label="Transaction Billings" name="billings" oils_persist:virtual="true" reporter:datatype="link"/>
   <field reporter:label...

Read more...

Revision history for this message
Michele Morgan (mmorgan) wrote :

Discovered this issue in our 2.9 system. I added Codey's fix to a working branch:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=6ae5890dd70bafe616839a2502821fb41732fea5

tags: added: pullrequest
Changed in evergreen:
assignee: nobody → Jeanette Lundgren (jlundgren)
Revision history for this message
Jeanette Lundgren (jlundgren) wrote :

I have tested this code and consent to signing off on it with my name, Jeanette Lundgren and my email address, <email address hidden>

Changed in evergreen:
assignee: Jeanette Lundgren (jlundgren) → nobody
tags: added: signedoff
Revision history for this message
Michele Morgan (mmorgan) wrote :

Force pushed a commit to change the author to Codey Kolasinski. Here's the link to the branch:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/LP_1234932_Fix_broken_links_in_combcirc

Revision history for this message
Codey Kolasinski (codeykolasinski) wrote :

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Signed-off-by: Codey Kolasinski <email address hidden>

Kathy Lussier (klussier)
Changed in evergreen:
milestone: none → 2.next
Revision history for this message
Kathy Lussier (klussier) wrote :

Thank you Michele, Codey and Jeanette! Signed off and merged to master, 2.10 and 2.9.

Changed in evergreen:
status: Triaged → Fix Committed
Changed in evergreen:
milestone: 2.next → 2.11-alpha
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.