Bug 35294

Summary: Typo in comment in C4 circulation: barocode
Product: Koha Reporter: Jessie Zairo <jzairo>
Component: Architecture, internals, and plumbingAssignee: Brendan Lawlor <blawlor>
Status: Pushed to main --- QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: blawlor, david, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes spelling errors in catalog code comments (barocode => barcode, and preproccess => preprocess).
Version(s) released in:
24.11.00
Attachments: Bug 35294: fix typo in catalogue/search.pl
Bug 35294: Fix typo in catalogue/search.pl
Bug 35294: Fix typo in catalogue/search.pl
Bug 35294: Fix typos in catalogue code comments
Bug 35294: Fix typos in catalogue code comments

Description Jessie Zairo 2023-11-08 17:32:51 UTC
There is a typo in comment in C4 circulation barocode
Comment 1 Brendan Lawlor 2024-06-05 20:43:15 UTC
Created attachment 167498 [details] [review]
Bug 35294: fix typo in catalogue/search.pl

This patch fixes a typo in a comment
To recreate:
1. git grep barocode to find the file and line # of the typo
catalogue/search.pl:462:# If index indicates the value is a barocode
To Test:
1. Apply patch
2. git grep barocode
3. See no results
Comment 2 David Nind 2024-06-05 23:53:17 UTC
Created attachment 167504 [details] [review]
Bug 35294: Fix typo in catalogue/search.pl

This patch fixes a typo in a comment.

Test plan:
1. git grep barocode to find the file and line # of the typo:
   catalogue/search.pl:462:# If index indicates the value is a barocode
2. Apply the patch.
3. git grep barocode
4. See no results.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2024-06-05 23:54:43 UTC
Comment on attachment 167498 [details] [review]
Bug 35294: fix typo in catalogue/search.pl

I edited the text of the commit message slightly. Hope that is OK.
Comment 4 Matt Blenkinsop 2024-06-06 13:01:42 UTC
Created attachment 167541 [details] [review]
Bug 35294: Fix typo in catalogue/search.pl

This patch fixes a typo in a comment.

Test plan:
1. git grep barocode to find the file and line # of the typo:
   catalogue/search.pl:462:# If index indicates the value is a barocode
2. Apply the patch.
3. git grep barocode
4. See no results.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Comment 5 Brendan Lawlor 2024-06-06 13:29:03 UTC
Created attachment 167547 [details] [review]
Bug 35294: Fix typos in catalogue code comments

Test plan:
1. git grep -n -E 'barocode|preproccess' to find the files and line # of typos
2. Apply the patch
3. git grep -E 'barocode|proccess'
4. See no results
Comment 6 Brendan Lawlor 2024-06-06 13:34:17 UTC
When I looked at this again I noticed another typo in the comment that appears in both catalogue/search.pl and catalogue/itemsearch.pl

This new patch fixes both typos barocode and preproccess. 

Should I obsolete the first patch and set this back to needs sign off?

Thanks David and Matt.
Comment 7 David Nind 2024-06-06 20:42:33 UTC
Comment on attachment 167541 [details] [review]
Bug 35294: Fix typo in catalogue/search.pl

I think adding a follow-up would have been sufficient.

I've obsoleted the original patch, and will add the sign-off lines to the new patch shortly.
Comment 8 David Nind 2024-06-06 20:44:57 UTC
Created attachment 167556 [details] [review]
Bug 35294: Fix typos in catalogue code comments

Test plan:
1. git grep -n -E 'barocode|preproccess' to find the files and line # of typos
2. Apply the patch
3. git grep -E 'barocode|proccess'
4. See no results

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Comment 9 Martin Renvoize 2024-06-13 13:32:43 UTC
Thanks for all the hard work!

Pushed to main for the next 24.11.00 release as RM Assistant