]> git.koha-community.org Git - koha.git/commit
Bug 29957: Add --keep-cookie option to koha-create
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 25 Mar 2022 08:20:46 +0000 (08:20 +0000)
committerKyle Hall <kyle@bywatersolutions.com>
Fri, 15 Apr 2022 11:15:21 +0000 (07:15 -0400)
commit4f5c0df5a4a832e57433fd2bb161d53bac9057d5
tree9aa00c85ab1cda096389cfba3ce58723c60e6074
parent6591123a5030b1152867c9ceb5ab67e40a473b2f
Bug 29957: Add --keep-cookie option to koha-create

As requested on comment99. Personally I do not feel that it
should be a requirement for adding a koha-conf entry though.
Note that koha-create does/did not even support adding multiple
line entries to koha-conf like we do here now.
But it might be a start for adding them?

Test plan:
The following tweak might help you to test without running the
complete koha-create, while still convincing you:

[1] cp debian/templates/koha-conf-site.xml.in /etc/koha/test.conf
[2] Add the following line in debian/scripts/koha-create. Append
    if after the BCRYPT_SETTINGS= line on L652:
    generate_config_file test.conf test.xml; exit
[3] Run debian/scripts/koha-create --keep-cookie A
[4] Check if you see <do_not_remove_cookie>A<..> in test.xml.
[5] Run debian/scripts/koha-create --keep-cookie A --keep-cookie B
[6] Check for two lines in test.xml
[7] Run debian/scripts/koha-create clone_name
[8] Check that the __KEEP_COOKIE__ line is gone in test.xml.
[9] Undo the tweak by removing test.xml and /etc/koha/test.conf,
    and removing the added line from koha-create.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
debian/scripts/koha-create
debian/templates/koha-conf-site.xml.in