Bug 39733

Summary: Update code comment with a TODO in misc/cronjobs/staticfines.pl
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: Command-line UtilitiesAssignee: Fridolin Somers <fridolin.somers>
Status: Pushed to main --- QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: trivial    
Priority: P5 - low CC: david, martin.renvoize, robin
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:
Version(s) released in:
25.05.00
Circulation function:
Bug Depends on:    
Bug Blocks: 39720    
Attachments: Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl
Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl
Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl

Description Fridolin Somers 2025-04-24 09:21:53 UTC
Bug 27486 renamed system preference "delimiter" into "CSVDelimiter"
fix in misc/cronjobs/staticfines.pl
Comment 1 Fridolin Somers 2025-04-24 09:25:15 UTC
Oh this is commented code, I replace with a TODO text
Comment 2 Fridolin Somers 2025-04-24 09:27:28 UTC
Created attachment 181415 [details] [review]
Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl

Bug 27486 renamed system preference "delimiter" into "CSVDelimiter"

This patch just replaces the commented code with a TODO text
Comment 3 David Nind 2025-04-25 22:03:06 UTC
Created attachment 181558 [details] [review]
Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl

Bug 27486 renamed system preference "delimiter" into "CSVDelimiter"

This patch just replaces the commented code with a TODO text

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Martin Renvoize (ashimema) 2025-05-13 09:09:34 UTC
Created attachment 182336 [details] [review]
Bug 39733: Remove system preference delimiter mention in misc/cronjobs/staticfines.pl

Bug 27486 renamed system preference "delimiter" into "CSVDelimiter"

This patch just replaces the commented code with a TODO text

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 5 Katrin Fischer 2025-05-15 06:36:12 UTC
Pushed for 25.05!

Well done everyone, thank you!