[Koha-bugs] [Bug 30803] output_error should not assume a 404 status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30803

Jonathan Druart  changed:

   What|Removed |Added

 Depends on||23329


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23329
[Bug 23329] tracklinks.pl accepts any url from a parameter for proxying if not
tracking
-- 
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 23329] tracklinks.pl accepts any url from a parameter for proxying if not tracking

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23329

Jonathan Druart  changed:

   What|Removed |Added

 Blocks||30803


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30803
[Bug 30803] output_error should not assume a 404 status
-- 
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 29012] Some rules are not saved when left blank while editing a 'rule' line in smart-rules.pl

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29012

Kevin Carnes  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

-- 
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 29012] Some rules are not saved when left blank while editing a 'rule' line in smart-rules.pl

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29012

Kevin Carnes  changed:

   What|Removed |Added

 Attachment #131054|0   |1
is obsolete||

--- Comment #14 from Kevin Carnes  ---
Created attachment 135194
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135194&action=edit
Bug 29012: Add default values for blank circulation rules that weren't saved to
the database

There are 5 fields that are not set if no value is provided when saving/editing
a rule in Administration->Circulation and fines rules
- issuelength
- hardduedate
- unseenrenewalsallowed
- rentaldiscount
- decreaseloanholds

This is problematic because it gives the impression these rules are set as
blank, but in reality they don't exist and the rule will fal back to the higher
level

To test:
1 - Set a rule for
Patron category: Teacher
Itemtype: All
Hard due date: (Today)
Lona period: 10
2 - Set a rule for
Patron category: Teacher
Itemtype: Books
Hard due date: (leave blank)
Loan period: 10
3 - Expected behaviour is Book item will checkout to teacher for 10 days, all
other types will be due yesterday at 25:59:00
4 - Checkout an non-book item type to teacher
5 - Hard due date applies
6 - Checkout a 'book' item type to teacher
7 - Hard due date applies - FAIL

-- 
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 30804] New: Optionally show information on existing items in order form

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30804

Bug ID: 30804
   Summary: Optionally show information on existing items in order
form
 Change sponsored?: ---
   Product: Koha
   Version: 20.11
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Acquisitions
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: katrin.fisc...@bsz-bw.de
QA Contact: testo...@bugs.koha-community.org

Some libraries would like to see information about existing items when creating
an order line for an existing record. This would be especially helpful for
multi-branch libraries that would like to coordinate their acquisitions.

-- 
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 30805] New: Missing default value on import_records.marxml_old

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30805

Bug ID: 30805
   Summary: Missing default value on import_records.marxml_old
 Change sponsored?: ---
   Product: Koha
   Version: 20.11
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P5 - low
 Component: Architecture, internals, and plumbing
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: katrin.fisc...@bsz-bw.de
QA Contact: testo...@bugs.koha-community.org
Depends on: 20144

We fixed a ton of missing default values on bug 20144, I believe this one might
be another to fix:

import DBD::mysql::st execute failed: Field 'marcxml_old' doesn't have a
default value [for Statement "INSERT INTO import_records (import_batch_id,
record_sequence, marc, marcxml, 
 record_type, encoding,
z3950random)

We see this error in an older version after the latest Debian update with a new
MariaDB version, but in master we still have:

`marcxml` longtext COLLATE utf8mb4_unicode_ci NOT NULL,


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20144
[Bug 20144] Test suite is failing with new default SQL modes
-- 
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 20144] Test suite is failing with new default SQL modes

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20144

Katrin Fischer  changed:

   What|Removed |Added

 Blocks||30805


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30805
[Bug 30805] Missing default value on import_records.marxml_old
-- 
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 30507] Value builder for UNIMARC field 183

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30507

Jérémy Breuillard  changed:

   What|Removed |Added

 Attachment #135121|0   |1
is obsolete||

--- Comment #6 from Jérémy Breuillard  ---
Created attachment 135195
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135195&action=edit
Bug 30507: Value builder for field 183

This patch provides values builders for the subfields $a and $2. Those are
unimarc official codes who help to
identify the kind of items
Test plan:
1)Go to Home > Cataloging > New record
2)Section 1 - field 183 : it contains subfields $a & $2
3)Inputs are writable but notice that no plugins are suggested
4)Apply patch and repeat 1) to 3)
5)Values builders plugins are available for $a and $2

-- 
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 30507] Value builder for UNIMARC field 183

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30507

Jérémy Breuillard  changed:

   What|Removed |Added

 Attachment #135195|0   |1
is obsolete||

--- Comment #7 from Jérémy Breuillard  ---
Created attachment 135196
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135196&action=edit
Bug 30507: Value builder for field 183

This patch provides values builders for the subfields $a and $2. Those are
unimarc official codes who help to
identify the kind of items
Test plan:
1)Go to Home > Cataloging > New record
2)Section 1 - field 183 : it contains subfields $a & $2
3)Inputs are writable but notice that no plugins are suggested
4)Apply patch and repeat 1) to 3)
5)Values builders plugins are available for $a and $2

-- 
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 30507] Value builder for UNIMARC field 183

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30507

Jérémy Breuillard  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

-- 
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 30323] Wrong error messages on OPAC password recovery form when translated

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30323

Katrin Fischer  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |INVALID

--- Comment #3 from Katrin Fischer  ---
After a lot of testing, this turned out to be a local issue. Thx again Owen for
taking the time to test!

-- 
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 29560] Add option to create MARC links when adding items to bundles

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29560

Martin Renvoize  changed:

   What|Removed |Added

 Status|ASSIGNED|Needs Signoff

-- 
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 29560] Add option to create MARC links when adding items to bundles

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29560

Martin Renvoize  changed:

   What|Removed |Added

 CC||julian.maur...@biblibre.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 29051] Seen renewal methods incorrectly blocked

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29051

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/


[Koha-bugs] [Bug 30799] Add REST API route to update biblio metadata

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30799

