Bug 10764

Summary: Update POD of C4::Items::GetItemStatus() to use TT syntax
Product: Koha Reporter: Owen Leonard <oleonard>
Component: DocumentationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: trivial    
Priority: P5 - low CC: bgkriegel, chrish, gmcharlt, jonathan.druart, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8670
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10764 - Update POD of C4::Items::GetItemStatus() to use TT syntax
Bug 10764 - Update POD of C4::Items::GetItemStatus() to use TT syntax
Bug 10764 - Update POD of C4::Items::GetItemStatus() to use TT syntax

Description Owen Leonard 2013-08-19 15:36:05 UTC
The POD of C4::Items::GetItemStatus() has an example of how to create a selector based on the output of the function, but it uses the old template syntax, not TT syntax.
Comment 1 Owen Leonard 2013-08-19 15:39:19 UTC Comment hidden (obsolete)
Comment 2 Magnus Enger 2013-08-20 08:21:22 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-08-20 11:51:51 UTC
Created attachment 20500 [details] [review]
Bug 10764 - Update POD of C4::Items::GetItemStatus() to use TT syntax

This patch updates the example template syntax in the POD for
C4::Items::GetItemStatus() to use Template Toolkit syntax.

To test, view the POD for C4::Items::GetItemStatus() and confirm that it
looks correct.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
This patch works as advertised (verified with "perldoc C4::Items"),
for GetItemStatus, but it does not fix a a similar example for
GetItemLocation in the same file, which still has the old template
syntax. So a followup or separate bug for that is called for.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
It seems the default option is not in used in templates.
Comment 4 Galen Charlton 2013-08-20 14:33:48 UTC
Pushed to master.  Thanks, Owen!
Comment 5 Tomás Cohen Arazi 2013-09-13 14:39:59 UTC
This patch has been pushed to 3.12.x, will be in 3.12.5.

Thanks Owen!
Comment 6 Bernardo Gonzalez Kriegel 2013-09-15 01:44:17 UTC
Pushed to 3.10.x, will be in 3.10.11
Comment 7 Chris Hall 2013-11-22 22:05:48 UTC
Pushed to 3.8.x, will be in 3.8.20.