Bug 13817 - Edit links on course items may point to the wrong course!
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 10 Mar 2015 17:38:50 +0000 (10:38 -0700)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 22 Apr 2015 17:35:30 +0000 (14:35 -0300)
commitb8da5462b0f72f8035589317f51800394ba700ae
tree3f96c473cfe516ecd079f3583ae18e472cb0b980
parent9cb1fe10d7ec9aa37f0f9fa691bf55cbed08959a
Bug 13817 - Edit links on course items may point to the wrong course!

If items are assigned to multiple courses, the "edit" links for a course
item may pass the wrong course id. This is really a cosmetic issue.

It appears that in Template Toolkit, if you reuse an existing variable
name in a foreach loop, it will not create a new temporary variable but
will instead re-use the existing variable leaving assigned at the end to
the last assignment, just as it would in Perl code.

This is not a problem when all items are assigned to one and only one
course, but can cause issues when we loop through the other courses an
item may be on.

This is solved easily by renaming the variable used in the foreach loop
to another unused name.

Test Plan:
1) Apply this patch
2) Try the edit and delete links for an item on reserve for a course
3) No changes in behavior should be noted

Signed-off-by: Tom Misilo <misilot@fit.edu>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt