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

Bug #1310619 reported by Chris Sharp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
2.6
Fix Released
Undecided
Unassigned
2.7
Fix Released
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

Tags: pullrequest
Revision history for this message
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)
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
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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