From d6688ef320cdeda5154ed1e374237688e79e3456 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 cba97c1f51..a7406777cf 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