appdb/ ./objectManager.php include/objectManag ...

WineHQ wineowner at wine.codeweavers.com
Sun Apr 22 10:52:06 CDT 2007


ChangeSet ID:	31086
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/04/22 10:52:06

Modified files:
	.              : objectManager.php 
	include        : objectManager.php 

Log message:
	Chris Morgan <cmorgan at alum.wpi.edu>
	Fix some inconsistent braces and add a comment to the objectManager code

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

Old revision  New revision  Changes     Path
 1.5           1.6           +2 -1       appdb/objectManager.php
 1.14          1.15          +5 -2       appdb/include/objectManager.php

Index: appdb/objectManager.php
diff -u -p appdb/objectManager.php:1.5 appdb/objectManager.php:1.6
--- appdb/objectManager.php:1.5	22 Apr 2007 15:52: 6 -0000
+++ appdb/objectManager.php	22 Apr 2007 15:52: 6 -0000
@@ -84,8 +84,9 @@ if($oObject->iId)
         break;
     }
 } else if ($aClean['sAction'] == "add")
+{
     $oObject->add_entry($REQUEST_URI, $sErrors);
-else
+} else
 {
     // if displaying a queue display the help for the given queue
     if($oObject->bIsQueue)
Index: appdb/include/objectManager.php
diff -u -p appdb/include/objectManager.php:1.14 appdb/include/objectManager.php:1.15
--- appdb/include/objectManager.php:1.14	22 Apr 2007 15:52: 6 -0000
+++ appdb/include/objectManager.php	22 Apr 2007 15:52: 6 -0000
@@ -300,6 +300,8 @@ class ObjectManager
         switch($aClean['sSubmit'])
         {
             case "Submit":
+                // if we have a valid iId then we are displaying an existing entry
+                // otherwise we should create the entry in the 'else' case
                 if($this->iId)
                 {
                     if(!$oObject->canEdit())
@@ -312,9 +314,10 @@ class ObjectManager
                         $oObject->unQueue();
 
                     $oObject->update();
-                }
-                else
+                } else
+                {
                     $oObject->create();
+                }
             break;
 
             case "Reject":



More information about the wine-cvs mailing list