@@ -, +, @@ - Configure plugins - Manage plugins ( install / uninstall ) - Use report plugins - Use tool plugins --- C4/Auth.pm | 1 - C4/Biblio.pm | 2 +- C4/Items.pm | 2 +- C4/UsageStats.pm | 1 - Koha/Patron.pm | 7 ++----- Koha/REST/Plugin/PluginRoutes.pm | 3 +-- Koha/REST/V1/Static.pm | 3 +-- Koha/Template/Plugin/KohaPlugins.pm | 12 ++++-------- admin/admin-home.pl | 2 +- admin/edi_accounts.pl | 5 +---- catalogue/detail.pl | 3 +-- installer/data/mysql/atomicupdate/bug_20415.sql | 1 + installer/data/mysql/sysprefs.sql | 1 - .../prog/en/modules/admin/preferences/enhanced_content.pref | 7 ------- misc/devel/install_plugins.pl | 4 +--- opac/opac-account.pl | 3 +-- plugins/plugins-enable.pl | 3 +-- plugins/plugins-home.pl | 2 +- plugins/plugins-uninstall.pl | 3 +-- plugins/plugins-upload.pl | 2 +- plugins/run.pl | 2 +- t/db_dependent/ImportBatch.t | 1 - t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t | 1 - t/db_dependent/Koha/Plugins/Patron.t | 1 - t/db_dependent/Koha/REST/Plugin/PluginRoutes.t | 2 -- t/db_dependent/Koha/Template/Plugin/KohaPlugins.t | 2 -- t/db_dependent/UsageStats.t | 1 - tools/stage-marc-import.pl | 3 +-- 28 files changed, 22 insertions(+), 58 deletions(-) create mode 100644 installer/data/mysql/atomicupdate/bug_20415.sql --- a/C4/Auth.pm +++ a/C4/Auth.pm @@ -506,7 +506,6 @@ sub get_template_and_user { OPACLocalCoverImages => C4::Context->preference('OPACLocalCoverImages'), AllowMultipleCovers => C4::Context->preference('AllowMultipleCovers'), EnableBorrowerFiles => C4::Context->preference('EnableBorrowerFiles'), - UseKohaPlugins => C4::Context->preference('UseKohaPlugins'), UseCourseReserves => C4::Context->preference("UseCourseReserves"), useDischarge => C4::Context->preference('useDischarge'), pending_checkout_notes => scalar Koha::Checkouts->search({ noteseen => 0 }), --- a/C4/Biblio.pm +++ a/C4/Biblio.pm @@ -3432,7 +3432,7 @@ sub _after_biblio_action_hooks { my $biblio_id = $args->{biblio_id}; my $action = $args->{action}; - if ( C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins") ) { + if ( C4::Context->config("enable_plugins") ) { my @plugins = Koha::Plugins->new->GetPlugins({ method => 'after_biblio_action', --- a/C4/Items.pm +++ a/C4/Items.pm @@ -2660,7 +2660,7 @@ sub _after_item_action_hooks { my $item_id = $args->{item_id}; my $action = $args->{action}; - if ( C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins") ) { + if ( C4::Context->config("enable_plugins") ) { my @plugins = Koha::Plugins->new->GetPlugins({ method => 'after_item_action', --- a/C4/UsageStats.pm +++ a/C4/UsageStats.pm @@ -216,7 +216,6 @@ sub BuildReport { NovelistSelectEnabled OpenLibraryCovers OpenLibrarySearch - UseKohaPlugins SyndeticsEnabled TagsEnabled CalendarFirstDayOfWeek --- a/Koha/Patron.pm +++ a/Koha/Patron.pm @@ -230,10 +230,7 @@ sub store { $self->privacy($default_privacy); # Call any check_password plugins if password is passed - if ( C4::Context->preference('UseKohaPlugins') - && C4::Context->config("enable_plugins") - && $self->password ) - { + if ( C4::Context->config("enable_plugins") && $self->password ) { my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'check_password', }); @@ -749,7 +746,7 @@ sub set_password { } } - if ( C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins") ) { + if ( C4::Context->config("enable_plugins") ) { # Call any check_password plugins my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'check_password', --- a/Koha/REST/Plugin/PluginRoutes.pm +++ a/Koha/REST/Plugin/PluginRoutes.pm @@ -45,8 +45,7 @@ sub register { my @plugins; - if ( C4::Context->preference('UseKohaPlugins') - && C4::Context->config("enable_plugins") ) + if ( C4::Context->config("enable_plugins") ) { # plugin needs to define a namespace @plugins = Koha::Plugins->new()->GetPlugins( --- a/Koha/REST/V1/Static.pm +++ a/Koha/REST/V1/Static.pm @@ -35,8 +35,7 @@ sub get { my $self = shift; my $c = $self->openapi->valid_input or return; - if ( C4::Context->preference('UseKohaPlugins') - && C4::Context->config("enable_plugins") ) + if ( C4::Context->config("enable_plugins") ) { my $path = $c->req->url->path->leading_slash(1); --- a/Koha/Template/Plugin/KohaPlugins.pm +++ a/Koha/Template/Plugin/KohaPlugins.pm @@ -47,8 +47,7 @@ to output to the head section of opac pages. =cut sub get_plugins_opac_head { - return q{} - unless C4::Context->preference('UseKohaPlugins'); + return q{} unless C4::Context->config("enable_plugins"); my $p = Koha::Plugins->new(); @@ -75,8 +74,7 @@ to output to the javascript section of at the bottom of opac pages. =cut sub get_plugins_opac_js { - return q{} - unless C4::Context->preference('UseKohaPlugins'); + return q{} unless C4::Context->config("enable_plugins"); my $p = Koha::Plugins->new(); @@ -103,8 +101,7 @@ to output to the head section of intranet pages. =cut sub get_plugins_intranet_head { - return q{} - unless C4::Context->preference('UseKohaPlugins'); + return q{} unless C4::Context->config("enable_plugins"); my $p = Koha::Plugins->new(); @@ -131,8 +128,7 @@ to output to the javascript section of at the bottom of intranet pages. =cut sub get_plugins_intranet_js { - return q{} - unless C4::Context->preference('UseKohaPlugins'); + return q{} unless C4::Context->config("enable_plugins"); my $p = Koha::Plugins->new(); --- a/admin/admin-home.pl +++ a/admin/admin-home.pl @@ -24,7 +24,7 @@ use Koha::Plugins; my $query = new CGI; -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); +my $plugins_enabled = C4::Context->config("enable_plugins"); my $mana_url = C4::Context->config('mana_config'); my ( $template, $loggedinuser, $cookie ) = get_template_and_user( --- a/admin/edi_accounts.pl +++ a/admin/edi_accounts.pl @@ -52,10 +52,7 @@ if ( $op eq 'acct_form' ) { ); $template->param( vendors => \@vendors ); - my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); - $template->param( plugins_enabled => $plugins_enabled ); - - if ( $plugins_enabled ) { + if ( C4::Context->config("enable_plugins") ) { my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'edifact', }); --- a/catalogue/detail.pl +++ a/catalogue/detail.pl @@ -64,8 +64,7 @@ my ( $template, $borrowernumber, $cookie, $flags ) = get_template_and_user( ); # Determine if we should be offering any enhancement plugin buttons -if ( C4::Context->preference('UseKohaPlugins') && - C4::Context->config('enable_plugins') ) { +if ( C4::Context->config('enable_plugins') ) { # Only pass plugins that can offer a toolbar button my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'intranet_catalog_biblio_enhancements_toolbar_button' --- a/installer/data/mysql/atomicupdate/bug_20415.sql +++ a/installer/data/mysql/atomicupdate/bug_20415.sql @@ -0,0 +1, @@ +DELETE FROM systempreferences WHERE variable='UseKohaPlugins'; --- a/installer/data/mysql/sysprefs.sql +++ a/installer/data/mysql/sysprefs.sql @@ -676,7 +676,6 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` ('useDischarge','','','Allows librarians to discharge borrowers and borrowers to request a discharge','YesNo'), ('UseEmailReceipts','0','','Send email receipts for payments and write-offs','YesNo'), ('UseICU','0','1','Tell Koha if ICU indexing is in use for Zebra or not.','YesNo'), -('UseKohaPlugins','0','','Enable or disable the ability to use Koha Plugins.','YesNo'), ('UseTransportCostMatrix','0','','Use Transport Cost Matrix when filling holds','YesNo'), ('UseWYSIWYGinSystemPreferences','0','','Show WYSIWYG editor when editing certain HTML system preferences.','YesNo'), ('viewISBD','1','','Allow display of ISBD view of bibiographic records','YesNo'), --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref @@ -368,13 +368,6 @@ Enhanced Content: yes: "Embed" no: "Don't embed" - YouTube links as videos. - Plugins: - - - - pref: UseKohaPlugins - choices: - yes: Enable - no: "Don't enable" - - the ability to use Koha Plugins. Note, the plugin system must also be enabled in the Koha configuration file to be fully enabled. OverDrive: - - Include OverDrive availability information with the client key --- a/misc/devel/install_plugins.pl +++ a/misc/devel/install_plugins.pl @@ -31,9 +31,7 @@ GetOptions( 'help|?' => \$help ); pod2usage(1) if $help; -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') - && C4::Context->config("enable_plugins"); -unless ($plugins_enabled) { +unless ( C4::Context->config("enable_plugins") ) { print "The plugin system must be enabled for one to be able to install plugins\n"; exit 1; --- a/opac/opac-account.pl +++ a/opac/opac-account.pl @@ -84,8 +84,7 @@ $template->param( payment_error => scalar $query->param('payment-error') || q{}, ); -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); -if ( $plugins_enabled ) { +if ( C4::Context->config("enable_plugins") ) { my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'opac_online_payment', }); --- a/plugins/plugins-enable.pl +++ a/plugins/plugins-enable.pl @@ -23,8 +23,7 @@ use C4::Context; use C4::Auth qw(check_cookie_auth); use Koha::Plugins::Handler; -die("Koha plugins are disabled!") - unless C4::Context->preference('UseKohaPlugins'); +die("Koha plugins are disabled!") unless C4::Context->config("enable_plugins"); my $input = new CGI; --- a/plugins/plugins-home.pl +++ a/plugins/plugins-home.pl @@ -30,7 +30,7 @@ use C4::Output; use C4::Debug; use C4::Context; -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); +my $plugins_enabled = C4::Context->config("enable_plugins"); my $input = new CGI; my $method = $input->param('method'); --- a/plugins/plugins-uninstall.pl +++ a/plugins/plugins-uninstall.pl @@ -29,8 +29,7 @@ use C4::Members; use C4::Debug; use Koha::Plugins::Handler; -die("Koha plugins are disabled!") - unless C4::Context->preference('UseKohaPlugins'); +die("Koha plugins are disabled!") unless C4::Context->config("enable_plugins"); my $input = new CGI; --- a/plugins/plugins-upload.pl +++ a/plugins/plugins-upload.pl @@ -31,7 +31,7 @@ use C4::Members; use C4::Debug; use Koha::Plugins; -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); +my $plugins_enabled = C4::Context->config("enable_plugins"); my $input = new CGI; --- a/plugins/run.pl +++ a/plugins/run.pl @@ -27,7 +27,7 @@ use C4::Output; use C4::Debug; use C4::Context; -my $plugins_enabled = C4::Context->preference('UseKohaPlugins') && C4::Context->config("enable_plugins"); +my $plugins_enabled = C4::Context->config("enable_plugins"); my $cgi = new CGI; --- a/t/db_dependent/ImportBatch.t +++ a/t/db_dependent/ImportBatch.t @@ -190,7 +190,6 @@ subtest "RecordsFromMarcPlugin" => sub { close $fh; t::lib::Mocks::mock_config( 'enable_plugins', 1 ); - t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); my $plugins = Koha::Plugins->new; $plugins->InstallPlugins; --- a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t +++ a/t/db_dependent/Koha/Plugins/Biblio_and_Items_plugin_hooks.t @@ -39,7 +39,6 @@ BEGIN { my $schema = Koha::Database->new->schema; my $builder = t::lib::TestBuilder->new; -t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); t::lib::Mocks::mock_config( 'enable_plugins', 1 ); subtest 'after_biblio_action() and after_item_action() hooks tests' => sub { --- a/t/db_dependent/Koha/Plugins/Patron.t +++ a/t/db_dependent/Koha/Plugins/Patron.t @@ -40,7 +40,6 @@ BEGIN { my $schema = Koha::Database->new->schema; my $builder = t::lib::TestBuilder->new; -t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); t::lib::Mocks::mock_config( 'enable_plugins', 1 ); subtest 'check_password hook tests' => sub { --- a/t/db_dependent/Koha/REST/Plugin/PluginRoutes.t +++ a/t/db_dependent/Koha/REST/Plugin/PluginRoutes.t @@ -48,7 +48,6 @@ subtest 'Bad plugins tests' => sub { # enable plugins t::lib::Mocks::mock_config( 'enable_plugins', 1 ); - t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); # remove any existing plugins that might interfere Koha::Plugins::Methods->search->delete; @@ -84,7 +83,6 @@ subtest 'Disabled plugins tests' => sub { # enable plugins t::lib::Mocks::mock_config( 'enable_plugins', 1 ); - t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); my $good_plugin; --- a/t/db_dependent/Koha/Template/Plugin/KohaPlugins.t +++ a/t/db_dependent/Koha/Template/Plugin/KohaPlugins.t @@ -24,7 +24,6 @@ BEGIN { use_ok('Koha::Plugin::Test'); } -t::lib::Mocks::mock_preference( 'UseKohaPlugins', 1 ); t::lib::Mocks::mock_config( 'enable_plugins', 1 ); my $schema = Koha::Database->new->schema; @@ -53,7 +52,6 @@ ok( index( $plugin->get_plugins_opac_head, 'Koha::Plugin::Test::opac_head' ) != ok( index( $plugin->get_plugins_intranet_js, 'Koha::Plugin::Test::intranet_js' ) != -1, 'Test plugin intranet_js return value is part of code returned by get_plugins_intranet_js' ); ok( index( $plugin->get_plugins_intranet_head, 'Koha::Plugin::Test::intranet_head' ) != -1, 'Test plugin intranet_head return value is part of code returned by get_plugins_intranet_head' ); -t::lib::Mocks::mock_preference('UseKohaPlugins',0); t::lib::Mocks::mock_config('enable_plugins',0); is( $plugin->get_plugins_opac_js, q{}, 'Test plugin opac_js return value is empty' ); is( $plugin->get_plugins_opac_head, q{}, 'Test plugin opac_head return value is empty' ); --- a/t/db_dependent/UsageStats.t +++ a/t/db_dependent/UsageStats.t @@ -477,7 +477,6 @@ sub mocking_systempreferences_to_a_set_value { NovelistSelectEnabled OpenLibraryCovers OpenLibrarySearch - UseKohaPlugins SyndeticsEnabled TagsEnabled CalendarFirstDayOfWeek --- a/tools/stage-marc-import.pl +++ a/tools/stage-marc-import.pl @@ -218,8 +218,7 @@ if ($completedJobID) { my @templates = GetModificationTemplates(); $template->param( MarcModificationTemplatesLoop => \@templates ); - if ( C4::Context->preference('UseKohaPlugins') && - C4::Context->config('enable_plugins') ) { + if ( C4::Context->config('enable_plugins') ) { my @plugins = Koha::Plugins->new()->GetPlugins({ method => 'to_marc', --