Koha/opac
Julian Maurice b168f4a2e9 Bug 21395: Make perlcritic happy
This patch adds a .perlcriticrc (copied from qa-test-tools) and fixes
almost all perlcrictic violations according to this .perlcriticrc
The remaining violations are silenced out by appending a '## no critic'
to the offending lines. They can still be seen by using the --force
option of perlcritic
This patch also modify t/00-testcritic.t to check all Perl files using
the new .perlcriticrc.
I'm not sure if this test script is still useful as it is now equivalent
to `perlcritic --quiet .` and it looks like it is much slower
(approximatively 5 times slower on my machine)

Test plan:
1. Run `perlcritic --quiet .` from the root directory. It should output
   nothing
2. Run `perlcritic --quiet --force .`. It should output 7 errors (6
   StringyEval, 1 BarewordFileHandles)
3. Run `TEST_QA=1 prove t/00-testcritic.t`
4. Read the patch. Check that all changes make sense and do not
   introduce undesired behaviour

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
2020-06-29 12:37:02 +02:00
..
clubs
errors
external/overdrive
rss
sci
sco Bug 25147: (RM follow-up) Update pref references. 2020-04-17 09:03:47 +01:00
svc Bug 24384: Add Access-Control-Allow-Origin support to OPAC reports svc 2020-03-19 09:23:13 +00:00
ilsdi.pl Bug 24537: Tidy code 2020-04-06 11:01:03 +01:00
maintenance.pl
oai.pl
opac-account-pay-paypal-return.pl Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
opac-account-pay-return.pl Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
opac-account-pay.pl Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
opac-account.pl Bug 20415: Remove UseKohaPlugins system preference 2020-03-26 11:42:02 +00:00
opac-addbybiblionumber.pl
opac-alert-subscribe.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-article-request-cancel.pl Bug 22068: (QA follow-up) Return meaningful error codes 2019-04-10 19:58:31 +00:00
opac-authorities-home.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-authoritiesdetail.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-basket.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-blocked.pl
opac-browse.pl Bug 24545: Fix newly added files 2020-02-24 13:31:27 +00:00
opac-browser.pl
opac-changelanguage.pl
opac-course-details.pl
opac-course-reserves.pl
opac-detail.pl Bug 25416: Let OPAC XSLTs know if the context is an anonymous session 2020-05-15 09:33:22 +01:00
opac-discharge.pl Bug 23723: using exit inside eval to stop sending output to browser doesn't work under plack 2019-10-07 12:21:30 +01:00
opac-downloadcart.pl Bug 17842: Simplify the code 2020-06-12 10:32:35 +02:00
opac-downloadshelf.pl
opac-export.pl
opac-ics.pl Bug 24840: Replace DateTime->now with dt_from_string 2020-04-08 11:54:23 +01:00
opac-idref.pl Bug 19640: fix OPAC IdRef webservice display 2017-12-07 09:37:09 -03:00
opac-illrequests.pl Bug 23173: Provide core infrastructure 2020-04-06 11:04:19 +01:00
opac-image.pl
opac-imageviewer.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-08-30 13:40:32 +00:00
opac-ISBDdetail.pl Bug 18936: (follow-up) Fix tests, replace old get_onshelfholds_policy method 2020-02-04 09:56:25 +00:00
opac-issue-note.pl Bug 22821: Rename method with ->inbound_email_address 2020-03-18 15:44:27 +00:00
opac-library.pl Bug 13388: Add library pages to the OPAC 2020-05-04 09:11:03 +01:00
opac-main.pl Bug 22880: (follow-up) Loose ends: Better DB update, global inclusion, etc. 2020-02-24 13:24:25 +00:00
opac-MARCdetail.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-memberentry.pl Bug 25769: Patron self modification triggers change request for date of birth to null 2020-06-24 14:23:33 +02:00
opac-messaging.pl Bug 18911: Add the ability to set the preferred lang for notice at the OPAC 2020-06-24 15:15:41 +02:00
opac-modrequest-suspend.pl
opac-modrequest.pl
opac-mymessages.pl
opac-news-rss.pl
opac-overdrive-search.pl
opac-passwd.pl
opac-password-recovery.pl
opac-patron-consent.pl Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
opac-patron-image.pl
opac-privacy.pl
opac-ratings-ajax.pl
opac-ratings.pl
opac-readingrecord.pl Bug 24827: Standardise on 'UTF-8' as the encoding name 2020-03-16 14:45:39 +00:00
opac-recordedbooks-search.pl
opac-registration-verify.pl
opac-renew.pl Bug 24759: Move OpacRenewalBranch code to Koha::Item 2020-03-24 10:47:49 +00:00
opac-reportproblem.pl Bug 4461: Use inbound_email_address to know if we display the dropdown list 2020-04-06 11:19:44 +01:00
opac-request-article.pl
opac-reserve.pl Bug 19718: Show message if patron has existing holds on a title 2020-02-17 11:27:27 +00:00
opac-restrictedpage.pl Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
opac-retrieve-file.pl
opac-review.pl Bug 25340: Pass biblio object to OPAC comments template 2020-05-04 08:53:37 +01:00
opac-routing-lists.pl
opac-search-history.pl Bug 23084: Replace grep {^$var$} with grep {$_ eq $var} 2020-02-17 10:44:45 +00:00
opac-search.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-sendbasket.pl
opac-sendshelf.pl
opac-serial-issues.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-shareshelf.pl
opac-shelves.pl Bug 25416: Let OPAC XSLTs know if the context is an anonymous session 2020-05-15 09:33:22 +01:00
opac-showmarc.pl Bug 25009: Avoid leakages in opac-showmarc.pl 2020-04-27 10:44:10 +01:00
opac-showreviews.pl Bug 21395: Make perlcritic happy 2020-06-29 12:37:02 +02:00
opac-suggestions.pl Bug 22774: (follow-up) Limit purchase suggestion in a specified time period 2020-04-14 16:42:17 +01:00
opac-tags.pl Bug 23276: Do not display tag if pref TagsEnabled is off 2020-06-15 10:32:29 +02:00
opac-tags_subject.pl
opac-topissues.pl
opac-user.pl Bug 25110: Allow patrons to add star ratings to titles on their summary/checkout page 2020-04-29 17:04:32 +01:00
tracklinks.pl
unapi Bug 24052: Rename XSLT_Handler 2020-03-24 10:42:23 +00:00