Hi, On Wed, Jul 2, 2008 at 12:59 PM, Andrew Moore <andrew.moore@liblime.com> wrote:
On Wed, Jul 2, 2008 at 11:36 AM, Andrew Moore <andrew.moore@liblime.com> wrote: After some investigation with Frederic, it appears that the failures in the test suite are because the tests assume that they can update the systempreferences table and have those updates appear immediately in the C4::Context. Since this is no longer the case, that assumption is invalid. We'll have to patch the test suite to deal with that change.
So, as far as the test suite is concerned, there's no problem with this patch.
The assumption that a program, including the test suite, should be able to change a system preference value and have the new value show up in the context is not an inherently bad one. I'll create a setter for sysprefs that updates both the system preferences table and the context. Regards, Galen -- Galen Charlton Koha Application Developer LibLime galen.charlton@liblime.com p: 1-888-564-2457 x709 _______________________________________________ Koha-devel mailing list Koha-devel@lists.koha.org http://lists.koha.org/mailman/listinfo/koha-devel