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

            Bug ID: 39160
           Summary: Mojolicious::Plugin::OAuth2 is loaded suboptimally
 Change sponsored?: ---
           Product: Koha
           Version: Main
          Hardware: All
                OS: All
            Status: NEW
          Severity: trivial
          Priority: P5 - low
         Component: Architecture, internals, and plumbing
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]

This is a deep cut, but the following line is not optimal:
$self->plugin( 'Mojolicious::Plugin::OAuth2' => $oauth_configuration );

Mojolicious will first try to load that as
Mojolicious::Plugin::Mojolicious::Plugin::OAuth2 before it realises that we've
already loaded Mojolicious::Plugin::OAuth2 earlier in Koha::REST::V1.

I reckon that should say:
$self->plugin( 'OAuth2' => $oauth_configuration );

--

In the grand scheme of things, it's not a big deal, but we can save ourselves
26 system calls by changing 1 line.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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