Bug 7226

Summary: Can't add tags on list
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: ListsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: major    
Priority: PATCH-Sent (DO NOT USE) CC: katrin.fischer, paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed fix
[SIGNED-OFF] Bug 7226 - can't add tags on list
[SIGNED-OFF] Bug 7280 - can't place hold without selecting on list

Description Nicole C. Engard 2011-11-15 15:47:55 UTC
when on a list and choosing to tag multiple items the tags are not added and there is no error.
Comment 1 Owen Leonard 2011-12-06 15:00:10 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2011-12-06 16:05:36 UTC
Created attachment 6602 [details] [review]
[SIGNED-OFF] Bug 7226 - can't add tags on list

Incorrect check for TagsInputOnList was preventing the
right JavaScript from being triggered, and markup was
nested inside the wrong logic for the confirmation
message to be displayed consistently.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 3 Nicole C. Engard 2011-12-06 16:09:21 UTC Comment hidden (obsolete)
Comment 4 Nicole C. Engard 2011-12-06 16:09:50 UTC
Comment on attachment 6603 [details] [review]
[SIGNED-OFF] Bug 7280 - can't place hold without selecting on list

Attached wrong.
Comment 5 Paul Poulain 2011-12-16 17:16:56 UTC
QA comment: small patch, template change only
prove xt/author/valid-templates.t 
xt/author/valid-templates.t .. ok     
prove xt/author/translatable-templates.t
xt/author/translatable-templates.t .. ok   


passed QA

Patch pushed, please test