[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-11-02 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

David Cook  changed:

   What|Removed |Added

 Resolution|--- |INVALID
 Status|Patch doesn't apply |RESOLVED

--- Comment #7 from David Cook  ---
I don't think this is needed thanks to
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112814

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-27 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Status|Needs Signoff   |Patch doesn't apply
 CC||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

David Cook  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |dc...@prosentient.com.au
   |ity.org |

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

David Cook  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=26673

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

David Cook  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

--- Comment #6 from David Cook  ---
Koha::Z3950Responder::RPN test plan:

0. Open 2 terminals

Terminal 1:
1. koha-z3950-responder --stop kohadev
2. perl misc/z3950_responder.pl

Terminal 2:
3. yaz-client 127.0.0.1:2100/biblios
4. #Note connection accepted
5. find @or @1=4 Bruce @1=4 Damian
6. #Note Number of hits: 9
7. #Iterate through results using "show" and notice the 245 has either Bruce or
Damian in it

Note that this just makes sure the koha-z3950-responder is still running. It
doesn't actually test that the Net::Z3950::RPN::* modules are actually doing
anything it seems. 

Ere Maijala might be the best person to verify this, since he's the one who
added Koha::Z3950Responder::RPN in Bug 13937 for "Add support for SRU and
Elasticsearch"

Before I added the 3rd patch "Load localized Net::Z3950::RPN::* modules", the
t/db_dependent/Koha/Z3950Responder/GenericSession.t test was failing, but now
it's passing.

--

(It's a shame we don't have a "lib" directory that contains all our Perl
modules, but that would require renaming too many files, so I don't see that
ever happening...)

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

--- Comment #5 from David Cook  ---
Created attachment 111669
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111669=edit
Bug 26683: Load localized Net::Z3950::RPN::* modules

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

--- Comment #4 from David Cook  ---
C4::Barcodes::ValueBuilder test plan:
1. Iterate through the "autoBarcode" options
2. For each autoBarcode option, try generating a barcode at
http://localhost:8081/cgi-bin/koha/cataloguing/additem.pl?biblionumber=29=scs_1602720764348

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

--- Comment #3 from David Cook  ---
Created attachment 111667
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111667=edit
Bug 26683: Decompose Koha::Z3950Responder::RPN into Net::Z3950::RPN::* packages

This patch reverts the usage of Net::Z3950::SimpleServer's
samples/render-search.pl example, and instead creates modified packages
which will be loaded before the standard ones due to these being located
higher up in the @INC path.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

David Cook  changed:

   What|Removed |Added

 Attachment #111657|0   |1
is obsolete||

--- Comment #2 from David Cook  ---
Created attachment 111666
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111666=edit
Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 26683] Fix Modules::RequireFilenameMatchesPackage Perl Critic Violations

2020-10-14 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26683

--- Comment #1 from David Cook  ---
Created attachment 111657
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111657=edit
Bug 26683: Decompose C4::Barcodes::ValueBuilder to make Perl critic happy

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/