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

Kyle M Hall <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

--- Comment #40 from Kyle M Hall <[email protected]> ---
> > By using a TT plugin, we don't have to modify each and every script that
> > uses the members toolbar.
> 
> Yes, but it's the way it's done everywhere into Koha.
> I still really don't like to see an other overhead to get this kind of info.

There is no sacrifice or performance using this method. In fact, based on my
followup, using the plugin results in somewhat better performance!

The key here is that just because we've always done something in a particular
way doesn't mean we should keep doing it that way. In fact, we have precedence
for this very pattern.

Once we introduced the plugin for getting system preferences, we began to
remove the template parameters that only passed system preferences. There is no
need to fetch the values in the script and pass them to the template, because
the script has absolutely no need for them. Doing so only added unnecessary
complexity and by using a plugin we greatly reduce that complexity. This is
essentially the exact same issue.

-- 
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