This table should be renamed more like patrons-table or patron-results-table
(In reply to Nick Clemens from comment #0) > This table should be renamed more like patrons-table or patron-results-table patrons-table ? You mean: patrons ? That is a not an operation that I would recommend to do just before release.
I think he refers to the names we use to refer tables to in columns configuration. So this is just a selector for JS.
I don't feel strongly about this because it's not a clear identifier no matter what the id is. The include file containing the table is called "circ-patron-search-results.inc," which is fairly explanatory. Maybe that should be the table name too?
I like having it contain circ to differentiate from the normal patron search. Maybe circ-patron-search ? Results seems not strictly necessary.