Bug 6640 - Template errors in defining default variable values causes information not to be displayed
Summary: Template errors in defining default variable values causes information not to...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Owen Leonard
QA Contact: Bugs List
URL:
Keywords:
: 6493 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-07-28 13:22 UTC by Nicole C. Engard
Modified: 2013-12-05 20:01 UTC (History)
3 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
example (13.52 KB, image/png)
2011-08-01 23:56 UTC, Nicole C. Engard
Details
Proposed fix (17.66 KB, patch)
2011-08-03 14:50 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2011-07-28 13:22:53 UTC
when you use the stats wizards for stats and look for stats on shelving locations the location names don't show on the screen (but there is a row for each). If you export the file to csv then the field names do show - so it sounds to me like a template/display issue.
Comment 1 Nicole C. Engard 2011-08-01 23:56:23 UTC
Created attachment 4806 [details]
example
Comment 2 Ian Walls 2011-08-02 11:01:25 UTC
Sounds like a Template::Toolkit scope issue...
Comment 3 Owen Leonard 2011-08-03 14:50:30 UTC
Created attachment 4828 [details] [review]
Proposed fix

An invalid template syntax for defining a variable's default
    value is found in the circ report template and several
    others:
    
    [% DEFAULT LibraryNameTitle="Koha Online" %]
    
    Based on the T:T docs this appears to be the correct way of defining
    a default value:
    
    [% LibraryNameTitle or "Koha Online" %]
    
    Reference:
    
    http://template-toolkit.org/docs/manual/Syntax.html#section_Implicit_Directives_GET_and_SET
Comment 4 Owen Leonard 2011-08-12 12:42:28 UTC
*** Bug 6493 has been marked as a duplicate of this bug. ***
Comment 5 Chris Cormack 2011-08-18 08:56:32 UTC
Patch pushed, please test
Comment 6 Chris Nighswonger 2011-10-26 18:02:08 UTC
The fix for this bug was published in the 3.4.5 release. If you were the reporter of this bug, please take time to verify the fix and update the status of this bug report accordingly. If the bug is fixed to your satisfaction, please close this report.