Bug 7577

Summary: Suggestion show page
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: enhancement    
Priority: P5 - low CC: paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 5346    
Bug Blocks: 8152    
Attachments: Bug 7577 Adds a display page for suggestions
Bug 7577 Adds a display page for suggestions
[SIGNED-OFF] Bug 7577: Adds a display page for suggestions

Description Jonathan Druart 2012-02-22 14:35:48 UTC
We would like to have a page where we could display a suggestion.
Comment 1 Jonathan Druart 2012-02-22 14:39:49 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-02-22 15:30:17 UTC
There are some HTML markup issues with the template for suggestion.pl:

Because the validator doesn't like when you use a <label> without an input, there is an alternative for displaying information which isn't in a form: <span class="label">. Using that will give you the same visual style and keep the validator happy.

I would probably make display of each of those lines conditional on the existence of the data:

        [% IF ( total ) %]<li>
          <span class="label">Total</span>[% total %]
        </li>[% END %]

...but others might disagree. I'm not sure when the absence of data is important data.

There's also a problem with the conditional display of the page's structural markup. There is a superfluous section at the bottom of the page source:

<div id="doc3" class="yui-t2">
<div id="bd">
    <div id="yui-main">
    <div class="yui-b">
Comment 3 Jonathan Druart 2012-02-22 16:48:53 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2012-03-09 17:04:05 UTC
Created attachment 8125 [details] [review]
[SIGNED-OFF] Bug 7577: Adds a display page for suggestions

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Ian Walls 2012-03-09 17:39:27 UTC
Largely a template change; the changes to suggestions.pl are only insofar as to add support for the 'show' op, and clean up some of the foreach statements.

Marking Passed QA