[Tickets #2794] NEW: multipart viewer against documentation

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2794
-----------------------------------------------------------------------
 Ticket             | 2794
 Created By         | bret at hotmail.com
 Summary            | multipart viewer against documentation
 Queue              | IMP
 Version            | HEAD
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


bret at hotmail.com (2005-10-17 02:37) wrote:

this is the code in the imp/lib/MIME/Viewer/multipart.php:

  /**
     * Render out the currently set contents.
     *
     * @access public
     *
     * @param array $params  An array with a reference to a MIME_Contents
     *                       object.
     *
     * @return string  The rendered text in HTML.
     */
    function render($params)
    {
        $contents = &$params[0];
		
        foreach ($this->mime_part->getParts() as $part) {
          $contents->buildMessagePart($part);
            
        }
     
    }

notice the missing return statement. how could this work? I've checked and
the caller actually uses the returned value as if it was actually a string