Summary: | Cronjob path in the AuthorityMergeLimit syspref description is wrong | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | System Administration | Assignee: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | aleisha, david, gmcharlt, jonathan.druart, lucas, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | String patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
Updates the system preference description with the correct path for the cronjob (misc/cronjobs/merge_authorities.pl).
|
|
Version(s) released in: |
20.11.00, 20.05.01, 19.11.07, 19.05.13
|
Circulation function: | |
Bug Depends on: | 9988 | ||
Bug Blocks: | |||
Attachments: |
Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong
Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong |
Description
Caroline Cyr La Rose
2020-05-06 16:34:12 UTC
Created attachment 104443 [details] [review] Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Created attachment 104444 [details] [review] Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Test plan: 1- Apply patch 2- Go to Administration > Global system preferences > Authorities 3- The path to merge_authorities.pl in the AuthorityMergeLimit description should now read misc/cronjobs/merge_authorities.pl I hope I did this right. I seldom do patches and forget the process in between each time... (In reply to Caroline Cyr La Rose from comment #3) > I hope I did this right. I seldom do patches and forget the process in > between each time... Looks like you remembered the process, as it works! 8-) However, the script name now shows in the system preference as 'merge_authority.pl', rather than 'merge_authorities.pl' 8-( The test plan is correct though. If you could fix that up, I'll sign it off. David Created attachment 104457 [details] [review] Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Test plan: 1- Apply patch 2- Go to Administration > Global system preferences > Authorities 3- The path to merge_authorities.pl in the AuthorityMergeLimit description should now read misc/cronjobs/merge_authorities.pl Hi David! Good thing you have eagle eyes! I have corrected the whole thing again (cronjobs + authorities) Created attachment 104458 [details] [review] Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Test plan: 1- Apply patch 2- Go to Administration > Global system preferences > Authorities 3- The path to merge_authorities.pl in the AuthorityMergeLimit description should now read misc/cronjobs/merge_authorities.pl Signed-off-by: David Nind <david@davidnind.com> (In reply to Caroline Cyr La Rose from comment #6) > Good thing you have eagle eyes! Only sometimes, today was your lucky day! 8-) Created attachment 105300 [details] [review] Bug 25394: Cronjob path in the AuthorityMergeLimit syspref description is wrong Test plan: 1- Apply patch 2- Go to Administration > Global system preferences > Authorities 3- The path to merge_authorities.pl in the AuthorityMergeLimit description should now read misc/cronjobs/merge_authorities.pl Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 20.11, thanks to everybody involved! backported to 20.05.x for 20.05.01 backported to 19.11.x for 19.11.07 Backported to 19.05.x branch for 19.05.13 |