Bug 30349

Summary: Cleanup bulkmarcimport.pl
Product: Koha Reporter: davewood
Component: MARC Bibliographic record staging/importAssignee: davewood
Status: Pushed to main --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, lucas, martin.renvoize
Version: MainKeywords: release-notes-needed
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29440
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00
Bug Depends on: 25539    
Bug Blocks:    
Attachments: patch 0001 fix indentation
use double dashes
more double dashes
Bug 30349: Tidy bulkmarcimport.pl to fix indentations
Bug 30349: Use double dash for multi character option names
Bug 30349: More double dashes
Bug 30349: Tidy bulkmarcimport.pl to fix indentations
Bug 30349: Use double dash for multi character option names
Bug 30349: More double dashes
Bug 30349: Add missing double dash (--delete) to POD

Description davewood 2022-03-24 11:09:29 UTC
the help page of bulkmarcimport.pl doesnt use double dashes for multi-character options.

Id like to update the help info and add multi-character options where they are missing. (e.g. --help for -h)
Comment 1 davewood 2022-03-24 11:36:05 UTC
Created attachment 132127 [details] [review]
patch 0001 fix indentation
Comment 2 davewood 2022-03-24 11:37:09 UTC
Created attachment 132128 [details] [review]
use double dashes

use double dashes, add some multi char options where only a single char option was available e.g. --help for -h
Comment 3 davewood 2022-03-24 11:37:39 UTC
powered by HKS3 (www.koha-support.eu)
Comment 4 davewood 2022-03-24 12:02:53 UTC
Created attachment 132130 [details] [review]
more double dashes
Comment 5 Katrin Fischer 2022-03-26 20:04:07 UTC
Don't forget to switch to "Needs sign-off" once this is ready for testing!
Comment 6 Lucas Gass 2023-04-14 15:23:54 UTC
A couple failures from the QA script:

 FAIL	misc/migration_tools/bulkmarcimport.pl
   FAIL	  forbidden patterns
		forbidden pattern: tab char (line 579)



Processing additional checks

	* Commit title does not start with 'Bug XXXXX: ' - 23a34f0d0b
	* Commit title does not start with 'Bug XXXXX: ' - 6b73fdeb22
	* Commit title does not start with 'Bug XXXXX: ' - 7d5ade6229


Can these be fixed please?
Comment 7 Martin Renvoize 2024-01-03 16:56:20 UTC
Created attachment 160492 [details] [review]
Bug 30349: Tidy bulkmarcimport.pl to fix indentations

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 8 Martin Renvoize 2024-01-03 16:56:22 UTC
Created attachment 160493 [details] [review]
Bug 30349: Use double dash for multi character option names

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 9 Martin Renvoize 2024-01-03 16:56:24 UTC
Created attachment 160494 [details] [review]
Bug 30349: More double dashes

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 10 Martin Renvoize 2024-01-03 16:57:48 UTC
Rebased, added proper commit lines and signed off
Comment 11 Jonathan Druart 2024-01-12 10:07:24 UTC
Created attachment 160913 [details] [review]
Bug 30349: Tidy bulkmarcimport.pl to fix indentations

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 12 Jonathan Druart 2024-01-12 10:07:26 UTC
Created attachment 160914 [details] [review]
Bug 30349: Use double dash for multi character option names

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 13 Jonathan Druart 2024-01-12 10:07:29 UTC
Created attachment 160915 [details] [review]
Bug 30349: More double dashes

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 14 Jonathan Druart 2024-01-12 10:07:32 UTC
Created attachment 160916 [details] [review]
Bug 30349: Add missing double dash (--delete) to POD

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 15 Katrin Fischer 2024-03-22 09:37:06 UTC
Pushed for 24.05!

Well done everyone, thank you!