Paul Derscheid  changed:

   What|Removed |Added

 Attachment #135102|0   |1
is obsolete||

--- Comment #2 from Paul Derscheid  ---
Created attachment 135197
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135197&action=edit
Bug 30799: Add REST API route to update biblio metadata

Example:

PUT /api/v1/biblios/{biblio_id}/metadata


...


Calling this endpoint will replace the current MARC record with the one
given in the request body, by calling C4::Biblio::ModBiblio

Test plan:
1. Try requesting this endpoint with your favorite API tool
2. Run `prove t/db_dependent/api/v1/biblios/metadata/put.t`

Signed-off-by: Paul Derscheid 

--- Works as advertised. Passes provided test.


Current status: 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 30799] Add REST API route to update biblio metadata

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30799

Paul Derscheid  changed:

   What|Removed |Added

 CC||paul.dersch...@lmscloud.de
 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 25126] MARC visibility settings not obeyed on cart normal and detailed views

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25126

Aude Charillon  changed:

   What|Removed |Added

 CC||aude.charillon@ptfs-europe.
   ||com

--- Comment #3 from Aude Charillon  ---
We've found the exact same situation as Lye Jia Lun describes on a site
recently upgraded to 21.11.05.
The site was on version 20.11.03 before where those options worked fine.

-- 
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 30507] Value builder for UNIMARC field 183

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30507

David Nind  changed:

   What|Removed |Added

 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 30507] Value builder for UNIMARC field 183

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30507

David Nind  changed:

   What|Removed |Added

 Attachment #135196|0   |1
is obsolete||

--- Comment #8 from David Nind  ---
Created attachment 135198
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135198&action=edit
Bug 30507: Value builder for field 183

This patch provides values builders for the subfields $a and $2. Those are
unimarc official codes who help to
identify the kind of items
Test plan:
1)Go to Home > Cataloging > New record
2)Section 1 - field 183 : it contains subfields $a & $2
3)Inputs are writable but notice that no plugins are suggested
4)Apply patch and repeat 1) to 3)
5)Values builders plugins are available for $a and $2

Signed-off-by: David Nind 

-- 
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 30275] Checkout renewals should be stored in their own table

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30275

--- Comment #194 from Tomás Cohen Arazi  ---
(In reply to Jonathan Druart from comment #193)
> (In reply to Martin Renvoize from comment #192)
> > Finally..  I couldn't replicate any unit test failures.. for me they all
> > pass first time (after a reset) and subsequent runs also pass too :?
> 
> Hum, confirmed. Cannot recreate today. Certainly something wrong with my DB
> yesterday.

Maybe worth adding a test for 2022-05-18? Kidding

-- 
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 30790] Add REST API route to list biblios

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30790

David Nind  changed:

   What|Removed |Added

 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 30790] Add REST API route to list biblios

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30790

David Nind  changed:

   What|Removed |Added

 Attachment #135089|0   |1
is obsolete||

--- Comment #7 from David Nind  ---
Created attachment 135199
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135199&action=edit
Bug 30790: Add REST API route to list biblios

Examples:

GET /api/v1/biblios
GET /api/v1/biblios?q={"title": "foo"}
GET /api/v1/biblios?q={"isbn": "1-23-456789-0"}
GET /api/v1/biblios?_order_by=title

Test plan:
1. Try requesting this endpoint with your favorite API tool
2. Run `prove t/db_dependent/api/v1/biblios/list.t`

Signed-off-by: David Nind 

-- 
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 29860] Useless warnings in regressions.t

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29860

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

--- Comment #3 from Tomás Cohen Arazi  ---
(In reply to Marcel de Rooy from comment #2)
> Tomas: What about changing status here ?

Oops

-- 
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 29828] If no content is added to default, but a translation, news/additional content entries don't show in list

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29828

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 30806] New: Use patron-title.inc in member-flags template

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30806

Bug ID: 30806
   Summary: Use patron-title.inc in member-flags template
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Templates
  Assignee: oleon...@myacpl.org
  Reporter: martin.renvo...@ptfs-europe.com
QA Contact: testo...@bugs.koha-community.org
Blocks: 30617

We should use the patron-title include everwhere we display a patron title
instead of templates mixing up formats.


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30617
[Bug 30617] [OMNIBUS] We should consistently use includes for patron name
display
-- 
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 30617] [OMNIBUS] We should consistently use includes for patron name display

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30617

Martin Renvoize  changed:

   What|Removed |Added

 Depends on||30806


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30806
[Bug 30806] Use patron-title.inc in member-flags template
-- 
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 30806] Use patron-title.inc in member-flags template

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30806

Martin Renvoize  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff
   Keywords||Academy

-- 
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 30806] Use patron-title.inc in member-flags template

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30806

Martin Renvoize  changed:

   What|Removed |Added

   Patch complexity|--- |Trivial patch

-- 
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 30806] Use patron-title.inc in member-flags template

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30806

--- Comment #1 from Martin Renvoize  ---
Created attachment 135200
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135200&action=edit
Bug 30806: Migrate to patron-title in member-flags

This patch updates the member-flags template to use the patron-title.inc
include wherever patron names are referenced.

Test plan
1) Navigate to a patron and click through to edit their permissions
2) Note how the patron name displays in the title, breadcrump and
   headings
3) Apply the patch and reload the page
4) Confirm the patron name still appears in each location and is
   consistently formatted and linked as you would expect

-- 
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 30807] New: Use patron-title.inc in patron payments pages

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30807

Bug ID: 30807
   Summary: Use patron-title.inc in patron payments pages
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Keywords: Academy
  Severity: enhancement
  Priority: P5 - low
 Component: Templates
  Assignee: oleon...@myacpl.org
  Reporter: martin.renvo...@ptfs-europe.com
QA Contact: testo...@bugs.koha-community.org
CC: oleon...@myacpl.org, testo...@bugs.koha-community.org
Blocks: 30617

