[Koha-bugs] [Bug 22407] OMNIBUS: Use DBIC relations to fetch related object rather than searching for the object

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22407

--- Comment #15 from David Cook  ---
(In reply to Julian Maurice from comment #14)
> What is there to check in this particular situation ? _new_from_dbic either
> dies or return a blessed object. If it dies then Koha::Patron::guarantor
> should die too, if it returns a blessed object, then there is nothing to
> check.

That's an interesting perspective. 

If it dies, it's probably going to bring down that entire Plack worker. That
seems problematic to me. We could try to trap the error in the caller, but in
practice that is rare in Koha. That being said, even if it doesn't die, we need
to handle a non-fatal exception too. Maybe it is better to die in the grand
scheme of things, as it'll more easily show code bugs and bad data.

If it doesn't die, what happens if you want to add a check to that object?
Maybe you only want to return the guarantor under certain conditions. It
probably makes sense to add that check here rather than in every caller of this
method. To do that, you'd have to refactor to use the syntax I described
anyway. Or maybe you're doing some debugging because the guarantor being
returned isn't correct. You can't add any logging in this code unless you
refactor to the syntax I described.

-- 
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 23653] Plack fails when http://swagger.io/v2/schema.json is unavailable and schema cache missing

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23653

--- Comment #61 from David Cook  ---
(In reply to Julian Maurice from comment #59)
> Failing QA to get an answer to comment 49

Yeah that's fair. I'll put it on my TODO list.

-- 
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 25950] REMOTE_ADDR set to null if client_ip in X-Forwarded-For matches a koha_trusted_proxies value

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25950

--- Comment #7 from David Cook  ---
Thanks, Didier!

For what it's worth, I have this running locally in production now, and it's
working well.

-- 
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 25920] Add liblocale-codes-perl package to fix ubuntu-stable (focal)

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25920

Mason James  changed:

   What|Removed |Added

   Severity|major   |blocker

-- 
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 25920] Add liblocale-codes-perl package to fix ubuntu-stable (focal)

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25920

Mason James  changed:

   What|Removed |Added

 CC||alei...@catalyst.net.nz,
   ||lu...@bywatersolutions.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 25920] Add liblocale-codes-perl package to fix ubuntu-stable (focal)

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25920

--- Comment #6 from Mason James  ---
(In reply to Mason James from comment #5)
> (In reply to Mason James from comment #0)
> > This bug tracks the task of adding the liblocale-codes-perl package, to fix
> > ubuntu-stable (focal)
> > 
> > looks like perl-5.30 on debian needs liblocale-codes-perl
> 
> 
> hmm, problem exists on debian 11 too

this seems to stop installation at the 'web installer' stage

-- 
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 20988] Internationalization: wrap all translatable text inside t() calls

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20988

Victor Grousset/tuxayo  changed:

   What|Removed |Added

 CC||vic...@tuxayo.net

--- Comment #52 from Victor Grousset/tuxayo  ---
+1 for proof-of-concept

Maybe also one template at a time, I don't know. Maybe beginning with the
proof-of-concepts and see if it's necessary to keep doing one by one?

-- 
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 25552] Add missing Claims Returned option to MarkLostItemsAsReturned

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25552

Kelly McElligott  changed:

   What|Removed |Added

 CC||ke...@bywatersolutions.com
 Status|Needs Signoff   |Failed QA

--- Comment #12 from Kelly McElligott  ---
Hi,
It would appear that the item does not remove until the page reloads? I would
encourage to reload during this transaction so the staff can see it come off of
the account.

-- 
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 25971] Allow arbitrary database field as runtime parameter

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25971

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 CC||n...@bywatersolutions.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 25971] Allow arbitrary database field as runtime parameter

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25971

George Williams (NEKLS)  changed:

   What|Removed |Added

 CC||geo...@nekls.org

--- Comment #1 from George Williams (NEKLS)  ---
Another awesome idea.  I'd definitely like to see this implemented.

-- 
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 25971] New: Allow arbitrary database field as runtime parameter

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25971

Bug ID: 25971
   Summary: Allow arbitrary database field as runtime parameter
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Reports
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: and...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org

Yet another runtime parameter idea, how about allowing a syntax like "SELECT *
from accountlines WHERE debit_type_code like <>"

So Koha then builds the parameter list based on a "select distinct code from
account_debit_types"

To make it even nicer, we could allow "<>" so we use the
code as our value but display the corresponding description.

-- 
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 25173] Create list of parameters to choose from via a query

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25173

--- Comment #2 from Andrew Fuerste-Henry  ---
Kicking this around a little more, would it be easier or cleaner to make the
parameter point at a saved report? So, for your example, you'd make one report
that's "SELECT shelfname FROM virtualshelves" and then in your second report
say "WHERE shelfname LIKE <>"

-- 
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 25970] Make account_debit_types and account_credit_types available as runtime parameters

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25970

--- Comment #2 from Andrew Fuerste-Henry  ---
(In reply to George Williams (NEKLS) from comment #1)
> This is an outstanding idea.

Well, I mean, it was yours :)

-- 
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 25273] Elasticsearch Authority matching is returning too many results

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25273

