https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28410
--- Comment #4 from Blou <[email protected]> --- I know I might sound blasphemous, but to me C4::Context should not carry this bagage, even at the cost of the nice OO design. I want the context. I think this includes the config file and the sysprefs, but this should not automatically get the whole schema. And the only way to do that would be to access the sysprefs without using Koha::Config::Syspref. Thus no Action::Log, no Koha::Object. I think the structure makes sense OO-wise, but as the great Scott Meyers (shit, I'm old) said : "Non-member functions improve encapsulation". In this case, it's more about pragmatism. A few old-school SELECT in C4::Context, and you reduce all your scripts memory footprints by a lot. Same with ZOOM? Does it make sense in C4::Context, in 2023 ? And Datetime::TimeZone... just to call C4::Context->tz in a few context. This could be in DateUtils, asking the context from C4::Context, instead of C4::Context carrying all that for a few calls. Maybe I'm naive, but it seems rather simple to reduce 40M of RES out of most Koha scripts. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
