Quick comments as I merged the code to a local branch for review:
1) There is some code duplication that could be reduced; I would like to see something like a commonly defined "can_bib_source_have_copies(bib_id)" JS function that just returns true or false.
2) Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/CDBI/config.pm should be taught about the new column on config.bib_source
Quick comments as I merged the code to a local branch for review:
1) There is some code duplication that could be reduced; I would like to see something like a commonly defined "can_bib_ source_ have_copies( bib_id) " JS function that just returns true or false.
2) Open-ILS/ src/perlmods/ lib/OpenILS/ Application/ Storage/ CDBI/config. pm should be taught about the new column on config.bib_source