--- Comment #14 from Nick Clemens  ---
(In reply to Julian Maurice from comment #12)
> Hi Nick,
> 
> Thanks for the explanation, it is much more clear to me now. The change
> makes sense and the patch works as expected.
> 
> However I would like to see some changes in the patch before validating it:
> 
> 1)
> 
> -ModZebra( $authid, 'specialUpdate', 'authorityserver', $record );
> +ModZebra( $authid, 'specialUpdate', 'authorityserver', { record =>
> $record, authtypecode => $authtypecode } );
> 
> This change is confusing. In ModZebra we now have a $record variable which
> is a hash that contain a 'record' key. Even with this simple patch I had to
> ask myself several times « what's this $record variable I'm looking at ? The
> hash or the MARC::Record ? ».
> Look at this line for instance:
> 
> +$record = $record->{record};
> 
> At some point in the subroutine, $record was a hasref, now it's a
> MARC::Record. This is the kind of things that make code hard to read, and
> make it easier for bugs to appear.
> 
> And I think it is not needed to pass the authtypecode to ModZebra, since it
> can be obtained from the MARC::Record.
> 
> 2)
> 
> -unless ($record) {
> +if ($record) {
> +$indexer->update_index_background( [$biblionumber],
> [$record] );
> +} else {
>  $record = GetMarcBiblio({
>  biblionumber => $biblionumber,
>  embed_items  => 1 });
> +$indexer->update_index_background( [$biblionumber], [{
> record => $record }] );
>  }
> -my $records = [$record];
> -$indexer->update_index_background( [$biblionumber], [$record] );
> 
> I think it was easier to read before : unless there is a record, fetch it;
> in all cases call update_index_background
> Now it's : if there is a record, call update_index_background, otherwise
> fetch the record and call update_index_background.
> This change was not needed, so why ? :)
> 
> 3)
> 
> -my $id = $record->id // $record->authid;
> +my $id = $record->id;
> 
> Again this change is not needed, but this time it causes a bug.
> Try this : misc/search_tools/rebuild_elasticsearch.pl -a -ai X (replace X by
> an existing authid)

1 - Yes, was trying to save a call to GuessAuthType, reverted
2 - This was a result of above, turning record into a hash if needed, undone
3 - It made sense at the time, but yes, it breaks, removed

-- 
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 25273] Elasticsearch Authority matching is returning too many results

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25273

Nick Clemens  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

-- 
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 25273] Elasticsearch Authority matching is returning too many results

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25273

--- Comment #13 from Nick Clemens  ---
Created attachment 106759
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106759=edit
Bug 25273: (follow-up)

-- 
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 25969] Checking in a found hold at a different branch then confirming the hold causes internal server error

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25969

Kyle M Hall  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com
   |ity.org |
   Severity|enhancement |normal

-- 
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 25969] Checking in a found hold at a different branch then confirming the hold causes internal server error

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25969

--- Comment #2 from Kyle M Hall  ---
Created attachment 106758
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106758=edit
Bug 25969: Checking in a found hold at a different branch then confirming the
hold causes internal server error

If a record level hold is filled and waiting at Library A, and the item
is checked in at Library B, and an attempt is made to re-fill the hold
with the item, Koha will return an ISE.

Test Plan:
1) Place a hold for Library A, at Library A, for pickup at Library A
2) Check in the item at Library A and fill the hold so it is waiting at Library
A
3) Log in as Library B, check in the same barcode
4) Note the request to fill the hold with the item
5) Choose to fill the hold
6) Note you get an internal server error
7) Apply this patch
8) Restart all the things!
9) Repeat steps 1-5
10) No ISE!

-- 
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 25969] Checking in a found hold at a different branch then confirming the hold causes internal server error

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25969

Kyle M Hall  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
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 25970] Make account_debit_types and account_credit_types available as runtime parameters

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25970

George Williams (NEKLS)  changed:

   What|Removed |Added

 CC||geo...@nekls.org

--- Comment #1 from George Williams (NEKLS)  ---
This is an outstanding idea.

-- 
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 25970] New: Make account_debit_types and account_credit_types available as runtime parameters

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25970

Bug ID: 25970
   Summary: Make account_debit_types and account_credit_types
available as runtime parameters
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Reports
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: and...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org

Accountlines reports would be easier to write if the new account_credit_types
and account_debit_types tables were available as runtime parameters.

-- 
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 25969] Checking in a found hold at a different branch then confirming the hold causes internal server error

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25969

--- Comment #1 from Kyle M Hall  ---
So the root cause is that we are not passing the itemnumber *for the scanned
barcode* back to the form, making itemnumber undef, so the Item object is
empty, so calling the biblio method causes an ISE.

-- 
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 25969] New: Checking in a found hold at a different branch then confirming the hold causes internal server error

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25969

Bug ID: 25969
   Summary: Checking in a found hold at a different branch then
confirming the hold causes internal server error
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Circulation
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: k...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org
CC: gmcha...@gmail.com, kyle.m.h...@gmail.com

If a record level hold is filled and waiting at Library A, and the item is
checked in at Library B, AddReturn() calls RevertWaitingStatus(), which unsets
found, and itemnumber ( if the hold was record level ). AddReturn() then retuns
the message 'ResFound', which triggers the hold fulfillment dialog, which
passes the reserve_id back to circulation.pl.

Then, circulation.pl attempts to get the biblio via the itemnumber, which the
hold doesn't have, causing an ISE.

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

--- Comment #118 from Maryse Simard  ---
I rebased the patches and merged the ones which were signed-off. By adding the
atomic update changes to it, it took care of the first qa script error. For the
second error, I directly altered the schema update patch to correct it.

The status has been set back to signed off, as requested.

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95856|0   |1
is obsolete||

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95860|0   |1
is obsolete||

--- Comment #117 from Maryse Simard  ---
Created attachment 106757
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106757=edit
Bug 12446: (follow-up) Update for Bug 17168

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95855|0   |1
is obsolete||

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95854|0   |1
is obsolete||

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95857|0   |1
is obsolete||

--- Comment #116 from Maryse Simard  ---
Created attachment 106756
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106756=edit
Bug 12446: Schema update Category.pm

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95859|0   |1
is obsolete||

