summaryrefslogtreecommitdiffstats
path: root/web/html
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-08-10 18:20:07 -0500
committerLukas Fleischer <archlinux@cryptocrack.de>2011-08-11 16:17:08 +0200
commit5d74a994ff3e67bb2d3a3e27828785eb6bda94d9 (patch)
tree3544312c42f058247f6b7635f5c60ad074325753 /web/html
parent0ac74ed7335927639e6d74f83ebc9010f77c6b1f (diff)
downloadaurweb-5d74a994ff3e67bb2d3a3e27828785eb6bda94d9.tar.xz
Make package creation and update atomic
Add BEGIN and COMMIT statements where it makes sense to do so. This allows the entire package creation or update process to be atomic and not be seen until it is complete. Signed-off-by: Dan McGee <dan@archlinux.org> Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web/html')
-rw-r--r--web/html/pkgsubmit.php6
1 files changed, 5 insertions, 1 deletions
diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php
index f7f0dd4..64281c7 100644
--- a/web/html/pkgsubmit.php
+++ b/web/html/pkgsubmit.php
@@ -299,6 +299,7 @@ if ($uid):
if (!$error) {
$dbh = db_connect();
+ db_query("BEGIN", $dbh);
$q = "SELECT * FROM Packages WHERE Name = '" . mysql_real_escape_string($new_pkgbuild['pkgname']) . "'";
$result = db_query($q, $dbh);
@@ -396,9 +397,12 @@ if ($uid):
# If we just created this package, or it was an orphan and we
# auto-adopted, add submitting user to the notification list.
if (!$pdata || $pdata["MaintainerUID"] === NULL) {
- pkg_notify(account_from_sid($_COOKIE["AURSID"]), array($packageID));
+ pkg_notify(account_from_sid($_COOKIE["AURSID"], $dbh), array($packageID), true, $dbh);
}
+ # Entire package creation process is atomic
+ db_query("COMMIT", $dbh);
+
header('Location: packages.php?ID=' . $packageID);
}