We should use the patron-title include everwhere we display a patron title
instead of templates mixing up formats.


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30617
[Bug 30617] [OMNIBUS] We should consistently use includes for patron name
display
-- 
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 30617] [OMNIBUS] We should consistently use includes for patron name display

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30617

Martin Renvoize  changed:

   What|Removed |Added

 Depends on||30807


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30807
[Bug 30807] Use patron-title.inc in patron payments pages
-- 
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 30807] Use patron-title.inc in patron payments pages

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30807

--- Comment #1 from Martin Renvoize  ---
Created attachment 135201
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135201&action=edit
Bug 30807: Migrate to patron-title in pay and paycollect

This patch updates the member-flags template to use the patron-title.inc
include wherever patron names are referenced.

Test plan
1) Navigate to a patron and click through to their accounts tab
2) Note how the patron name displays in the title, breadcrumb and
   headings
3) Click through to make a payment
4) Note how the patron name displays in the title, breadcrumb and
   headings
5) Apply the patch and reload the page
6) Confirm the patron name still appears in each location and is
   consistently formatted and linked as you would expect

Note: This patch also removes a superflous hidden title form element
that was simply passed to and from the controller but not actually used
in any way.

-- 
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 30807] Use patron-title.inc in patron payments pages

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30807

Martin Renvoize  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
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 30807] Use patron-title.inc in patron payments pages

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30807

Martin Renvoize  changed:

   What|Removed |Added

   Patch complexity|--- |Trivial patch

-- 
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 30802] numReturnedItemsToShow doesn't show more than 20 items on the return screen

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30802

--- Comment #2 from Caroline Cyr La Rose  ---
I tested in a sandbox in case it was my installation that was acting up, and
the sandbox is limited to 14, completely ignoring the value in
numReturnedItemsToShow (which was set to 20 by default). Even after I changed
the pref to 50, the returns.pl page only shows 14.

-- 
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 30808] New: Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

Bug ID: 30808
   Summary: Release team 22.11
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P5 - low
 Component: About
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: martin.renvo...@ptfs-europe.com
QA Contact: testo...@bugs.koha-community.org
CC: da...@davidnind.com, dc...@prosentient.com.au,
fridolin.som...@biblibre.com,
jonathan.druart+k...@gmail.com,
katrin.fisc...@bsz-bw.de, k...@bywatersolutions.com,
m.de.r...@rijksmuseum.nl,
testo...@bugs.koha-community.org, vic...@tuxayo.net,
wainuiwitikap...@catalyst.net.nz
Depends on: 29300

Add the next release team :)


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29300
[Bug 29300] Release team 22.05
-- 
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 29300] Release team 22.05

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29300

Martin Renvoize  changed:

   What|Removed |Added

 Blocks||30808


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808
[Bug 30808] Release team 22.11
-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

Martin Renvoize  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |martin.renvoize@ptfs-europe
   |ity.org |.com

-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

--- Comment #1 from Martin Renvoize  ---
Created attachment 135202
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135202&action=edit
Bug 30808: Add the 22.05 release team.

Test plan
1/ Check against
   https://wiki.koha-community.org/wiki/Release_Teams

-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

Martin Renvoize  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

Martin Renvoize  changed:

   What|Removed |Added

   Keywords||rel_22_05_candidate

-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

--- Comment #2 from Martin Renvoize  ---
This should be one of the last things pushed as you do go through the release
process Fridolin :)

-- 
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 30808] Release team 22.11

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30808

Martin Renvoize  changed:

   What|Removed |Added

 CC|dc...@prosentient.com.au,   |
   |k...@bywatersolutions.com,  |
   |testo...@bugs.koha-communit |
   |y.org,  |
   |wainuiwitikapark@catalyst.n |
   |et.nz   |

-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

Shi Yao Wang  changed:

   What|Removed |Added

 Attachment #135175|0   |1
is obsolete||

--- Comment #6 from Shi Yao Wang  ---
Created attachment 135203
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135203&action=edit
Bug 27519: Normalize Show/Don't show in system preferences

Replace Display/Don't display (and some other pairs) by Show/Don't show in
system preferences.

Test plan:
1- Go to administration > system preferences
2- View any system preference with Display/Don't display pair of options
(e.g. OPACAcquisitionDetails)
3- Apply the patch
4- Refresh the page and notice it is now a Show/Don't show pair instead
5- Loop the steps with other system preferences until satisfied

Signed-off-by: Lucas Gass 

-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

--- Comment #7 from Shi Yao Wang  ---
(In reply to David Nind from comment #5)
> I love consistency for terminology, so this is great!
> 
> I have two suggestions to move the "Show/Don't show" options to the start of
> the system preference text, rather than at the end, to be as consistent as
> possible with how it is used elsewhere:
> 
> - noItemTypeImages: 
>   . Updated text: Show itemtype icons in the staff interface: Show/Don't show
>   . Change to: Show/Don't show itemtype icons in the staff interface.
> 
> - OpacNoItemTypeImages:
>   . Updated text: Show itemtype icons in the OPAC: Show/Don't show
>   . Change to: Show/Don't show itemtype icons in the OPAC.
> 
> I'll also add to the terminology list:
> https://wiki.koha-community.org/wiki/Terminology

Added the suggested changes in noItemTypeImages and OpacNoItemTypeImages.

-- 
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 30809] New: Add method to delete analytic link from holdings table

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30809

Bug ID: 30809
   Summary: Add method to delete analytic link from holdings table
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Cataloging
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: and...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org
CC: m.de.r...@rijksmuseum.nl

When using analytic cataloging, the bib detail page for the child record shows
an item from the host record and offers all of the usual edit/delete options.
We should also offer an option to remove the analytic link right from the
holdings table on the bib detail page. Currently, one needs to edit the child
bib and remove the 773 to break the link.

-- 
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 30810] New: We should move sample_quotes out of en

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30810

