[Koha-bugs] [Bug 8798] Add the use of DBIx::Class

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Sep 24 19:03:25 CEST 2012


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

wajasu <matted-34813 at mypacks.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12443|0                           |1
        is obsolete|                            |

--- Comment #32 from wajasu <matted-34813 at mypacks.net> ---
Created attachment 12481
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12481&action=edit
SIGNED-OFF-Bug-8798-Add-the-use-of-DBIx-Class-Rename


Test:
koha at biblio:~/kohaclone$ prove t/Context.t
t/Context.t .. ok   
All tests successful.
Files=1, Tests=1,  1 wallclock secs ( 0.02 usr  0.00 sys +  0.78 cusr  0.10
csys =  0.90 CPU)
Result: PASS

koha at biblio:~/kohaclone$ ./koha_perl_deps.pl -m -u
and didn't see libdbix-class-schema-loader mentioned, as its aligned with
official debian squeeze version 7000-1 package.

Note: This patch only really gets DBIx into the environment.  The functionality
of the dbicdump generated classes is not tested here, but this patch enables a
more gradual rollout of DBIx functionality.  It would be worth noting in a
comment, what the exact command line that was used to generate the
Koha::Schema. and what version of libdbix-class-schema-loader that was used to
generate the source. 

FYI: Typically one would not check in generated source, but generate it at
build time, but DBIx uses a hash of the content in the source to determine if
it has been changed. Once someone changes it, the dbicdump won't overwrite it
anymore in future runs, unless configured to.

Note: Someone may want to address if we would like to apply the DateTime
component, to get dates inflated, if that is something worth doing across the
board.  Also, someone mentioning a desire to use "Versioned" classes, probably
for the use of database migrations.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list