Bug 22875

Summary:	Documentation misleading for import_patrons command line script
Product:	Koha	Reporter:	Colin Campbell <colin.campbell>
Component:	Command-line Utilities	Assignee:	Colin Campbell <colin.campbell>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	trivial
Priority:	P5 - low	CC:	martin.renvoize, nick, robin
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	String patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:	19.05.00, 18.11.06
Circulation function:
Attachments:	Document verbose option more fully Bug 22875: Document verbose option in import_patrons

Description Colin Campbell 2019-05-09 10:31:36 UTC

In the import_patrons.pl command line script the documentation for the --verbose option merely states Be verbose it does not indicate that the option is repeatable and that 2 repetitions are required to get a report of lines that were rejected. This is also different from some previous versions which reported failed lines on a single --verbose

Comment 1 Colin Campbell 2019-05-09 10:38:30 UTC

Created attachment 89497 [details] [review]
Document verbose option more fully

Trivial documentation change
Clarifies usage of verbose option

Comment 2 Katrin Fischer 2019-05-11 23:19:52 UTC

Created attachment 89625 [details] [review]
Bug 22875: Document verbose option in import_patrons

Docs did not state that verbose can be repeated or that you
need more than 1 repetition to see lines that have failed

To test:
- Run ./import_patrons.pl -h
- Verify the additional documentation for -v is shown.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Comment 3 Katrin Fischer 2019-05-11 23:20:17 UTC

Documentation change in CLI script, moving straight to PQA.

Comment 4 Nick Clemens (kidclamp) 2019-05-15 12:17:30 UTC

Awesome work all!

Pushed to master for 19.05

Comment 5 Martin Renvoize (ashimema) 2019-05-30 19:12:49 UTC

Pushed to 18.11.x or 18.11.06