Bug ID: 30810
   Summary: We should move sample_quotes out of en
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Architecture, internals, and plumbing
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: martin.renvo...@ptfs-europe.com
QA Contact: testo...@bugs.koha-community.org

The sample quotes are all quotes from American presidents.. not really things
that everyone wants/needs to translate at translation time.

We should move them out of /en/optional and into /localization/en-US. This
would stop them from appearing in the translations and give other cultures the
opportunity to add their own more relevant sample quotes.

-- 
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 30810] We should move sample_quotes out of en

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30810

Martin Renvoize  changed:

   What|Removed |Added

 CC||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 30810] We should move sample_quotes out of en

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30810

--- Comment #1 from Martin Renvoize  ---
That, or reduce them to a very small list of samples that are very Koha
specific and might be meaningful to a wider audience.

Personal favourite, but inappropriate

rangi: #thingsihavelearnt if there is a mad scheme a library somewhere will be
doing it ... except madder

-- 
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 23659] Allow hold pickup location to default to item home branch

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23659

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 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 23659] Allow hold pickup location to default to item home branch

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23659

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Attachment #135120|0   |1
is obsolete||

--- Comment #4 from Andrew Fuerste-Henry  ---
Created attachment 135204
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135204&action=edit
Bug 23659: Add DefaultHoldPickupLocation system preference

On the staff side behaviour differs for default pickup location when placing a
hold or
placing an overridden hold. Additionally, the behaviour has changed betwee
master and stables

We should provide a consistent default, and allow the library to specify their
choice

Note this only affects staff client as there is only a single dropdown on OPAC
and it is not
tied to items

To test:
 1 - Apply patch
 2 - Update database
 3 - Find a record with items from various branches, and at least one with a
different home/holding branch
 4 - Ensure there is an item that requires override to hold, and 
AllowHoldPolicyOverride  is enabled
 4 - Attempt to place hold
 5 - Confirm all dropdowns default to logged in library
 6 - Set DefaultHoldPickupLocation to item's home branch
 7 - Refresh and confirm all dropdowns match item home library except biblio
level hold - still logged in library
 8 - Set DefaultHoldPickupLocation to item's holding branch
 9 - Refresh and confirm defaults
10 - Mark one of the items holding library as 'not a pickup location' in
Admin->Libraries
11 - Refresh and confirm dropdown is now empty for that item

Signed-off-by: Andrew Fuerste-Henry 

-- 
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 30810] We should move sample_quotes out of en

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30810

Martin Renvoize  changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

-- 
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 23659] Allow hold pickup location to default to item home branch for item-level holds

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23659

Andrew Fuerste-Henry  changed:

   What|Removed |Added

Summary|Allow hold pickup location  |Allow hold pickup location
   |to default to item home |to default to item home
   |branch  |branch for item-level holds

-- 
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 30811] New: Allow hold pickup location to default to item home/holding branch for bib-level holds

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30811

Bug ID: 30811
   Summary: Allow hold pickup location to default to item
home/holding branch for bib-level holds
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P5 - low
 Component: Hold requests
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: and...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org
CC: gmcha...@gmail.com
Depends on: 23659

Bug 23659 adds the ability to make item-level holds default their pickup
location to either the item homebranch or the item holdingbranch. It would be
great to be able to do this for bib-level holds, too -- that would mean the
hold does not truly have a pickup branch until an item is captured, at which
point the pickup location gets set to wherever the captured item is.


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23659
[Bug 23659] Allow hold pickup location to default to item home branch for
item-level holds
-- 
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 23659] Allow hold pickup location to default to item home branch for item-level holds

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23659

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Blocks||30811


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30811
[Bug 30811] Allow hold pickup location to default to item home/holding branch
for bib-level holds
-- 
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 28950] serialsUpdate cron does not mark an issue late until the next issue is expected

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28950

Nick Clemens  changed:

   What|Removed |Added

   Patch complexity|--- |Trivial patch
 Status|NEW |Needs Signoff

-- 
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 28950] serialsUpdate cron does not mark an issue late until the next issue is expected

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28950

--- Comment #1 from Nick Clemens  ---
Created attachment 135205
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135205&action=edit
Bug 28950: Mark serial late when past grace period

This script had some oddities:
- There was a LEFT JOIN to subscription, but no error printed if no
subscription
  I changed this to a JOIN as there is a constraint, so this shoudl not happen
- Publisheddate was checked after the SQL, moved into query
- We checked for the next issue, and marked late only if that issue was already
expected
  This overruled grace period, which should be the mnarker for a serial being
late
  The grace period should be extended if you wish to wait for next issue
- If no next published date, we reported an error on the planneddate
- Script without confirm had no reporting

-- 
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 30275] Checkout renewals should be stored in their own table

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30275

Martin Renvoize  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

-- 
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 30812] New: subfield 9 deleted if no other data is available

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30812

Bug ID: 30812
   Summary: subfield 9 deleted if no other data is available
 Change sponsored?: ---
   Product: Koha
   Version: 21.05
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P5 - low
 Component: Cataloging
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: d...@plix.at
QA Contact: testo...@bugs.koha-community.org
CC: m.de.r...@rijksmuseum.nl

DNB exports the ISBN with dashes in 020$9, which we import via SRU while
cataloguing.

Therefore we added a subfield 9 to field 020.

But when 020 contains no data except the dashed ISBN in $9, the value of $9 is
deleted.

I assume this is caused by some special cases needed to delete the koha_id of
an authority (which is often stored in $9).

To reproduce:

* Go to a framwork, field 020, add a subfield 9, visible in tab 0, OPAC,
Intranet and edito
* Go to a book, edit it, enter something in 020$a and 020$9, save
* The values are stored
* Edit again, remove the value from 020$a, save
* the value from 020$9 is gone, too

I can maybe take a look tomorrow to try to figure out what's happening..

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 CC||martin.renvoize@ptfs-europe
   ||.com

