https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15504

--- Comment #82 from Andrii Nugged <[email protected]> ---
QA-question:

This sub:

    sub update_lastseen {
        my ( $self, $activity ) = @_;
        return $self if !C4::Context->preference('TrackLastPatronActivity');

        my $tracked_activities = {
            map { ( lc $_, 1 ); } split /\s*\,\s*/,
            C4::Context->preference('TrackLastPatronActivityTriggers')
        };
        return $self unless $tracked_activities->{$activity};

        ...

is called very often (on each Auth-request, for each borrower, so each page
load or whatever happens, then for every other event).

Is that "split" of static variable from
preference('TrackLastPatronActivityTriggers') justified to be called that way
EVERY time, splitting again and again?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
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/

Reply via email to