Summary: | Remove C4::Dates from members/moremember.pl | ||
---|---|---|---|
Product: | Koha | Reporter: | Marc Véron <veron> |
Component: | Architecture, internals, and plumbing | Assignee: | Marc Véron <veron> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | hector.hecaxmmx, m.de.rooy, tomascohen |
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: | |||
Bug Depends on: | |||
Bug Blocks: | 14870, 14990 | ||
Attachments: |
Bug 14925 - Remove C4::Dates from members/moremember.pl
Bug 14925 - Remove C4::Dates from members/moremember.pl [SIGNED-OFF]Bug 14925 - Remove C4::Dates from members/moremember.pl Bug 14925 - Remove C4::Dates from members/moremember.pl Bug 14925: (QA followup) tidy code block |
Description
Marc Véron
2015-09-29 16:13:38 UTC
Created attachment 42954 [details] [review] Bug 14925 - Remove C4::Dates from members/moremember.pl This patch removes C4::Dates from members/moremember.pl To test: - Go to Details page of several members - Verify that all dates display OK, including restrictions Created attachment 42955 [details] [review] Bug 14925 - Remove C4::Dates from members/moremember.pl This patch removes C4::Dates from members/moremember.pl To test: - Go to Details page of several members - Verify that all dates display OK, including restrictions Just a dumb question: But wouldn't it be easier to combine all those patches a little bit more? For instance per module or so? Every report needs its own signoff and qa etc.. Created attachment 43284 [details] [review] [SIGNED-OFF]Bug 14925 - Remove C4::Dates from members/moremember.pl This patch removes C4::Dates from members/moremember.pl To test: - Go to Details page of several members - Verify that all dates display OK, including restrictions Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised. All dates displayed OK Created attachment 43288 [details] [review] Bug 14925 - Remove C4::Dates from members/moremember.pl This patch removes C4::Dates from members/moremember.pl To test: - Go to Details page of several members - Verify that all dates display OK, including restrictions Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised. All dates displayed OK Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 43299 [details] [review] Bug 14925: (QA followup) tidy code block Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Patch pushed to master. Thanks Marc! |