--- Comment #115 from Maryse Simard  ---
Created attachment 106755
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106755=edit
Bug 12446: (follow-up) Code cleaning

Some variable name change,
remove unused code
and other small fixes.

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95858|0   |1
is obsolete||

--- Comment #114 from Maryse Simard  ---
Created attachment 106754
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106754=edit
Bug 12446: (follow-up) Remove mandatory transfered values

With the new system preference AdditionalGuarantorField, one can add
data to be transfered to the guarantee, but cannot remove default ones.

This patch renames the syspref to GuarantorFields and set hardcoded
default values as the preference's default.

To test:
1. After database update, check that the default value for the syspref
GuarantorFields contains these columns:
streetnumber|address|address2|city|state|zipcode|country|
branchcode|phonepro|mobile|email|emailpro|fax
2. Set the value as empty
3. Try adding a guarantor to a patron (from the guarantor's page or the
guarantee's): no values should be imported.
4. Check that it still works with different values for the preference.

-- 
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 12446] Enable an adult to have a guarantor

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

Maryse Simard  changed:

   What|Removed |Added

  Attachment #95853|0   |1
is obsolete||

--- Comment #113 from Maryse Simard  ---
Created attachment 106753
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106753=edit
Bug 12446: Enable an adult to have a guarantor

Conflicts have been fixed. Also attributes in system preference :
AdditionalGuarantorField are now displayed correctly in all scenarios.

I have rebased every patch.
This includes all the features from the previous patches.

Removed the dollar signs and stopped using C4::Category

On the patron category page, there is a new field to specify whether patrons
can be guarantee or not.
The atomic update script adds this new field and sets it to true for categories
of type "C" and "P".

1) Apply the patch
2) Run updatedatabase.pl to add AdditionalGuarantorField to preferences and
canbeguarantee to categories.
3) Go to patron category administration, edit a category and change the value
of Can Be Guarantee to yes (for Child and Professional it should have been set
by default)
4) See below to test various scenarios

- Enable an adult to have a guarantor:
0.0) Create or edit a patron category of type 'A'.
0.1) Set the "Can be guarantee" field to "Yes" and save.
1) Select an adult patron -> details tab;
2) Click edit button;
3) validate "guarantor information" fieldset;

- Shows guarantees' fines in the guarantor's page:
4) Click Set to patron button and select a guarantor;
5) Generate a fine;
6) Select guarantor patron -> details tab;
7) Validate guarantee's fine information.

- Transfer some guarantor's information to the guarantee while adding a
guarantee:
0) Select an adult patron;
1) Insert all information;
2) Click Add guarantee bouton;
3) Validate Garantor information/Main address/Contact fieldsets filled;

- Transfer some guarantor's information to the guarantee while creating a new
patron
0) Select an adult patron;
1) Insert all information;
2) Add a new adult patron;
3) Click "Set to patron" button;
4) Select the adult patron from 0);
5) Validate Garantor information/Main address/Contact fieldsets filled;

- Transfer guarantor's alternate address/contact to the guarantee while
creating a new patron:
0) Search AdditionalGuarantorField preference;
1) Insert some additional database columns from alternate address/contact to be
transferred from guarantor;
2) Select an adult patron;
3) Insert all information;
4) Add a new adult patron;
5) Click "Set to patron" button;
6) Select the adult patron from 0);
7) Validate Garantor information/Main address/Contact fieldsets filled;
8) Validate Garantor additional alternate address/contact filled;

- Transfer guarantor's alternate address/contact to the guarantee while adding
a guarantee:
0) Search AdditionalGuarantorField preference;
1) Insert some additional database columns from alternate address/contact to be
transferred from guarantor;
3) Select an adult patron;
4) Insert all information;
5) Click Add guarantee bouton;
6) Validate Garantor information/Main address/Contact fieldsets filled;
7) Validate Garantor additional alternate address/contact filled;

Signed-off-by: Hayley Mapley 

-- 
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 25964] Data loss possible when items are modified

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25964

Victor Grousset/tuxayo  changed:

   What|Removed |Added

 CC||vic...@tuxayo.net

--- Comment #5 from Victor Grousset/tuxayo  ---
My test plan was to apply this patch on top of bug 20271 and run:
prove t/db_dependent/Items.t
to confirm that it's fixed. See bug 20271 comment 243.

And then apply it to master, run the same test to confirm that it still passes.
And attach the signed off patch here (the on-top-of-master one)


Is that a good test? If so we can mark this ticket as signed off

-- 
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 25964] Data loss possible when items are modified

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25964

Victor Grousset/tuxayo  changed:

   What|Removed |Added

 Attachment #106695|0   |1
is obsolete||

--- Comment #4 from Victor Grousset/tuxayo  ---
Created attachment 106752
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106752=edit
Bug 25964: Prevent data loss when editing items from a MARC record

Coming from:
  Bug 23463: Use new method Koha::Object->set_or_blank

We have DB fields that are not mapped with MARC fields, for instance paidfor.
They are not handled correctly.

In ModItemFromMarc, we get a MARC record in parameter and update the item in
DB. But we are loosing the fields that are not in the MARC record

Signed-off-by: Victor Grousset/tuxayo 

-- 
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 24958] Remember last selected tab in SQL reports

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24958

Barbara Johnson  changed:

   What|Removed |Added

 CC||barbara.johnson@bedfordtx.g
   ||ov

-- 
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 25968] New: Make logs sort by date descending as a default

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25968

Bug ID: 25968
   Summary: Make logs sort by date descending as a default
 Change sponsored?: ---
   Product: Koha
   Version: 19.11
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Tools
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: barbara.john...@bedfordtx.gov
QA Contact: testo...@bugs.koha-community.org

It would be helpful to have the logs sort by date descending so that you can
view the newest items at the top of the page rather than having to scroll to
the bottom.

-- 
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 25907] When cancelling a waiting hold on returns.pl, looks for new hold to fill without rescanning barcode

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25907

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106453|0   |1
is obsolete||

--- Comment #3 from Andrew Fuerste-Henry  ---
Created attachment 106751
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106751=edit
Bug 25907: When cancelling a waiting hold on returns.pl, looks for new hold to
fill without rescanning barcode

Right now, after canceling a hold on the checkin screen, a librarian
needs to rescan the item barcode to check and trap the next hold. This
is an extra unnecessary step.

Test Plan:
1) Place two or more holds on a record
2) Check in the item
3) Confirm the hold
4) Check in the item again
5) Cancel the hold
6) Koha should now display the modal asking to confirm the next hold
   without the need to rescan the barcode!

Signed-off-by: Owen Leonard 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106690|0   |1
is obsolete||

--- Comment #72 from Andrew Fuerste-Henry  ---
Created attachment 106750
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106750=edit
Bug 24860: Update existing Schema files [DO NOT PUSH]

Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106689|0   |1
is obsolete||

--- Comment #71 from Andrew Fuerste-Henry  ---
Created attachment 106749
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106749=edit
Bug 24860: Add hold's volume to holds table on circulation.pl and moremember.pl

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106686|0   |1
is obsolete||

--- Comment #68 from Andrew Fuerste-Henry  ---
Created attachment 106746
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106746=edit
Bug 24860: Add ability to select a volume when placing a hold - OPAC

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106688|0   |1
is obsolete||

--- Comment #70 from Andrew Fuerste-Henry  ---
Created attachment 106748
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106748=edit
Bug 24860: Skip non-matching volume holds in HoldsQueue

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106687|0   |1
is obsolete||

--- Comment #69 from Andrew Fuerste-Henry  ---
Created attachment 106747
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106747=edit
Bug 24860: Skip non-matching volume holds in CheckReserves

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106685|0   |1
is obsolete||

--- Comment #67 from Andrew Fuerste-Henry  ---
Created attachment 106745
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106745=edit
Bug 24860: Add ability to select a volume when placing a hold

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106684|0   |1
is obsolete||

--- Comment #66 from Andrew Fuerste-Henry  ---
Created attachment 106744
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106744=edit
Bug 24860: Implement reserves.volume_id

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104964|0   |1
is obsolete||

--- Comment #109 from ByWater Sandboxes  ---
Created attachment 106740
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106740=edit
Bug 24857: Copy the volume to the item's enumchron

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104966|0   |1
is obsolete||

--- Comment #110 from ByWater Sandboxes  ---
Created attachment 106741
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106741=edit
Bug 24857: Add user permission for volumes management

Signed-off-by: Rebecca  Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106682|0   |1
is obsolete||

--- Comment #64 from Andrew Fuerste-Henry  ---
Created attachment 106742
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106742=edit
Bug 24860: Add reserves.volume_id

This feature builds upon bug 24857 and allows placing holds that target a
specific volume of a record. It is patterned after the feature that allows
limiting hold selection by itemtype ( AllowHoldItemTypeSelection ).

Test Plan:
1) Apply bug 24857 and this bug's patches
2) Run updatedatabase.pl
3) Restart all the things!
4) Enable the sysprefs EnableVolumes and EnableVolumeHolds
5) Create a record, items and volumes, with each item being assigned to
a volume
6) Place a hold for a patron
7) Note the new selector to choose a volume in addition to record and item
level holds
8) Place a volume level hold
9) Check in an item from the record that is not part of that volume
10) Note the hold is not trapped for that item
11) Check in an item from the record that *is* part of that volume
12) Note the hold is trapped for that item
13) Place a volume level hold for another item
14) Run the holds queue builder
15) Note the holds queue targets only items from that volume
16) Check out all the items of that volume to other patrosn
17) Re-run the holds queue builder
18) Note the holds queue no longer has a line for that hold,
as all items that could fill that hold are now unavailable

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24860] Add ability to place volume level holds

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24860

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #106683|0   |1
is obsolete||

--- Comment #65 from Andrew Fuerste-Henry  ---
Created attachment 106743
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106743=edit
Bug 24860: Add new syspref EnableVolumeHolds

Signed-off-by: Andrew Fuerste-Henry 
Signed-off-by: Your Full Name 

Signed-off-by: Rebecca Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104959|0   |1
is obsolete||

--- Comment #104 from ByWater Sandboxes  ---
Created attachment 106735
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106735=edit
Bug 24857: Add volume management to detail.pl

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104963|0   |1
is obsolete||

--- Comment #108 from ByWater Sandboxes  ---
Created attachment 106739
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106739=edit
Bug 24857: Update existing Schema files [DO NOT PUSH]

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104960|0   |1
is obsolete||

--- Comment #105 from ByWater Sandboxes  ---
Created attachment 106736
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106736=edit
Bug 24857: Add ability to attach newly cataloged items to a volume

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104961|0   |1
is obsolete||

--- Comment #106 from ByWater Sandboxes  ---
Created attachment 106737
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106737=edit
Bug 24857: Add ability to manage item's volumes from detail.pl

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104962|0   |1
is obsolete||

--- Comment #107 from ByWater Sandboxes  ---
Created attachment 106738
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106738=edit
Bug 24857: Delete a volume when the last item on that volume is deleted

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104958|0   |1
is obsolete||

--- Comment #103 from ByWater Sandboxes  ---
Created attachment 106734
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106734=edit
Bug 24857: Implement volume related controllers

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104956|0   |1
is obsolete||

--- Comment #101 from ByWater Sandboxes  ---
Created attachment 106732
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106732=edit
Bug 24857: API spec

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104957|0   |1
is obsolete||

--- Comment #102 from ByWater Sandboxes  ---
Created attachment 106733
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106733=edit
Bug 24857: Koha::Biblio::Volume methods

This patch introduces the following methods:
- add_item
- del_item
- items
- to_api_mapping
- store (overloaded)

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104953|0   |1
is obsolete||

--- Comment #98 from ByWater Sandboxes  ---
Created attachment 106729
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106729=edit
Bug 24857: Add new Schema files

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104955|0   |1
is obsolete||

--- Comment #100 from ByWater Sandboxes  ---
Created attachment 106731
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106731=edit
Bug 24857: Add Object Methods

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104954|0   |1
is obsolete||

--- Comment #99 from ByWater Sandboxes  ---
Created attachment 106730
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106730=edit
Bug 24857: Add Koha Object(s)

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 24857] Add ability to create "Volumes" for records, and assign items to volumes

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857

ByWater Sandboxes  changed:

   What|Removed |Added

 Attachment #104952|0   |1
is obsolete||

--- Comment #97 from ByWater Sandboxes  ---
Created attachment 106728
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106728=edit
Bug 24857: Update Database

We have a partner that needs Volume Level Holds.
This is a feature that some ILSs have that allows the creation of volumes at
the bibliographic level,
with items being optionally assigned to a given volume. This facilitates the
ability to place holds
at a volume level, rather than on a specific item or any available item on a
record.

Test Plan:
1) Apply the patches for this bug
2) Run updatedatabase.pl
3) Restart all the things!
4) Enable the new syspref EnableVolumes
5) Browse to detail.pl for a record with items
6) Note the new volumes tab
7) Test creating, editing and deleting volumes
8) On the Holdings tab, select one or more items using the checkboxes
8) Note new selection options for setting a volume for items,
   and for clearing a volume for items.
9) Test adding and clearing the volume set for items

Signed-off-by: Andrew Fuerste-Henry 

Signed-off-by: Rebecca  Coert 

-- 
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 23816] Allow to have different password strength and length settings for different patron categories

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23816

--- Comment #22 from Martin Renvoize  ---
Another rebase request ;)... two QAers interested here.. grab them whilst it's
hot ;)

-- 
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 25025] Drag-and-drop cover image upload

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25025

Heather  changed:

   What|Removed |Added

 CC||heather_hernan...@nps.gov

-- 
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 25408] CanBookBeReserved & CanItemBeReserved should check "opacitemholds" policy

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25408

--- Comment #19 from Arthur Suzuki  ---
yay, thanks! :)

-- 
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 24167] We should support installation on Debian 10 [OMNIBUS]

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24167
Bug 24167 depends on bug 25633, which changed state.

Bug 25633 Summary: Update debian/control.ini file for 20.05 release cycle
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25633

   What|Removed |Added

 Status|Pushed to oldstable |RESOLVED
 Resolution|--- |FIXED

-- 
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 25633] Update debian/control.ini file for 20.05 release cycle

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25633

Mason James  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|Pushed to oldstable |RESOLVED

-- 
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 25591] Update list-deps for Debian 10 and Ubuntu 20.04

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25591

Mason James  changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

-- 
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 24167] We should support installation on Debian 10 [OMNIBUS]

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24167
Bug 24167 depends on bug 25591, which changed state.

Bug 25591 Summary: Update list-deps for Debian 10 and Ubuntu 20.04
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25591

   What|Removed |Added

 Status|Pushed to oldstable |RESOLVED
 Resolution|--- |FIXED

-- 
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 25640] [OMNIBUS] Ubuntu 20.04 support

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25640
Bug 25640 depends on bug 25591, which changed state.

Bug 25591 Summary: Update list-deps for Debian 10 and Ubuntu 20.04
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25591

   What|Removed |Added

 Status|Pushed to oldstable |RESOLVED
 Resolution|--- |FIXED

-- 
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 25591] Update list-deps for Debian 10 and Ubuntu 20.04

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25591

Mason James  changed:

   What|Removed |Added

 Status|Pushed to oldstable |RESOLVED
 Resolution|--- |FIXED

--- Comment #36 from Mason James  ---
(In reply to Aleisha Amohia from comment #34)
> backported to 19.11.x for 19.11.07, as recommended for Bug 22522

this has been resolved now, closing

-- 
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 24874] Printing is broken on opac-results.pl page

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24874

Mason James  changed:

   What|Removed |Added

 Status|Pushed to stable|RESOLVED
 Resolution|--- |FIXED

--- Comment #29 from Mason James  ---
(In reply to Lucas Gass from comment #28)
> backported to 19.05.x for 19.05.10

this has been resolved now

-- 
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 23128] Missing Net::OAuth2::AuthorizationServer dependency

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23128

Mason James  changed:

   What|Removed |Added

   Assignee|m...@kohaaloha.com   |tomasco...@gmail.com

--- Comment #14 from Mason James  ---
(In reply to Jonathan Druart from comment #6)
> Wiki pages need to be updated:
> 
> https://wiki.koha-community.org/wiki/Koha_on_Debian#Debian_Packages_on_Koha
> 
> https://wiki.koha-community.org/wiki/Koha_on_ubuntu_-_packages#Introduction
> 
> 
> Also should not we inform the ML?

i've updated the following wiki page...
https://wiki.koha-community.org/wiki/Koha_on_Debian

and made a request to merge the following page...
 https://wiki.koha-community.org/wiki/Koha_on_ubuntu_-_packages


i think this issue is now resolved :0)

-- 
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 23128] Missing Net::OAuth2::AuthorizationServer dependency

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23128

--- Comment #13 from Mason James  ---
(In reply to Mason James from comment #12)
> (In reply to David Cook from comment #11)
> > (In reply to Jonathan Druart from comment #10)
> > > What is needed here?
> > 
> > I imagine Mason probably just needs to remove libcryptx-perl from
> > http://debian.koha-community.org/koha/pool/main/libc/libcryptx-perl/
> 
> i've updated the libcryptx-perl package for 20.05, built for deb9
>  20.05|main|amd64: libcryptx-perl 0.063-1~bpo9+1
> 
> the libcryptx-perl package for 19.11 is still built for deb8
>  19.11|main|amd64: libcryptx-perl 0.059-1~koha2 
> 
> i have not updated the 19.11 libcryptx-perl package yet, as i dont want to
> break 19.11/deb8 support for those who are still running this combination

the libcryptx-perl package has now been updated for 20.05 and 19.11 releases

-- 
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 21373] Checkout not possible when biblio level hold but other items could satisfy it

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21373

Joonas Kylmälä  changed:

   What|Removed |Added

  Attachment #79378|0   |1
is obsolete||

-- 
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 21373] Checkout not possible when biblio level hold but other items could satisfy it

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21373

Joonas Kylmälä  changed:

   What|Removed |Added

  Attachment #79377|0   |1
is obsolete||

-- 
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 25950] REMOTE_ADDR set to null if client_ip in X-Forwarded-For matches a koha_trusted_proxies value

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25950

didier  changed:

   What|Removed |Added

 Attachment #106647|0   |1
is obsolete||

--- Comment #6 from didier  ---
Created attachment 106722
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106722=edit
Bug 25950: Remove  in X-Forwarded-For from proxy tests

This patch removes the  ip address in the X-Forwarded-For
header from being tested against koha_trusted_proxies.

Without this patch, REMOTE_ADDR will be set to null, if the 
ip address matches against koha_trusted_proxies.

To Test:
1. Run the unit test t/Koha/Middleware/RealIP.t

Signed-off-by: Didier Gautheron 

-- 
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 25950] REMOTE_ADDR set to null if client_ip in X-Forwarded-For matches a koha_trusted_proxies value

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25950

didier  changed:

   What|Removed |Added

   Patch complexity|Small patch |Trivial patch
 Status|Needs Signoff   |Signed Off

-- 
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 15851] "Analytics: Show analytics" showing regardless of whether they have analytics

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15851

--- Comment #40 from Tomás Cohen Arazi  ---
(In reply to Hugo Agud from comment #39)
> I hae tested the patch and it works fine! however the code applied to opac
> is not applied to staff xslt and it shows the links always, even when there
> is not analytics.

My idea was to have feedback on the approach, and then move to other areas. The
'how to find analytics' discussion (i.e. the queries that are done) should be
moved to another bug, for example.

Thanks for testing, and I will add the intranet counterpart for you to SO ASAP.

-- 
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 19889] LocalHoldsPriority needs exclusions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19889

--- Comment #22 from Christopher Brannon  ---
(In reply to Tomás Cohen Arazi from comment #21)

> So the result is similar, but ugly-ish. And involves an error prone
> free-text field with pipe-separated values. So I don't see any advantage on
> it. If we were to expose this condition on the API also, we would need to
> add methods to extract the boolean with this kind of comparison, instead of
> just reading what's on the category table.

What if it wasn't a free-text field?  What if we did something similar to what
is being done in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22844?  Select the
categories from a list rather than free typing?

I'd prefer to keep these settings together, but if you think it makes more
sense to put exclusions in the patron categories, then that would be
acceptable.  It works.  My biggest concern is trying to debug behavior.  It's
one thing to have the items with their own exclusions in the record.  But
trying to figure out why it excluding items in some cases and not others
because of a patron category exclusion might be more challenging.  If there
were some indicator when the exclusion is skipped somewhere, then I would have
no problem with it.  If it is logged, then it would be easy to trace.

I'm just trying to make sure that our enhancement doesn't become a headache for
those using it to try and understand system behavior.  Circulation rules alone
are hard to follow when dealing with multiple branches.

-- 
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 25958] Allow LongOverdue cron to include/exclude specified lost values

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25958

--- Comment #2 from Andrew Fuerste-Henry  ---
Yes, absolutely! We've already got several options in the cron that can only be
set at the command line. It would be good to move toward making those options
all configurable from the staff client so those sysprefs are an accurate
representation of the cron's behavior.

-- 
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 25965] Create SIP2 client daemon with HTTPS API endpoint

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25965

--- Comment #1 from Lari Strand  ---
Or is there a way to leave the sip server socket connection open for future
messages, in a way that it still allows me to catch the sip server's response,
and when another message is being sent in the same address, check if the
connection already exists and use that instead of creating a new socket
connection, directly in the module?

-- 
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 25671] Install 1 Koha plugin for X Koha instances using the CLI

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25671

--- Comment #7 from David Cook  ---
The code is still WIP, but I probably won't get back to this for a week or two. 

Looking forward to getting back to this though.

-- 
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 25967] New: Plugin upgrade should remove existing code before installing new code

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25967

Bug ID: 25967
   Summary: Plugin upgrade should remove existing code before
installing new code
 Change sponsored?: ---
   Product: Koha
   Version: unspecified
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Plugin architecture
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: dc...@prosentient.com.au
QA Contact: testo...@bugs.koha-community.org

At the moment, upgrading a plugin really means overwriting the existing plugin
files with new files.

This is fine for files that are added or modified, but it won't remove files
that have been deleted.

Of course, we don't necessarily know what files to delete when we're upgrading
a plugin, because we don't track the files, and the files can be a bit messy.

For instance, "/var/lib/koha/kohadev/plugins/Koha/Plugin/Com/ByWaterSolutions"
is the root for both KitchenSink and CoverFlow.

This is probably an infeasible idea due to existing installs, but just pointing
out the flaw in the plugin architecture.

-- 
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 22873] C4::SIP::ILS::Transation::FeePayment->pay $disallow_overpayment does nothing

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22873

Kyle M Hall  changed:

   What|Removed |Added

 CC||k...@bywatersolutions.com

--- Comment #3 from Kyle M Hall  ---
It is used in a SIP option. Take a look at etc/SIPconfig.xml in master.

-- 
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 25671] Install 1 Koha plugin for X Koha instances using the CLI

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25671

--- Comment #6 from David Cook  ---
The following would work too but it wouldn't be efficient with a KPZ file or a
HTTP(S) URL, because it would be re-downloading and/or re-unzipping every time:

koha-foreach perl misc/koha_plugin.pl --install
https://github.com/bywatersolutions/koha-plugin-coverflow/releases/download/v2.4.35/koha-coverflow-plugin-v2.4.35.kpz

koha-foreach perl misc/koha_plugin.pl --uninstall
Koha::Plugin::Com::ByWaterSolutions::CoverFlow

That said, it could be done with an unzipped plugin. That could be interesting.
You wouldn't even need a KPZ file. You could point it right at a git checkout
for instance. (That could actually be really useful from a Koha plugin
development point of view as well.)

--

Perhaps I should rename misc/koha_plugin.pl to misc/koha_plugin_manager.pl. 

--

A goal of mine is to use this management feature to actually install a Koha
Plugin for all my many libraries which is itself a Web-based Koha Plugin
manager (inspired by the "Game Package Registry for Unity release" from Google
for the Unity IDE). That Web-based Koha Plugin manager will then easily let
Koha intranet users decide which plugins they want to install, but it'll only
include a list of curated plugins.

Alternatively, I might use this management feature to install Koha plugins
which used to be local customizations, and popular third-party plugins like
Bywater's CoverFlow. 

Ultimately, I think we need a tool like this for dealing with plugins on any
kind of scale that becomes manually unmanageable (e.g. 40 instances, 80
instances, etc.)

-- 
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 25671] Install 1 Koha plugin for X Koha instances using the CLI

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25671

--- Comment #5 from David Cook  ---
I'm also going to make it so that you can provide a KPZ file or an unzipped
KPZ, so misc/koha_plugin.pl will be very flexible in terms of inputs.

The Debian script koha-plugin will download plugins over HTTP/HTTPS, and then
unpack downloaded/local KPZ files into a staging directory.

The staging directory will then be copied to each instance specified.

It should be pretty sweet.

I want to add enable/disable options as well at some point, but that's not
critical yet.

-- 
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 25966] New: Renewal of lost item generates overdue fine

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25966

Bug ID: 25966
   Summary: Renewal of lost item generates overdue fine
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P5 - low
 Component: Circulation
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: and...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org
CC: gmcha...@gmail.com, kyle.m.h...@gmail.com

If an item is both overdue and lost, renewal of that item will generate fines.
This behavior is inconsistent with the behavior of the fines cron, which
explicitly excludes lost items from fines.
I expect there are libraries that prefer the current behavior but also
libraries that would rather not generate overdue fines on renewal of lost
items. So perhaps a system preference is in order? Do/Don't charge fines on
renewal of lost items?

-- 
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 25671] Install 1 Koha plugin for X Koha instances using the CLI

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25671

--- Comment #4 from David Cook  ---
koha-shell -c "perl misc/koha_plugin.pl --install
https://github.com/bywatersolutions/koha-plugin-coverflow/releases/download/v2.4.35/koha-coverflow-plugin-v2.4.35.kpz;
kohadev

koha-shell -c "perl misc/koha_plugin.pl --install
https://github.com/bywatersolutions/koha-plugin-kitchen-sink/releases/download/v2.1.39/koha-plugin-kitchen-sink-v2.1.39.kpz;
kohadev

koha-shell -c "perl misc/koha_plugin.pl --uninstall
Koha::Plugin::Com::ByWaterSolutions::CoverFlow" kohadev

koha-shell -c "perl misc/koha_plugin.pl --uninstall
Koha::Plugin::Com::ByWaterSolutions::KitchenSink" kohadev

-- 
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 25965] New: Create SIP2 client daemon with HTTPS API endpoint

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25965

Bug ID: 25965
   Summary: Create SIP2 client daemon with HTTPS API endpoint
 Change sponsored?: ---
   Product: Koha
   Version: unspecified
  Hardware: All
OS: All
Status: NEW
  Severity: new feature
  Priority: P5 - low
 Component: REST API
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: lmstr...@gmail.com

I am trying to pass messages to a sip server using a new endpoint I created in
swagger/openApi. This endpoint uses the guidelines described here:

http://biblstandard.dk/rfid/dk/rfid_sip2_over_https.htm

The idea is to get rid of the tunneling/VPN implementations that we use, and
use HTTPS instead to pass sip messages back and forth inside an XML. The Apache
server handles the HTTPS stuff so I don't have had to worry about that so far
when communicating with the REST api, hence the SipOHttp naming in my module.
At the moment I can send the XML POST messages formed like in the linked
guideline to the endpoint (XML wrapped SIP2 message to
.../api/v1/sipmessages/). The api forwards the body content of the request to
my module Koha::REST::V1::SipOHttp. The module validates and parses the
contents of the XML message, extracts the information about the sip server the
message is intended to (using the "login:" parameter inside the XML) and the
sip message itself (inside the  variable). Then the module
passes the sip message on to the sip server using IO:Socket::INET.

