appdb/include monitor.php

[WineHQ]

ChangeSet ID:	30947
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/03/09 21:36:28

Modified files:
	include        : monitor.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
	Don't allow duplicate monitors, check for duplicates in monitor::create()

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

Old revision  New revision  Changes     Path
 1.12          1.13          +6 -0       appdb/include/monitor.php

Index: appdb/include/monitor.php
diff -u -p appdb/include/monitor.php:1.12 appdb/include/monitor.php:1.13
--- appdb/include/monitor.php:1.12	10 Mar 2007  3:36:28 -0000
+++ appdb/include/monitor.php	10 Mar 2007  3:36:28 -0000
@@ -60,6 +60,12 @@ class Monitor {
      */
     function create($iUserId, $iAppId=0, $iVersionId=0)
     {
+        /* Check for duplicate entries */
+        $oMonitor = new monitor();
+        $oMonitor->find($iUserId, $iVersionId);
+        if($oMonitor->iVersionId)
+            return FALSE;
+
         $hResult = query_parameters("INSERT INTO appMonitors (versionId, appId, userId) ".
                                     "VALUES ('?', '?', '?')",
                                     $iVersionId, $iAppId, $iUserId);