Bug 10894 - ID required for mainaddress fieldset in create new patron
Summary: ID required for mainaddress fieldset in create new patron
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Colin Campbell
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-09-16 15:40 UTC by David Roberts
Modified: 2014-05-26 21:04 UTC (History)
4 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:
Circulation function:

Attachments
proposed patch (1.08 KB, patch)
2013-09-16 15:56 UTC, Colin Campbell 		Details | Diff | Splinter Review
[SIGNED-OFF] Bug 10894 Add id to main address in member entry (1.17 KB, patch)
2013-09-16 18:57 UTC, Owen Leonard 		Details | Diff | Splinter Review
Bug 10894 Add id to main address in member entry (1.17 KB, patch)
2013-09-20 16:52 UTC, Kyle M Hall (khall) 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Roberts 2013-09-16 15:40:28 UTC 
To allow customisation with jQuery.
Comment 1 Colin Campbell 2013-09-16 15:56:12 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2013-09-16 18:57:58 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall (khall) 2013-09-20 16:52:02 UTC 
Created attachment 21284 [details] [review]
Bug 10894 Add id to main address in member entry

Add an id to wase customization via jquery, an id is already present
on alt address but missing from main address

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Galen Charlton 2013-09-20 17:20:28 UTC 
Pushed to master.  Thanks, David!  This looks like your first code contribution to Koha, so congratulations!
Comment 5 Tomás Cohen Arazi (tcohen) 2013-11-11 15:14:48 UTC 
This patch has been pushed to 3.12.x, will be in 3.12.7.

Thanks David! Congratulations for your first commit!