appdb/include application.php

WineHQ wineowner at wine.codeweavers.com
Sat Jan 28 17:04:21 CST 2006


ChangeSet ID:	22584
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2006/01/28 17:04:21

Modified files:
	include        : application.php 

Log message:
	Tony Lambregts <tony.lambregts at gmail.com>
	Application class fixes.  Let update be silent when we want it to be.  Get rid of redundant message in unqueue().  Fix logic in SendNotificationMail().

Patch: http://cvs.winehq.org/patch.py?id=22584

Old revision  New revision  Changes     Path
 1.40          1.41          +4 -7       appdb/include/application.php

Index: appdb/include/application.php
diff -u -p appdb/include/application.php:1.40 appdb/include/application.php:1.41
--- appdb/include/application.php:1.40	28 Jan 2006 23: 4:21 -0000
+++ appdb/include/application.php	28 Jan 2006 23: 4:21 -0000
@@ -138,7 +138,7 @@ class Application {
      * Update application.
      * Returns true on success and false on failure.
      */
-    function update()
+    function update($bSilent=false)
     {
         $sWhatChanged = "";
 
@@ -200,7 +200,7 @@ class Application {
             $oCatAfter = new Category($this->iCatId);
             $sWhatChanged .= "Vendor was changed from ".$oCatBefore->sName." to ".$oCatAfter->sName.".\n\n";
         }
-        if($sWhatChanged)
+        if($sWhatChanged and !$bSilent)
             $this->SendNotificationMail("edit",$sWhatChanged);
         return true;
     }
@@ -265,9 +265,6 @@ class Application {
             // we send an e-mail to intersted people
             $this->mailSubmitter();
             $this->SendNotificationMail();
-
-            // the application has been unqueued
-            addmsg("The application has been unqueued.", "green");
         }
     }
 
@@ -355,7 +352,7 @@ class Application {
         switch($sAction)
         {
             case "add":
-                if(!$this->sQueued == 'true')
+                if($this->sQueued == 'false') // Has been accepted.
                 {
                     $sSubject = $this->sName." has been added by ".$_SESSION['current']->sRealname;
                     $sMsg  = APPDB_ROOT."appview.php?appId=".$this->iAppId."\n";
@@ -366,7 +363,7 @@ class Application {
                         $sMsg .= "\n";
                     }
                     addmsg("The application was successfully added into the database.", "green");
-                } else // Application queued.
+                } else
                 {
                     $sSubject = $this->sName." has been submitted by ".$_SESSION['current']->sRealname;
                     $sMsg .= "This application has been queued.";



More information about the wine-cvs mailing list