Bugzilla – Attachment 34503 Details for
Bug 11879
Add a new field to patron record: main contact method
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[Signed-off] Fix syspref BorrowerUnwantedField
Signed-off-Fix-syspref-BorrowerUnwantedField.patch (text/plain), 7.31 KB, created by
Marc Véron
on 2014-12-18 04:17:55 UTC
(
hide
)
Description:
[Signed-off] Fix syspref BorrowerUnwantedField
Filename:
MIME Type:
Creator:
Marc Véron
Created:
2014-12-18 04:17:55 UTC
Size:
7.31 KB
patch
obsolete
>From d583e268c4ce6424cbb276c7d405e9e798dafd32 Mon Sep 17 00:00:00 2001 >From: simith <simith@inlibro.com> >Date: Wed, 17 Dec 2014 09:16:32 -0500 >Subject: [PATCH] [Signed-off] Fix syspref BorrowerUnwantedField >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >How I tested: >Apply 1st patch >Apply this patch >Edit syspref BorrowerUnwantedField to exclude fields >Edit patron's contact information >Make sure that fields do not longer appear in drop down for Main contact > >Wors as expected > >http://bugs.koha-community.org/show_bug.cgi?id=11879 >Signed-off-by: Marc Véron <veron@veron.ch> >--- > installer/data/mysql/updatedatabase.pl | 13 ++-- > .../prog/en/modules/members/memberentrygen.tt | 68 ++++++++++++-------- > members/memberentry.pl | 12 ++++ > 3 files changed, 60 insertions(+), 33 deletions(-) > >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index 15b58ce..654c02a 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -8672,12 +8672,6 @@ if ( CheckVersion($DBversion) ) { > print "Upgrade to $DBversion done (Bug 10402: Move bookseller contacts to separate table)\n"; > SetVersion($DBversion); > } >-$DBversion = "XXX"; >-if ( CheckVersion($DBversion) ) { >- $dbh->do("ALTER TABLE `borrowers` ADD COLUMN `primary_contact_method` VARCHAR(45) NULL DEFAULT NULL;"); >- print "Upgrade to $DBversion done (Bug 11879: Add a new borrower field : main contact method)\n"; >- SetVersion($DBversion); >-} > > $DBversion = "3.17.00.017"; > if ( CheckVersion($DBversion) ) { >@@ -9591,6 +9585,13 @@ if ( CheckVersion($DBversion) ) { > SetVersion ($DBversion); > } > >+$DBversion = "XXX"; >+if ( CheckVersion($DBversion) ) { >+ $dbh->do("ALTER TABLE `borrowers` ADD COLUMN `primary_contact_method` VARCHAR(45) NULL DEFAULT NULL;"); >+ print "Upgrade to $DBversion done (Bug 11879: Add a new borrower field : main contact method)\n"; >+ SetVersion($DBversion); >+} >+ > =head1 FUNCTIONS > > =head2 TableExists($table) >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt >index b2eff65..e0cfc73 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt >@@ -637,7 +637,7 @@ > [% IF ( mandatoryemailpro ) %]<span class="required">Required</span>[% END %] > </li> > [% END %] >- [% UNLESS nofax %] >+ [% UNLESS nofax %] > <li> > [% IF ( mandatoryfax ) %] > <label for="fax" class="required"> >@@ -648,44 +648,58 @@ > <input type="text" id="fax" name="fax" value="[% fax %]" /> > [% IF ( mandatoryfax ) %]<span class="required">Required</span>[% END %] > </li> >+ [% END %] > <li> > <label for="primary_contact_method">Main Contact Method</label> > <select id="primary_contact_method" name="primary_contact_method"> > <option value=""></option> >- [% IF ( primary_contact_method == 'phone' ) %] >- <option value="phone" selected="selected">Primary phone</option> >- [% ELSE %] >- <option value="phone">Primary phone</option> >+ [% FOREACH contactprincipalloo IN contactprincipalloop %] >+ [% IF ( contactprincipalloo.currentis_phone ) %] >+ [% IF ( primary_contact_method == 'phone' ) %] >+ <option value="phone" selected="selected">Primary phone</option> >+ [% ELSE %] >+ <option value="phone">Primary phone</option> >+ [% END %] > [% END %] >- [% IF ( primary_contact_method == 'phonepro' ) %] >- <option value="phonepro" selected="selected">Secondary phone</option> >- [% ELSE %] >- <option value="phonepro">Secondary phone</option> >+ [% IF ( contactprincipalloo.currentis_phonepro ) %] >+ [% IF ( primary_contact_method == 'phonepro' ) %] >+ <option value="phonepro" selected="selected">Secondary phone</option> >+ [% ELSE %] >+ <option value="phonepro">Secondary phone</option> >+ [% END %] > [% END %] >- [% IF ( primary_contact_method == 'mobile' ) %] >- <option value="mobile" selected="selected">Other phone</option> >- [% ELSE %] >- <option value="mobile">Other phone</option> >+ [% IF ( contactprincipalloo.currentis_mobile ) %] >+ [% IF ( primary_contact_method == 'mobile' ) %] >+ <option value="mobile" selected="selected">Mobile phone</option> >+ [% ELSE %] >+ <option value="mobile">Mobile phone</option> >+ [% END %] > [% END %] >- [% IF ( primary_contact_method == 'email' ) %] >- <option value="email" selected="selected">Primary email</option> >- [% ELSE %] >- <option value="email">Primary email</option> >+ [% IF ( contactprincipalloo.currentis_email ) %] >+ [% IF ( primary_contact_method == 'email' ) %] >+ <option value="email" selected="selected">Primary email</option> >+ [% ELSE %] >+ <option value="email">Primary email</option> >+ [% END %] > [% END %] >- [% IF ( primary_contact_method == 'emailpro' ) %] >- <option value="emailpro" selected="selected">Secondary email</option> >- [% ELSE %] >- <option value="emailpro">Secondary email</option> >+ [% IF ( contactprincipalloo.currentis_emailpro ) %] >+ [% IF ( primary_contact_method == 'emailpro' ) %] >+ <option value="emailpro" selected="selected">Secondary email</option> >+ [% ELSE %] >+ <option value="emailpro">Secondary email</option> >+ [% END %] > [% END %] >- [% IF ( primary_contact_method == 'fax' ) %] >- <option value="fax" selected="selected">Fax</option> >- [% ELSE %] >- <option value="fax">Fax</option> >+ [% IF ( contactprincipalloo.currentis_fax ) %] >+ [% IF ( primary_contact_method == 'fax' ) %] >+ <option value="fax" selected="selected">Fax</option> >+ [% ELSE %] >+ <option value="fax">Fax</option> >+ [% END %] > [% END %] >+ [% END %] > </select> > </li> >- [% END %] >- </ol> >+ </ol> > </fieldset> > [%END # hide fieldset %] > >diff --git a/members/memberentry.pl b/members/memberentry.pl >index 7a0329f..cd7bd7a 100755 >--- a/members/memberentry.pl >+++ b/members/memberentry.pl >@@ -547,6 +547,18 @@ foreach (qw(C A S P I X)) { > $template->param('typeloop' => \@typeloop, > no_categories => $no_categories); > if($no_categories){ $no_add = 1; } >+ >+my @contactprincipalloop; >+my @fieldArray = qw(phone phonepro mobile email emailpro fax); >+foreach my $field (@fieldArray) { >+ if ( !(grep { $field eq $_ } @field_check)){ >+ push @contactprincipalloop,{ >+ 'currentis_' . $field => 1 >+ }; >+ } >+} >+$template->param('contactprincipalloop' => \@contactprincipalloop); >+ > # test in city > if ( $guarantorid ) { > $select_city = getidcity($data{city}); >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11879
:
25745
|
25922
|
25941
|
31300
|
31301
|
31320
|
34351
|
34482
|
34502
|
34503
|
34571
|
34572
|
34573
|
41209
|
41220
|
41221
|
41222
|
46101
|
46284
|
46285
|
46286
|
48073
|
48074
|
48075
|
48076
|
48077
|
48078
|
48079
|
48556
|
48557
|
48558
|
48559
|
50433
|
52607
|
52608
|
52614
|
52615
|
52616
|
52617
|
52618
|
52619
|
52620
|
53229
|
53688
|
53695
|
53705
|
53707
|
53708
|
53709
|
53710
|
53711
|
53712
|
53713
|
55946
|
56285
|
56286
|
58580
|
58591
|
58592
|
58593
|
63238
|
63239
|
65811
|
65812
|
69028
|
69029
|
69050
|
71955
|
71956
|
71957
|
71958
|
72632
|
72633
|
72634
|
72635
|
73987
|
73988
|
73989
|
73990
|
74123
|
74124
|
74125
|
74126
|
74818
|
75652
|
83757
|
83758
|
84017
|
84018
|
84019
|
84020
|
84021
|
84022
|
88358
|
88359
|
88360
|
88361
|
88362
|
88363
|
88394
|
88395
|
88396
|
88397
|
88398
|
88399
|
88400
|
93850
|
93851
|
93852
|
93853
|
93854
|
93855
|
93856
|
93900
|
93901
|
94456
|
94457
|
94458
|
94459
|
94460
|
94461
|
94463
|
94464
|
94465
|
94466
|
94467
|
94468
|
96432
|
100817
|
100843
|
112062
|
112063
|
112064
|
112557
|
112559
|
112560
|
118959
|
118960
|
118961
|
118962
|
118985
|
118986
|
118987
|
118988
|
120947
|
120948
|
120949
|
120950
|
122778