[Koha-bugs] [Bug 18086] overduerules table is lacking foreign key constraints

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri May 29 19:43:47 CEST 2020


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18086

Victor Grousset/tuxayo <victor at tuxayo.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #16 from Victor Grousset/tuxayo <victor at tuxayo.net> ---
(In reply to Victor Grousset/tuxayo from comment #15)
> Should the following problem be fixed?
> 
> > "Email" and "print" aren't saved when ticked, they disappear after saving.
> 
> It seems the lasted patch only fixed the compilation error.

I wanted to doublecheck and something else went wrong

- be on master code
- reset_all
- apply patch
- updatedatabase
- go to /cgi-bin/koha/tools/overduerules.pl
- fill on the first line, some delay, some letter, tick everything you can
- submit
- "Changes saved"
- the UI however shows nothing of what was inputted
- try again on the same line
- still nothing
- pick a library
- try again
- this time it works

So there is something wrong with the default triggers.
(Tested on Debian 10 and Debian 9.)


- be on master code + the patch
- spin up the instance (koha testing docker)
- go to /cgi-bin/koha/tools/overduerules.pl
- fill on the first line, some delay, some letter, tick everything you can
- submit
- all saved except email and phone
- happens only for default triggers

caveat: maybe the 2nd case is wrong. Maybe one shouldn't build a dev env with
the path already applied. I don't know how DB are handled when integrated in
master which makes this work in master. And might prevent from working on a
submitted patch.

Anyway, first case is valid, there is still an issue.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list