--- Comment #9 from Martin Renvoize  ---
Good spot finding 'SkipHoldTrapOnNotForLoanValue' David, to me that pref is
slightly mid-described.. it feels like it should affect not just trapping at
checking but also this HoldsQueue and pendingreserves too.

For history, the notforloan handling was dropped from here by mistake
originally in bug 3142 where it looks like we assumed the same logic was
already taking place in the filter_by_for_hold. (Bad assumption.. we go from
filtering on `!= 0` to filtering on `'<=' => 0`

An initial 'fix' would be to restore the `!= 0` found in
get_items_that_can_fill I believe.. though I'm not sure how the chaining would
treat it..

I'm not entirely a fan of the current implementation in this patch.. your
adding a query to fetch all items available for loans instead of just adding
the 'notforloan' limit to the existing overall query.

I'll have a read of the unit tests now to check they make sense, then I think
we can golf this a little further.

As for the 'SkipHoldTrapOnNotForLoanValue', that might be a nice thing to add
support for whilst we're here.

-- 
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 30747] Column settings on otherholdings table in detail.tt doesnt work

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30747

--- Comment #13 from Lucas Gass  ---
Thank you Marcel!

-- 
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 30802] numReturnedItemsToShow doesn't show more than 20 items on the return screen

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30802

--- Comment #3 from Shi Yao Wang  ---
Created attachment 135206
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135206&action=edit
Bug 30802: Make numReturnedItemsToShow function properly

Replaced the hardcoded 20 by the value of numReturnedItemsToShow to
display the right number of items in check in.

Test plan:
1-  Make lots of checkouts, at least like 40 (I used the batchCheckouts
feature)
2-  Go to Circulation > Check in
3-  Return 21 items
The last 20 items returned will be displayed
4-  Change numReturnedItemsToShow to 50
5-  Return a couple more items
Only the last 20 returned items are displayed
6-  Change numReturnedItemsToShow to 10
7-  Return a couple more items
Only the last 10 returned items are displayed
8-  Apply the patch
9-  Change numReturnedItemsToShow back to 20
10- Do steps 1 to 7 again, but this time step 5 returns the right amount
of items

-- 
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 30802] numReturnedItemsToShow doesn't show more than 20 items on the return screen

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30802

Shi Yao Wang  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |shi-yao.w...@inlibro.com
   |ity.org |
 CC||caroline.cyr-la-rose@inlibr
   ||o.com,
   ||philippe.blo...@inlibro.com
   ||, shi-yao.w...@inlibro.com
 Status|NEW |Needs Signoff

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

--- Comment #10 from Martin Renvoize  ---
Wow, the prefs do make this complicated!

OK.. I spent some time working through this.. 'Trap' is very much about 'At
Checkin' prompts.. in this case we're building lists of items to go and find..
so the preferences don't really come into play.

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #135081|0   |1
is obsolete||

--- Comment #11 from Martin Renvoize  ---
Created attachment 135207
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135207&action=edit
Bug 30730: Holds to Pull should not list items with a notforloan status

Remove items with a notforloan value other than 0 because items that are
potentially holdable (with negative notforloan status) are not wanted in holds
to pull.

Test plan:
1- Have a bib with 2 items, both holdable. Give one of these items a notforloan
status with a negative value (like Ordered in the testing docker data).
2- Place a title-level hold on your bib
3- Confirm your bib is showing on Holds to Pull as expected, prompting staff to
pull the item that does not have a not for loan status (correct!)
4- Take the item that does not have a not for loan status and check it out to a
different patron, make sure not to fill or cancel your hold
5- Reload Holds to Pull, see your title is still listed and now Koha is
directing staff to the item with a notforloan status
6- Apply the patch
7- Reload Holds to Pull, see your title with a notforloan status is gone
8- Run `prove t/db_dependent/Koha/Holds.t` and all tests should pass

Signed-off-by: David Nind 
Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #135082|0   |1
is obsolete||

--- Comment #12 from Martin Renvoize  ---
Created attachment 135208
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135208&action=edit
Bug 30730: Unit tests

Added an entry in holds for an item with negative notforloan value to
test if get_items_that_can_fill returns the right items.

Signed-off-by: David Nind 
Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

--- Comment #13 from Martin Renvoize  ---
Created attachment 135209
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135209&action=edit
Bug 30730: (follow-up) Simplify query

This patch drops the 'fetch all non-loanable items' query used in the
subsequent query as part of the 'not_in' and instead replaces it with a
simple filter on 'notforloan => 0'

Test plan
Run the included unit tests before and apply applying the followup.

Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #135207|0   |1
is obsolete||

--- Comment #14 from Martin Renvoize  ---
Created attachment 135210
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135210&action=edit
Bug 30730: Holds to Pull should not list items with a notforloan status

Remove items with a notforloan value other than 0 because items that are
potentially holdable (with negative notforloan status) are not wanted in holds
to pull.

Test plan:
1- Have a bib with 2 items, both holdable. Give one of these items a notforloan
status with a negative value (like Ordered in the testing docker data).
2- Place a title-level hold on your bib
3- Confirm your bib is showing on Holds to Pull as expected, prompting staff to
pull the item that does not have a not for loan status (correct!)
4- Take the item that does not have a not for loan status and check it out to a
different patron, make sure not to fill or cancel your hold
5- Reload Holds to Pull, see your title is still listed and now Koha is
directing staff to the item with a notforloan status
6- Apply the patch
7- Reload Holds to Pull, see your title with a notforloan status is gone
8- Run `prove t/db_dependent/Koha/Holds.t` and all tests should pass

Signed-off-by: David Nind 
Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #135208|0   |1
is obsolete||

--- Comment #15 from Martin Renvoize  ---
Created attachment 135211
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135211&action=edit
Bug 30730: Unit tests

Added an entry in holds for an item with negative notforloan value to
test if get_items_that_can_fill returns the right items.

Signed-off-by: David Nind 
Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #135209|0   |1
is obsolete||

