what _parts will look like AND for which _render{Part} methods exist.
To extend Zend_Db_Select for a new SQL part, you'd have to include its
name (as a new constant) into the _partsInit array as well as the
corresponding _render{Part}() method. Basically, its a way of keeping
developers honest when it comes to extending Zend_Db_Select for new
functionality. (It also is required for resetting values.)
Out of curiosity, why do you feel this is a bug? Is it hindering a user
case?
-ralph
Mathieu Suen wrote:
> Hi,
>
> I think that the Zend_Db_Select::assemble method is bugy.
> I would expected something like:
>
> /**
> * Converts this object to an SQL SELECT string.
> *
> * @return string This object as a SELECT string.
> */
> public function assemble()
> {
> $sql = self::SQL_SELECT;
> foreach (array_keys($this->_parts) as $part) {
> $method = '_render' . ucfirst($part);
> if (method_exists($this, $method)) {
> $sql = $this->$method($sql);
> }
> }
> return $sql;
> }
>
>
> If so I can create un bug report. With the atteched patch
>
> Thanks
>
没有评论:
发表评论