On Thursday 29 June 2006 1:51 pm, Tony Lambregts wrote:
This patch breaks the monitor notification emails.
more specificly...
diff -u -p appdb/include/monitor.php:1.5 appdb/include/monitor.php:1.6 --- appdb/include/monitor.php:1.5 Thu Jun 29 17:43:08 2006 +++ appdb/include/monitor.php Thu Jun 29 17:43:08 2006 @@ -93,19 +93,25 @@ class Monitor {
function SendNotificationMail($sAction="add",$sMsg=null) {
/* set $aAppName appropriately */if(isset($this->iVersionId))$sAppName = Application::lookup_name($oVersion->iAppId)."".Version::lookup_name($this->iVersionId); + else
$sAppName = Application::lookup_name($oVersion->iAppId);It looks like Application::lookup_name($oVersion->iAppId)returns "". Also people can only monitor application versions they cannot monitor application familys at this time.
Ahh yes. Using the version before the object exists. Fixing that.
Are they supposed to be able to monitor application familes? I'm confused as to whether that is a bug or just how things are working.
Chris