--- Comment #16 from Martin Renvoize  ---
Created attachment 135212
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135212&action=edit
Bug 30730: (follow-up) Simplify query

This patch drops the 'fetch all non-loanable items' query used in the
subsequent query as part of the 'not_in' and instead replaces it with a
simple filter on 'notforloan => 0'

Test plan
Run the included unit tests before and apply applying the followup.

Signed-off-by: Martin Renvoize 

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

--- Comment #17 from Martin Renvoize  ---
OK.. I'm happy this restores the previous functionality.. we have Unit tests
and the code is simplified.

QA scripts are happy

Passing QA

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

   Keywords||regression,
   ||rel_22_05_candidate

-- 
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 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

Martin Renvoize  changed:

   What|Removed |Added

 QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe
   |y.org   |.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 30337] Holds to Pull ( pendingreserves.pl ) ignores holds if priority 1 hold is suspended

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30337

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #134972|0   |1
is obsolete||

--- Comment #6 from Martin Renvoize  ---
Created attachment 135213
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135213&action=edit
Bug 30337: Holds to Pull ( pendingreserves.pl ) ignores holds if priority 1
hold is suspended

Holds to Pull ( pendingreserves.pl ) ignores holds if priority 1 hold is
suspended,
and shows the wrong number of "patrons with holds" which presumably tells the
librarian how many copies to pull from the shelves.

Test Plan:
1) Create a record with one or more holdable items
2) Place two holds on the record
3) Note they show in the holds to pull report
4) Suspend the priority 2 hold
5) Note the report continues showing that 2 patrons have holds
6) Resume the priority 2 hold
7) Suspend the priority 1 hold
8) Note the hold disappears from the holds to pull report
9) Apply this patch
10) Restart all the things!
11) Reload the holds to pull report
12) The report should show one hold that needs an item pulled to fill it!

Signed-off-by: David Nind 
Signed-off-by: Caroline Cyr La Rose 
Signed-off-by: Martin Renvoize 

-- 
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 30337] Holds to Pull ( pendingreserves.pl ) ignores holds if priority 1 hold is suspended

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30337

Martin Renvoize  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA
 CC||martin.renvoize@ptfs-europe
   ||.com

--- Comment #7 from Martin Renvoize  ---
Works as expected and resolves a regression.  QA script is happy.

I'd love to have seen some work to move some of the queries here out of the
controller and up into the Koha::Holds class (so we could also have unit tests
added)..

But as a fix, this does the job and is pragmatic so I can't really fail it ;P

Passing

-- 
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 29828] If no content is added to default, but a translation, news/additional content entries don't show in list

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29828

--- Comment #13 from Elaine Bradtke  ---
(In reply to David Nind from comment #8)
> Testing notes (koha-testing-docker):
> 
> - When testing, I was getting some "flaky" behaviour with new news items not
> displaying.
> 
> - In the end, I emptied the additional_contents table (truncate
> additional_contents;) and started again.
> 
> - I think the issue was the difference between the date on my computer (we
> are NZST - so date was 19 May 2022) compared with koha-testing-docker (UTC
> time 18 May 2022): news wasn't displaying in the OPAC and staff interface as
> the date of the news item was in the future (that is, the publication date
> had not yet been reached).

I have seen the same flaky behaviour.  I'm eight hours behind the catalogue I
maintain, so there are times when they're on a different date.  Good to know
this is a potential problem.

-- 
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 30751] Print autorenewal notice to console when running in verbose.

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30751

Nick Clemens  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #3 from Nick Clemens  ---
Right now, the error is not set in the issue object if not confirm:
$auto_renew->auto_renew_error($error)->store if $confirm;

I suspect we need:
$auto_renew->auto_renew_error($error)
$auto_renew->store if $confirm;

Additionally:
173 if ($confirm){
174 my $date_due = AddRenewal( $auto_renew->borrowernumber,
$auto_renew->itemnumber, $auto_renew->branchcode, undef, undef, undef, 0 );
175 $auto_renew->auto_renew_error(undef)->store;
176 }


Otherwise the messages I get don't have the correct info/status

There may be other instances like this to check as 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 30755] too_soon should not be counted as an error in autorenewals

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30755

Nick Clemens  changed:

   What|Removed |Added

 CC||n...@bywatersolutions.com

--- Comment #5 from Nick Clemens  ---
The digests will receive an issue marked as 'auto_too_soon' - there should not
be a case of an issue marked for auto-renewal returning 'too_soon' without auto

The default notice should be updated to handle these - if you are getting a
digest I think you do want to see to:
Book A renewed
Book B not renewed (too soon)


Otherwise we should nex ton those in the loop

Beyond that - if all errors are too_soon we shouldn't trigger a notice, the
updated flag should not be sent and the digest should be skipped. Can you
confirm if you are seeing auto_too_soon on other notices, or notices with all
auto_too_soon?

-- 
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 24403] [OMNIBUS] OpacHiddenItems should hide items everywhere in the OPAC

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24403

Lisette Scheer  changed:

   What|Removed |Added

 CC||lisettesla...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 30730] Holds to Pull should not list items with a notforloan status

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30730

--- Comment #18 from David Nind  ---
Thanks Martin, apologies for confusing things

-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

