Legacy Stat Cat fields in reporter.classic_item_list need to be links, not text

Bug #1310619 reported by Chris Sharp on 2014-04-21
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Low
Unassigned
2.6
Undecided
Unassigned
2.7
Undecided
Unassigned

Bug Description

The reporter.classic_item_list view includes Legacy Stat Cat 1 and Legacy Stat Cat 2 fields, which were defined in fm_IDL.xml as "text" fields. However, since reports need the ability to include/exclude null values for these fields, they need to be links, not text, which just reports out the ID number of the asset.stat_cat anyway (not much use to the end user).

Branch to fix this issue:

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

Evergreen 2.5.1
OpenSRF 2.2.1
Ubuntu 12.04 LTS
PostgreSQL 9.1

Chris Sharp (chrissharp123) wrote :

After a quick check with Mike Rylander, I amended my above branch to make LEFT JOIN the default behavior, since not all copies are mapped to legacy stat cats.

Changed in evergreen:
milestone: none → 2.6.1
tags: added: pullrequest
Changed in evergreen:
milestone: 2.6.1 → 2.6.2
Galen Charlton (gmc) on 2014-09-09
Changed in evergreen:
milestone: 2.6.3 → 2.6.4
Changed in evergreen:
importance: Undecided → Low
assignee: nobody → Chris Sharp (chrissharp123)
Changed in evergreen:
milestone: 2.6.4 → none
status: New → Triaged
Ben Shum (bshum) wrote :

Pushed to master, rel_2_7 and rel_2_6. Minor commit alterations to add csharp's signoff author line and also the proper LP# for tracking purposes.

Changed in evergreen:
status: Triaged → Fix Committed
assignee: Chris Sharp (chrissharp123) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers