From 55e1bf14026636ab4f4c96e18b494eac0a3e12c9 Mon Sep 17 00:00:00 2001 From: Dan Wells Date: Mon, 14 Oct 2013 16:07:17 -0400 Subject: [PATCH] Fix typo in acq code to allow quick PO creation When using 'Actions for this Record'->'View/Place Orders', the 'Create Purchase Order' button dies due to an undefined variable, 'filter'. Looking at the commit which caused the problem, it seems like this variable should have been 'state'. Simple testing bears this out. Original change was part of: LP1193095 lineitem batch actions sanity filters Signed-off-by: Dan Wells Signed-off-by: Bill Erickson --- Open-ILS/web/js/ui/default/acq/common/li_table.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/acq/common/li_table.js b/Open-ILS/web/js/ui/default/acq/common/li_table.js index 2bca1265da..9344ad91ef 100644 --- a/Open-ILS/web/js/ui/default/acq/common/li_table.js +++ b/Open-ILS/web/js/ui/default/acq/common/li_table.js @@ -518,7 +518,7 @@ function AcqLiTable() { ) { console.log("getSelected states = " + state); if (all && callback) - return this.getAll(callback, id_only, filter); + return this.getAll(callback, id_only, state); var indices = {}; /* use to uniqify. needed in paging situations. */ dojo.forEach(this.selectors, -- 2.43.2