--- Comment #8 from David Nind  ---
(In reply to Shi Yao Wang from comment #7)
> (In reply to David Nind from comment #5)
> > I love consistency for terminology, so this is great!
> > 
> > I have two suggestions to move the "Show/Don't show" options to the start of
> > the system preference text, rather than at the end, to be as consistent as
> > possible with how it is used elsewhere:
> > 
> > - noItemTypeImages: 
> >   . Updated text: Show itemtype icons in the staff interface: Show/Don't 
> > show
> >   . Change to: Show/Don't show itemtype icons in the staff interface.
> > 
> > - OpacNoItemTypeImages:
> >   . Updated text: Show itemtype icons in the OPAC: Show/Don't show
> >   . Change to: Show/Don't show itemtype icons in the OPAC.
> > 
> > I'll also add to the terminology list:
> > https://wiki.koha-community.org/wiki/Terminology
> 
> Added the suggested changes in noItemTypeImages and OpacNoItemTypeImages.

Hi Shi Yao.

Thanks for updating.

However, I think I may have confused things!

I meant having the dropdown list with the choice at the front, rather than at
the end of the sentence - eliminating the "double" show. For example:

- For OpacNoItemTypeImages it should read as: [Show/Don't show] itemtype icons
in the OPAC.

- Rather than: Show/Don't show itemtype icons in the OPAC: [Show/Don't show]

Hope that makes sense!

David

-- 
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 28556] Make guarantor information available in notices

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28556

Shi Yao Wang  changed:

   What|Removed |Added

 Attachment #130340|0   |1
is obsolete||

--- Comment #4 from Shi Yao Wang  ---
Created attachment 135214
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135214&action=edit
Bug 28556: Guarantor's information is available in notice

This patch adds the guarantor's information when sending overdue
notice.

To test:
1. Make sure that there is an overdue letter.
 a) Go in Tools/Notices & Slips.
 b) Search for the code "ODUE" and note the "Name" of the letter.
 c) Click on "Edit".
 d) Click on "Email" and make sure that there is text inside the box.
 e) Add the following code in the textbox:
C/O <> <> 
 f) Click on "Save".

2. Set up the status trigger.
 a) Go in Tools/Overdue notice/status triggers.
 b) In "First" tab, put a number of days in the "Delay" column (ex: 1).
 c) In the column "Letter", put the "Name" of the letter of step 1a) .
 d) Check "Email".
 e) Click "Save".

3. Create a patron with a guarantor.
 a) Go in Administration/Patron categories.
 b) Create a child category AND a adult category.
 c) Create an adult patron.
 d) Create a child patron.
 e) Go in the child patron's page and click "Edit".
 f) In the "Patron guarantor" section, click add guarantor.
 g) Search for your adult patron and click "Select".
 h) Select a relationship for the guarantor (ex: Mother).
 i) Click on "Save".

4. Borrow an overdue item.
 a) Click on the arrow next to "Search button" (Top of the screen).
 b) Click on "Items search".
 c) Click on search.
 d) Copy a barcode in the list.
 e) Search for your child patron.
 f) Go in his page and click on "Checkout".
 g) Paste the barcode copied at step 4d) .
 h) Click on "Checkout settings".
 i) Put a date in the past greater than the number of days that you put
on step 2b).
 j) Click on "Check out".
 k) Click on "Yes" Notice tht there is a message saying that Patron has
ITEMS OVERDUE.

5. Test the patch.
 a) In the terminal, run ./misc/cronjobs/overdue_notices.pl
 b) Return in your child patron's page.
 c) Click on "Notice" and you should see a letter.
 d) Click on the letter to see the content.
 c) Notice that you can't see the guarantor's informations (Adult patron).
 d) Apply the patch.
 e) Repeat steps a) to d)
 f) Notice that you can see the guarantor's informations.

This also works if you decide to add a non-patron guarantor instead of a
patron guarantor at step 3.

NOTE: If you want to delete the letters, go in your database and run:
- delete from message_queue;

-- 
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 28556] Make guarantor information available in notices

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28556

Shi Yao Wang  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff
 CC||shi-yao.w...@inlibro.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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

Shi Yao Wang  changed:

   What|Removed |Added

 Attachment #135203|0   |1
is obsolete||

--- Comment #9 from Shi Yao Wang  ---
Created attachment 135215
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135215&action=edit
Bug 27519: Normalize Show/Don't show in system preferences

Replace Display/Don't display (and some other pairs) by Show/Don't show in
system preferences.

Test plan:
1- Go to administration > system preferences
2- View any system preference with Display/Don't display pair of options
(e.g. OPACAcquisitionDetails)
3- Apply the patch
4- Refresh the page and notice it is now a Show/Don't show pair instead
5- Loop the steps with other system preferences until satisfied

Signed-off-by: Lucas Gass 

-- 
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 30789] Improve performance of AddBiblio

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30789

Nick Clemens  changed:

   What|Removed |Added

 Attachment #135071|0   |1
is obsolete||

--- Comment #3 from Nick Clemens  ---
Created attachment 135216
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135216&action=edit
Bug 30789: Improve performance of AddBiblio

AddBiblio calls TransformMarcRecordToKoha. We are only interested in getting
biblio and biblioitems info, so we can pass 'no_items' to save some field
lookups

Benchmarking saw a ~75% increase in the performance of this call

To test:
1 - Stage and import some records
2 - Confirm it works the same before and after patch
3 - Add a biblio via cataloguing, confirm it works

-- 
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 30813] New: Refactor TransformMarcToKoha to remove TransformMarcToKohaOneField

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30813

Bug ID: 30813
   Summary: Refactor TransformMarcToKoha to remove
TransformMarcToKohaOneField
 Change sponsored?: ---
   Product: Koha
   Version: master
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P5 - low
 Component: Architecture, internals, and plumbing
  Assignee: koha-bugs@lists.koha-community.org
  Reporter: n...@bywatersolutions.com
QA Contact: testo...@bugs.koha-community.org

While testing bug 30789 it was found that passing the already retrieved marc
subfield structure into TransformMarckToKohaOneField increased performance by
~20%

This seemed odd as the routine is already retrieving from cache unsafe

Rather than pass around this shared variable, it seems to make sense to allow
TransformMarcToKoha to take a list of fields to process, rather than have a
seperate routine that must be called

-- 
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 30813] Refactor TransformMarcToKoha to remove TransformMarcToKohaOneField

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30813

Nick Clemens  changed:

   What|Removed |Added

 Depends on||30789


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30789
[Bug 30789] Improve performance of AddBiblio
-- 
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 30789] Improve performance of AddBiblio

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30789

