http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7365

Ian Walls <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |In Discussion

--- Comment #13 from Ian Walls <[email protected]> ---
Okay, so this patch uses a bolt-on approach.  Three new modules are added
(along with a unit test!), along with a new Perl dependency, and then
C4/ImportExportFramework.pm is modified to use those modules instead of direct
MySQL "SHOW COLUMNS FROM".

It seems like it'd be easier to use $dbh->column_info(undef, undef, $table,
'%') or some modification thereof to get this information.  We'd then be using
a standardized, packaged database abstraction layer, instead of having to make
our own, and it'd give us more than just MySQL and Postgres.  The resulting
change would likely be fewer lines of code than what we've got here currently.

Moving to In Discussion, as we've likely still got a lot to talk about in terms
of an implementation plan for adding database agnosticity.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to