Summary: | Minor improvements on code documentation (omnibus) | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Architecture, internals, and plumbing | Assignee: | Marcel de Rooy <m.de.rooy> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | gmcharlt, jonathan.druart, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 11856: Add confirm option to POD in advance_notices.pl
[Signed off] Bug 11856: Add confirm option to POD in advance_notices.pl Bug 11856: Add confirm option to POD in advance_notices.pl Bug 11856: Add confirm option to POD in advance_notices.pl |
Description
Marcel de Rooy
2014-02-27 08:26:11 UTC
Created attachment 27232 [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. Created attachment 28139 [details] [review] [Signed off] 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> Created attachment 28279 [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> 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> Pushed to master and 3.16.x. Thanks, Marcel! |