Module: appdb Branch: master Commit: dacc985d984fa0c87c1893b7fba9eafa153848b6 URL: http://source.winehq.org/git/appdb.git/?a=commit;h=dacc985d984fa0c87c1893b7f...
Author: Alexander Nicolaysen Sørnes <alexander@linux-xqqm.(none)> Date: Wed Nov 11 22:22:35 2009 +0100
Cache whether an app is maintained
---
include/application.php | 9 +++++++++ include/maintainer.php | 25 +++++++++++++++++++++++++ tables/appdb_tables.sql | 1 + 3 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/include/application.php b/include/application.php index 256afa1..3251261 100644 --- a/include/application.php +++ b/include/application.php @@ -39,6 +39,7 @@ class Application { var $sSubmitTime; var $iSubmitterId; var $aVersionsIds; // an array that contains the versionId of every version linked to this app. + var $bHasMaintainer; var $aVersions; // Array of version objects belonging to this app var $iMaintainerRequest; /* Temporary variable for tracking maintainer requests on app submission. Value denotes type of request */ @@ -78,6 +79,7 @@ class Application { // and return an id into the appData table here $this->sWebpage = Url::normalize($oRow->webPage); $this->sState = $oRow->state; + $this->bHasMaintainer = $oRow->hasMaintainer == 'true' ? true : false; }
/* fetch versions of this application, if there are any */ @@ -242,6 +244,13 @@ class Application { return true; }
+ public function updateMaintainerState() + { + $this->bHasMaintainer = maintainer::appHasMaintainer($this->iAppId); + + $hResult = query_parameters("UPDATE appFamily SET hasMaintainer = '?' WHERE appId = '?'", $this->bHasMaintainer ? 'true' : 'false', $this->iAppId); + } + /** * Deletes the application from the database. * and request the deletion of linked elements. diff --git a/include/maintainer.php b/include/maintainer.php index 3a39729..489bafb 100644 --- a/include/maintainer.php +++ b/include/maintainer.php @@ -229,6 +229,9 @@ class maintainer /* this objects id is the insert id returned by the database */ $this->iMaintainerId = query_appdb_insert_id();
+ if(!$this->mustBeQueued()) + $this->updateAppMaintainerState(); + /* If this is a non-queued maintainer submission, remove the user's non- super maintainer entries for the application's versions. This check is also done in unQueue() */ @@ -257,6 +260,8 @@ class maintainer { $sStatusMessage = "<p>The maintainer was successfully added into the database</p>\n";
+ $this->updateAppMaintainerState(); + //Send Status Email $sEmail = $oUser->sEmail; if ($sEmail) @@ -348,6 +353,8 @@ class maintainer if(!$hResult) return FALSE;
+ $this->updateAppMaintainerState(); + return TRUE; }
@@ -406,6 +413,12 @@ class maintainer return $hResult; }
+ public function updateAppMaintainerState() + { + $oApp = new application($this->iAppId); + $oApp->updateMaintainerState(); + } + function getSubmitterEmails() { $sRecipients = ''; @@ -587,6 +600,18 @@ class maintainer return query_num_rows($hResult); }
+ /* Returns true if the given app has a maintainer, false otherwise */ + public function appHasMaintainer($iAppId) + { + $hResult = query_parameters("SELECT COUNT(maintainerId) as count FROM appMaintainers WHERE appId = '?' AND superMaintainer = '1' AND state = 'accepted'", $iAppId); + + if(!$hResult) + return false; + + $oRow = mysql_fetch_object($hResult); + return $oRow->count > 0; + } + /* if given an appid or a version id return a handle for a query that has */ /* the user ids that are maintainers for this particular appid or version id */ function getMaintainersForAppIdVersionId($iAppId = null, $iVersionId = null) diff --git a/tables/appdb_tables.sql b/tables/appdb_tables.sql index f41c5fd..69ab049 100644 --- a/tables/appdb_tables.sql +++ b/tables/appdb_tables.sql @@ -41,6 +41,7 @@ create table appFamily ( submitTime datetime NOT NULL, submitterId int(11) NOT NULL default '0', state enum('accepted','queued','rejected','deleted') NOT NULL default 'accepted', + hasMaintainer enum('true','false') NOT NULL default 'false', key(appId) );
-
Alexander Nicolaysen Sørnes