appdb/ ./appbrowse.php ./appsubmit.php ./appvi ...

[Chris Morgan]

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