April 2018 - Koha-bugs - lists.koha-community.org

[Bug 11897] Stock Rotation for Koha
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11897 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #74010|0 |1 is obsolete| | --- Comment #208 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Created attachment 74197 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74197&action=ed… Bug 11897: Add Stock Rotation atomic update schema. * installer/data/mysql/atomicupdate/stockrot_tables.sql: New file. * installer/data/mysql/kohastructure.sql (stockrotationrotas) (stockrotationstages, stockrotationitems): New tables. * installer/data/mysql/sysprefs.sql: Add Stockrotation sysprefs. * installer/data/mysql/userflags.sql: Add Stockrotaiton userflag. * installer/data/mysql/userpermissions.sql: Add Stockrotation userpermissions. * koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref: Add Stockrotation menu. Signed-off-by: Kathleen Milne <kathleen.milne(a)cne-siar.gov.uk> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 Andrew Isherwood <andrew.isherwood(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |20563 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20563 [Bug 20563] ILL request list gives no indication of source and/or target -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 Andrew Isherwood <andrew.isherwood(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |20581 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20581 [Bug 20581] Allow manual selection of custom ILL request statuses -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 Andrew Isherwood <andrew.isherwood(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |20548 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20548 [Bug 20548] Remove copyright clearance workflow from staff created ILL requests -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 Andrew Isherwood <andrew.isherwood(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |20556 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20556 [Bug 20556] Marking ILL request as complete results in "Internal server error" -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15219] Server-side processing and pagination on checkouts tables
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15219 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply CC| |katrin.fischer(a)bsz-bw.de --- Comment #23 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Please rebase! Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 15219: Server-side processing and pagination on checkouts tables Using index info to reconstruct a base tree... M admin/columns_settings.yml M installer/data/mysql/sysprefs.sql M koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc M koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref M koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt M koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc Auto-merging installer/data/mysql/sysprefs.sql Auto-merging admin/columns_settings.yml Failed to merge in the changes. Patch failed at 0001 Bug 15219: Server-side processing and pagination on checkouts tables The copy of the patch that failed is found in: /home/vagrant/kohaclone/.git/rebase-apply/patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-15219-Server-side-processing-and-pagination-on-niral1.patch -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12446] Enable an adult to have a guarantor
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12446] Enable an adult to have a guarantor
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446 --- Comment #61 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Comment on attachment 72883 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72883 Bug 12446 - Enable an adult to have a guarantor Review of attachment 72883: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=124…) ----------------------------------------------------------------- Hi Roch, I've done some initial review, but I am not quite happy with this as is. The biggest problem I see is that it does much more than the bug title advertises. The flag on patron categories and the new preference to decide which patron fields to copy should have been at least 2 bugs. There is also changes to the display like displaying the account balance (which should probably use Price TT?). Please consider splitting the bug when working on it again. 1) QA script: FAIL koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/guarantor_search.tt FAIL forbidden patterns forbidden pattern: Contains old bootstrap style for buttons (see bug 16239) (line 20) 2) Spinter review: ::: installer/data/mysql/atomicupdate/bug_12446-EnableAdultGarantee.sql @@ +6,5 @@ > + > +-- ********* -- > +-- STRUCTURE -- > +-- ********* -- > +ALTER TABLE categories ADD COLUMN `canbeguarantee` tinyint(1) NOT NULL default '0'; Please use AFTER to declare where the column should be added to match with the kohastructure statement. Because depending on when this is pushed, it might lead to a different database structure between new and old installations otherwise. ::: installer/data/mysql/kohastructure.sql @@ +303,4 @@ > `hidelostitems` tinyint(1) NOT NULL default '0', -- are lost items shown to this category (1 for yes, 0 for no) > `category_type` varchar(1) NOT NULL default 'A', -- type of Koha patron (Adult, Child, Professional, Organizational, Statistical, Staff) > `BlockExpiredPatronOpacActions` tinyint(1) NOT NULL default '-1', -- wheither or not a patron of this category can renew books or place holds once their card has expired. 0 means they can, 1 means they cannot, -1 means use syspref BlockExpiredPatronOpacActions > + `default_privacy` ENUM( 'default', 'never', 'forever' ) NOT NULL DEFAULT 'default', -- Default privacy setting for this patron category, This change is not necessary, no comma needed after the comment. ::: installer/data/mysql/sysprefs.sql @@ +6,5 @@ > ('AcquisitionDetails', '1', '', 'Hide/Show acquisition details on the biblio detail page.', 'YesNo'), > ('AcqViewBaskets','user','user|branch|all','Define which baskets a user is allowed to view: his own only, any within his branch or all','Choice'), > ('AcqWarnOnDuplicateInvoice','0','','Warn librarians when they try to create a duplicate invoice','YesNo'), > +('AdditionalGuarantorField','',NULL,'Additional fields name to be transfer from guarantor to guarantee.','free'), > +('AddPatronLists','categorycode','categorycode|category_type','Allow user to choose what list to pick up from when adding patrons','Choice'), Change to AddPatronLists seems misplaced here. ::: koha-tmpl/intranet-tmpl/prog/en/includes/member-main-address-style-us.inc @@ +8,4 @@ > <label for="streetnumber"> > [% END %] > Street number: </label> > + <input type="text" id="streetnumber" name="streetnumber" size="5" value="[% streetnumber %]" /> Why only changes to the ...style-us and not the other templates? ::: koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt @@ +154,3 @@ > [% IF logged_in_user.can_see_patron_infos( guarantee ) %] > + <td><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantee.borrowernumber %]">[% guarantee.firstname | html %] [% guarantee.surname | html %]</a></td> > + <td style='text-align:right'>[% guarantee.account.balance %]</td> What has this to do with making it possible for adults to be guarantors? ;) ::: members/moremember.pl @@ +209,5 @@ > + } > + $template->param( guarantees => @guarantees); > + $template->param( amounttot => sprintf("%.2f",$totalmount)); > +} > +( $template->param( adultborrower => 1 ) ) if ( $category_type eq 'A' || $category_type eq 'I' ); If we set it on the patron category, why check here for I and A? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11708] Display all basketgroups on one page, and new column aqbasketgroups.closeddate
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11708 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #204 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Please rebase and check for changed needed! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12395] Save order line's creator
by bugzilla-daemon＠bugs.koha-community.org 16 Apr '18

16 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12395 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #97 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Patch doesn't apply, please rebase and check for changes needed. -- You are receiving this mail because: You are watching all bug changes.

1 0