Nick Clemens  changed:

   What|Removed |Added

 Blocks||30813


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30813
[Bug 30813] Refactor TransformMarcToKoha to remove TransformMarcToKohaOneField
-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

David Nind  changed:

   What|Removed |Added

 Attachment #135215|0   |1
is obsolete||

--- Comment #10 from David Nind  ---
Created attachment 135217
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135217&action=edit
Bug 27519: Normalize Show/Don't show in system preferences

Replace Display/Don't display (and some other pairs) by Show/Don't show in
system preferences.

Test plan:
1- Go to administration > system preferences
2- View any system preference with Display/Don't display pair of options
(e.g. OPACAcquisitionDetails)
3- Apply the patch
4- Refresh the page and notice it is now a Show/Don't show pair instead
5- Loop the steps with other system preferences until satisfied

Signed-off-by: Lucas Gass 

Signed-off-by: David Nind 

-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

--- Comment #11 from David Nind  ---
Thanks Shi Yao for updating! I've added my sign off as well, to show the QA
I've checked the change0

-- 
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 27519] Normalize Show/Don't show vs Display/Don't display in system preferences

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27519

--- Comment #12 from Shi Yao Wang  ---
No problem David.

Thanks for the sign offs!

-- 
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 29860] Useless warnings in regressions.t

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29860

David Nind  changed:

   What|Removed |Added

 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 29860] Useless warnings in regressions.t

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29860

David Nind  changed:

   What|Removed |Added

 Attachment #129348|0   |1
is obsolete||

--- Comment #4 from David Nind  ---
Created attachment 135218
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135218&action=edit
Bug 29860: Remove item type-related warnings

To test:
1. Run:
   $ kshell
  k$ prove t/db_dependent/selenium/regressions.t
=> FAIL: You see several
item-level_itypes set but no itemtype set for item (985) at
/kohadevbox/koha/Koha/Schema/Result/Item.pm line 905.
2. Apply this patch
3. Repeat 1
=> SUCCESS: No more item type warnings
4. Sign off :-D

Signed-off-by: Tomas Cohen Arazi 

Signed-off-by: David Nind 

-- 
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 30781] Use of uninitialized value $val in substitution iterator at /usr/share/koha/lib/C4/Letters.pm line 665.

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30781

David Nind  changed:

   What|Removed |Added

 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 30781] Use of uninitialized value $val in substitution iterator at /usr/share/koha/lib/C4/Letters.pm line 665.

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30781

David Nind  changed:

   What|Removed |Added

 Attachment #135192|0   |1
is obsolete||

--- Comment #8 from David Nind  ---
Created attachment 135219
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135219&action=edit
Bug 30781: Add unit test for substitute with undefined value

Add unit test that generates warn.
This may be usefull as no regression test.

Run prove t/db_dependent/Letters.t
You see warn :
t/db_dependent/Letters.t .. 1/84 Use of uninitialized value $val in
substitution iterator at /kohadevbox/koha/C4/Letters.pm line 607.

Signed-off-by: David Nind 

-- 
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 30781] Use of uninitialized value $val in substitution iterator at /usr/share/koha/lib/C4/Letters.pm line 665.

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30781

David Nind  changed:

   What|Removed |Added

 Attachment #135193|0   |1
is obsolete||

--- Comment #9 from David Nind  ---
Created attachment 135220
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135220&action=edit
Bug 30781: Fix warning in GetPreparedLetter

Test by running :
  prove t/db_dependent/Letters.t
Check you dont see warning :
  Use of uninitialized value $val in substitution iterator

Signed-off-by: David Nind 

-- 
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 29260] UNIMARC 210a is reported to Author (meeting/conference) when upgrading an authority through Z3950

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29260

Katrin Fischer  changed:

   What|Removed |Added

Summary|UNIMARC 210a is reported to |UNIMARC 210a is reported to
   |Autor (meeting/conference)  |Author (meeting/conference)
   |when upgrading an authority |when upgrading an authority
   |through Z3950   |through Z3950

-- 
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 30755] too_soon should not be counted as an error in autorenewals

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30755

--- Comment #6 from David Cook  ---
(In reply to Nick Clemens from comment #5)
> The digests will receive an issue marked as 'auto_too_soon' - there should
> not be a case of an issue marked for auto-renewal returning 'too_soon'
> without auto
> 
> The default notice should be updated to handle these - if you are getting a
> digest I think you do want to see to:
> Book A renewed
> Book B not renewed (too soon)
> 
> 
> Otherwise we should nex ton those in the loop
> 
> Beyond that - if all errors are too_soon we shouldn't trigger a notice, the
> updated flag should not be sent and the digest should be skipped. Can you
> confirm if you are seeing auto_too_soon on other notices, or notices with
> all auto_too_soon?

That's what I'm confused about. Doesn't the following mean that auto_too_soon
errors aren't even being counted?

https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=29381&attachment=127947

-- 
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 21366] add plack reload

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21366

--- Comment #4 from David Cook  ---
(In reply to Joonas Kylmälä from comment #2)
> In koha-suomi fork we use start-stop-daemon to send the HUP signal, that's
> probably the preferable way since koha-plack uses start-stop-daemon already
> to send the process kill signal.

I'm only seeing "start-stop-daemon" for koha-plack --stop for some reason...

-- 
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 21366] add plack reload

2022-05-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21366

--- Comment #5 from David Cook  ---
(In reply to Fridolin Somers from comment #1)
> To trigger this reload, juste run :
> 
> /bin/kill -HUP $MASTER_PID
> 
> MASTER_PID beeing the pid of the starman master process, can be fetched in
> pid file.
> 
> Source :
> https://metacpan.org/pod/distribution/Starman/script/starman#SIGNALS

I'd forgotten about this comment, but I just used the following today in prod:

kill -SIGHUP $(cat /var/run/koha//plack.pid)

I wanted to enable plugins and then reload after installing the plugin, and it
was so elegant.

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