<p>Maybe you can implement one for Propel too?</p>

<p><a href="http://www.strictlyphp.com/blog/2008/09/19/zend_pagination-adapter-for-propel/">I've written one for Propel</a> which uses the symfony DbFinder plugin to mimic Doctrine's query format.<br />
But it is probably better to implement without that plugin. <ac:emoticon ac:name="smile" /></p>

<p>Maybe it would also be a good idea to pass a Doctrine_Collection to the adapater, additional to Doctrine_Query. This would be a more generic approach.</p>

<p>Jason, your examples contain calls to setRouteName(), but those were only present in pre-release versions of Zend_Paginator. You should probably update your examples to reflect that.</p>

<p>Otherwise, looks good. <ac:emoticon ac:name="smile" /></p>

<ac:macro ac:name="note"><ac:parameter ac:name="title">Zend Official Response</ac:parameter><ac:rich-text-body>

<p>This proposal is accepted to the <strong>Zend Extras Incubator</strong> for development.</p>

<p>We would like to see the following concerns met:</p>
<ul>
<li>This proposal and the proposal for <a href="http://framework.zend.com/wiki/display/ZFPROP/Zend_Auth_Adapter_Doctrine_Table">Zend_Auth_Adapter_Doctrine_Table</a> are combined into the ZendX_Doctrine namespace.</li>
<li>Name changed to ZendX_Doctrine_Paginator_Adapter or something similar</li>
<li>With this ZendX_Doctrine namespace, we would like to explore the possibility of allowing the top level class (presumably ZendX_Doctrine) to set up (bootstrap) this component for usage with doctrine- as well as checks to ensure that doctrine is available for use.</li>
</ul>

</ac:rich-text-body></ac:macro>

<p>Zend_Paginator_Adapter_Propel is now 'ready for review'. </p>

<p>Jason, what is the status of this proposal? Do you plan to get this in to the 1.8 release?</p>

<p>That would be awesome!</p>

<p>I almost forgot about this thing. I was recently asked about it on IRC, so I will finish it now if possible. Not sure though want needs to be done except the tests... (any dependencies?) </p>

<p>... Was actually referring to this proposal: <a class="external-link" href="http://framework.zend.com/wiki/pages/viewpage.action?pageId=3866950">http://framework.zend.com/wiki/pages/viewpage.action?pageId=3866950</a></p>

<p>I would strongly suggest ZendX_Doctrine_Paginator or ZendX_Doctrine_PaginatorAdapter for the name. (Same goes for Propel.)</p>

<p>I have found a bug, by the way. Really, it's a bug in Doctrine, but you can fix it in this adapter. Namely that Doctrine doesn't check for null values before trying to set them in places where they will cause problems.</p>

<ac:macro ac:name="code"><ac:default-parameter>php</ac:default-parameter><ac:plain-text-body><![CDATA[
/**

• Returns a collection of items for a page.
  *
• @param integer $offset Page offset
• @param integer $itemCountPerPage Number of items per page
• @return Doctrine_Collection
  */
  public function getItems($offset, $itemCountPerPage)
  {
  if ($offset !== null)
  Unknown macro: { $this->_query->offset($offset); }

  if ($itemCountPerPage !== null)

  Unknown macro: { $this->_query->limit($itemCountPerPage); }

  return $this->_query->execute();
  }
  ]]></ac:plain-text-body></ac:macro>

  <p>If we want to return an array, we should do this instead:</p>

  <ac:macro ac:name="code"><ac:default-parameter>php</ac:default-parameter><ac:plain-text-body><![CDATA[
  /**

  • Returns an array of items for a page.
    *
  • @param integer $offset Page offset
  • @param integer $itemCountPerPage Number of items per page
  • @return array
    */
    public function getItems($offset, $itemCountPerPage)
    {
    if ($offset !== null)

if ($itemCountPerPage !== null)

return $this->_query->execute(array(), Doctrine::HYDRATE_ARRAY);
}
]]></ac:plain-text-body></ac:macro>

<p>Either way, the collection should handle like an array.</p>

<p>One more comment.</p>

<p>Hydration should be optional to the user with a method like setHydrationMode(). I would recommend setting up the controller like this:</p>

<ac:macro ac:name="code"><ac:default-parameter>php</ac:default-parameter><ac:plain-text-body><![CDATA[
/**

• Constructor.
  *
• @param Doctrine_Query $query The select query
• @param integer $hydrationMode Hydration mode
  */
  public function __construct(Doctrine_Query $query, $hydrationMode = null)
  {
  $this->_query = $query;

if ($hydrationMode !== null)

}
]]></ac:plain-text-body></ac:macro>

<p>Just wanted to say that this component works fantastic <ac:emoticon ac:name="smile" /><br />
And that I also love the idea to pass the Doctrine_Collection to the adapater, additional to Doctrine_Query</p>

<p>Is there a working version somewhere?</p>