Bugzilla – Attachment 21669 Details for
Bug 10611
C4::Context->dbh checks if the DB is still running
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 10611: Followup: remove useless calls to dbh->disconnect
SIGNED-OFF-Bug-10611-Followup-remove-useless-calls.patch (text/plain), 1.32 KB, created by
Bernardo Gonzalez Kriegel
on 2013-09-30 17:28:56 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 10611: Followup: remove useless calls to dbh->disconnect
Filename:
MIME Type:
Creator:
Bernardo Gonzalez Kriegel
Created:
2013-09-30 17:28:56 UTC
Size:
1.32 KB
patch
obsolete
>From d69df798c1d391d7be5ee08c3f4cc31298bb0d27 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Mon, 30 Sep 2013 13:46:39 +0200 >Subject: [PATCH] [SIGNED-OFF] Bug 10611: Followup: remove useless calls to > dbh->disconnect > >These 3 calls to disconnect are done at the end of the script, they are >useless. > >Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> >--- > installer/install.pl | 2 -- > misc/cronjobs/j2a.pl | 1 - > sms/sms_listen.pl | 1 - > 3 files changed, 4 deletions(-) > >diff --git a/installer/install.pl b/installer/install.pl >index 58388cb..f682e95 100755 >--- a/installer/install.pl >+++ b/installer/install.pl >@@ -382,8 +382,6 @@ elsif ( $step && $step == 3 ) { > ); > } > } >- >- $dbh->disconnect; > } > } > else { >diff --git a/misc/cronjobs/j2a.pl b/misc/cronjobs/j2a.pl >index 87edaa4..75d4da1 100755 >--- a/misc/cronjobs/j2a.pl >+++ b/misc/cronjobs/j2a.pl >@@ -234,4 +234,3 @@ if ( not $noaction) { > } > $sth->finish( ); > } >-$dbh->disconnect(); >diff --git a/sms/sms_listen.pl b/sms/sms_listen.pl >index b36b25d..a7420d9 100755 >--- a/sms/sms_listen.pl >+++ b/sms/sms_listen.pl >@@ -108,7 +108,6 @@ foreach my $user(@phones){ > } > print $reply; > } >-$dbh->disconnect; > > sub send_message { > my ($mes,$message,$smsid)=@_; >-- >1.7.9.5
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 10611
:
19765
|
20273
|
21264
|
21655
|
21656
|
21657
|
21667
|
21668
|
21669
|
22103