mirror of
https://gitlab.archlinux.org/archlinux/aurweb.git
synced 2025-02-03 10:43:03 +01:00
Use non-persistant connections
There is really no need to use persistent connections to the database in this day and age. Most PHP development guides recommend against it, and the new mysqli interface doesn't even include the functionality. Add a matching but currently unused db_disconnect() function while we are at it. Reference counting will cover us for the most part, however. Signed-off-by: Dan McGee <dan@archlinux.org> Signed-off-by: Loui Chang <louipc.ist@gmail.com>
This commit is contained in:
parent
94bfae4f03
commit
3b4662a58e
1 changed files with 15 additions and 4 deletions
|
@ -261,8 +261,7 @@ function uid_from_sid($sid="") {
|
|||
# connect to the database
|
||||
#
|
||||
function db_connect() {
|
||||
|
||||
$handle = mysql_pconnect(AUR_db_host, AUR_db_user, AUR_db_pass);
|
||||
$handle = mysql_connect(AUR_db_host, AUR_db_user, AUR_db_pass);
|
||||
if (!$handle) {
|
||||
die("Error connecting to AUR database: " . mysql_error());
|
||||
}
|
||||
|
@ -273,15 +272,27 @@ function db_connect() {
|
|||
return $handle;
|
||||
}
|
||||
|
||||
# disconnect from the database
|
||||
# this won't normally be needed as PHP/reference counting will take care of
|
||||
# closing the connection once it is no longer referenced
|
||||
#
|
||||
function db_disconnect($db_handle="") {
|
||||
if ($db_handle) {
|
||||
mysql_close($db_handle);
|
||||
return TRUE;
|
||||
}
|
||||
return FALSE;
|
||||
}
|
||||
|
||||
# wrapper function around db_query in case we want to put
|
||||
# query logging/debuggin in.
|
||||
# query logging/debugging in.
|
||||
#
|
||||
function db_query($query="", $db_handle="") {
|
||||
if (!$query) {
|
||||
return FALSE;
|
||||
}
|
||||
if (!$db_handle) {
|
||||
$db_handle = db_connect();
|
||||
die("DB handle was not provided to db_query");
|
||||
}
|
||||
$result = @mysql_query($query, $db_handle);
|
||||
return $result;
|
||||
|
|
Loading…
Add table
Reference in a new issue