My problem is that the sip server closes the connection after this first SC
device login message has been passed and the response has been read back in my
module:

(from sip2.log)
.
[INFO] [undef]@10.0.3.217: OUTPUT MSG: '941' C4::SIP::Sip::write_msg
/home/koha/Koha/C4/SIP/Sip.pm (187) 
[DEBUG] SIPDEVICE@10.0.3.217: raw_transport: uname/inst: 'SIPDEVICE/SIPDEVICE'
C4::SIP::SIPServer::raw_transport /home/koha/Koha/C4/SIP/SIPServer.pm (174)
[DEBUG] SIPDEVICE@10.0.3.217: EOF returned on read
C4::SIP::SIPServer::read_request /home/koha/Koha/C4/SIP/SIPServer.pm (349) 
[INFO] SIPDEVICE@10.0.3.217: raw_transport: shutting down
C4::SIP::SIPServer::raw_transport /home/koha/Koha/C4/SIP/SIPServer.pm (178)

So I need a way to keep this socket connection open for other future messages.
Without touching the sip server's code itself, the only solution I can come up
with is a proxy server that handles and passes on the REST api's incoming
messages to sockets and keeps each invidual socket connection open to specific
instances of the sip servers (I suppose the sip server itself ends the
connections when needed so you wouldn't have to worry about it in the sender's
end).
Another solution would be to build a new api separate from the swagger/Openapi
one. Like an all-in-one solution that deals with the requests and keeps tabs on
the open socket connections.

Let me know if you have ideas regarding this.

Ps. I'm a trainee in Koha Suomi (Finland) so excuse me if the description of my
problem is a bit vague and lacks some correct terms used. I'm happy to fill in
any details I've left out.

-- 
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 25723] Improve efficiency of holiday calculation

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25723

Martin Renvoize  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

--- Comment #42 from Martin Renvoize  ---
Tested and signed off by Heart of England NHS Trust.

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #20 from Martin Renvoize  ---
Created attachment 106720
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106720=edit
Bug 25723: Remove tests for removed method

We removed the exception_holidays (and single_holidays) methods as part
of this patchset. As such, we should no longer test them ;)

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #18 from Martin Renvoize  ---
Created attachment 106718
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106718=edit
Bug 25723: Use the same code for single and exception holidays

This patch refactors is_exception_holiday and is_single_holiday to
utilise a single _holidays method which combines the logic of the
previous single_holidays and exception_holidays methods.

As Koha::Calendar is instantiated at a branch level, we also move split
the cache into branches too.

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #21 from Martin Renvoize  ---
Created attachment 106721
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106721=edit
Bug 25723: (QA follow-up) Silent POD warning

I didn't introduce the method that is warned about, but adding POD for
it seemed reasonable and will silence the QA Script warning.

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #17 from Martin Renvoize  ---
Created attachment 106717
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106717=edit
Bug 25723: Use a hasref for lookup

Rather than using an array of dates and iterating them, use a hashref
and do a direct lookup

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #19 from Martin Renvoize  ---
Created attachment 106719
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106719=edit
Bug 25723: Update cache flushing calls

This patch updates the previous single_holidays and exeption_holidays
cache flushing calls to match the new cache key structure of the updated
routines.

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #16 from Martin Renvoize  ---
Created attachment 106716
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106716=edit
Bug 25723: Drop DateTime::Set from cpanfile

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #15 from Martin Renvoize  ---
Created attachment 106715
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106715=edit
Bug 25723: (follow-up) Fix Unit Test

The unit test previously relied on the internals of exception_holidays.

Signed-off-by: Emma Perks 

-- 
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 14315] Slow checkouts, caused by many 'special_holidays' definitions

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14315

--- Comment #14 from Martin Renvoize  ---
Created attachment 106714
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106714=edit
Bug 25723: Drop DateTime::Set dependancy

This patch removes our use of DateTime::Set and replaces it with a quick
hashref implimentation

Signed-off-by: Emma Perks 

-- 
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 25671] Install 1 Koha plugin for X Koha instances using the CLI

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25671

--- Comment #3 from David Cook  ---
Turns out that the easier way to do this was to create "misc/koha_plugin.pl"
which will be called by koha-plugin...

koha-shell -c "perl misc/koha_plugin.pl --uninstall
Koha::Plugin::Com::ByWaterSolutions::CoverFlow" kohadev

-- 
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 25752] Current directory not kept when using koha-shell

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25752

--- Comment #15 from Julian Maurice  ---
(In reply to Jonathan Druart from comment #13)
> root@615331e1369d:koha(master)$ cd /tmp
> root@615331e1369d:tmp$ koha-shell -c pwd kohadev
> /tmp
> root@615331e1369d:tmp$ koha-shell kohadev
> kohadev-koha@615331e1369d:/tmp$ pwd
> /tmp

Sorry, I misread the patch. The problem is with the --login option:

# cwd is /home/koha/env/master/src
# user test-koha created by hand so $HOME is different than with package
install
% debian/scripts/koha-shell --login -c pwd test
/home/test-koha
% debian/scripts/koha-shell --login test   
% pwd
/home/koha/env/master/src

Compare with sudo:

# cwd is /home/koha/env/master/src
% sudo --login -u test-koha pwd
/home/test-koha
% sudo --login -u test-koha
% pwd
/home/test-koha

-- 
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 25944] Bug in ill_requests patch schema

2020-07-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25944

Martin Renvoize  changed:

   What|Removed |Added

   Severity|normal  |major

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


  1   2   >