Bug 11856 - Minor improvements on code documentation (omnibus)
Summary: Minor improvements on code documentation (omnibus)
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Marcel de Rooy
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-27 08:26 UTC by Marcel de Rooy
Modified: 2015-06-04 23:30 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 11856: Add confirm option to POD in advance_notices.pl (1.72 KB, patch)
2014-04-17 10:06 UTC, Marcel de Rooy
Details | Diff | Splinter Review
[Signed off] Bug 11856: Add confirm option to POD in advance_notices.pl (1.86 KB, patch)
2014-05-09 14:42 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 11856: Add confirm option to POD in advance_notices.pl (1.84 KB, patch)
2014-05-15 13:18 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 11856: Add confirm option to POD in advance_notices.pl (1.91 KB, patch)
2014-05-15 13:18 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel de Rooy 2014-02-27 08:26:11 UTC
This is meant for submitting minor documentation improvemens in code.
If they are trivial, and you "tested" the fix yourself (of course ;), please add your own signoff. The QA team will most probably not mind.
Comment 1 Marcel de Rooy 2014-04-17 10:06:16 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2014-05-09 14:42:19 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-05-15 13:18:21 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-05-15 13:18:40 UTC
Created attachment 28280 [details] [review]
Bug 11856: Add confirm option to POD in advance_notices.pl

Also removes a commented line that changed the value of $confirm.
If it would default to 1, we do not need it at all.
I am not saying that we need it, but just documenting that we have it.

Small change in documentation.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Galen Charlton 2014-05-23 13:54:19 UTC
Pushed to master and 3.16.x.  Thanks, Marcel!