Zend FR

Salut,

D'après ton code tu pagines tes données après les avoir toutes remontées.

Il faut passer un select à Zend_Paginator et non pas les données, il se chargera lui même de compléter et d'exécuter la requête comme il se doit.

Tu dois ajouter une méthode spécifique dans ta collection (table).

Par exemple :

public function getDataPaginator($pageSize = 20, $page = 1)
{
    $select = $this->select()
        ->setIntegrityCheck(false)
        ->from(array('m' => 'mails'))
        ->join(array('r' => 'recipients'), 'm.mail_id = r.mail_id')
        ->where("r.recipient_type = 'from'") 
    );

    $paginator = Zend_Paginator::factory($select);
    $paginator->setItemCountPerPage((int)$pageSize);
    $paginator->setCurrentPageNumber((int)$page);
    
    return $paginator;
}

Zend_Paginator implémente l'interface IteratorAggregate, ce qui te permet de passer l'objet paginator dans une boucle foreach par ex. A chaque itération, Zend_Paginator va exécuter une requête pour remonter la liste des items de la page en cours.

Après si on pousse plus loin, je dirais que retourner un objet paginator depuis une collection n'est pas une bonne pratique, mais c'est la méthode ZF

Si c'était pour moi, je créerais moi même les requêtes (ou tu peux utiliser Zend_Paginator) pour retourner une collection de données pour la page en cours + un nombre total d'items et je construirais le paginateur côté front, dans le contrôleur, avec l'adapter "null".

Comme ça je remonte bien une collection d'objets métiers comme il se doit. La pagination est du ressort du frontend. Attention, c'est juste une remarque hein, rien d'obligatoire.


A+ benjamin.

Hello.

Non, ce n'est pas bon

Dans la factory de Zend_Paginator tu passes soit un objet DbSelect qui correspond à la requête originale, soit un chiffre pour utiliser l'adapter NULL.

Là c'est pas bon parce que tu passes un objet DbSelect qui n'a rien à voir avec les données que tu affiches, donc que va faire Zend_Paginator ? Il va considérer que ta requête originale c'est celle retournée par countMails(), il va la compléter pour ajouter les limit et va en exécuter automatiquement une 2ème pour calculer le nombre total d'enregistrements correspondant à cette requête.

De plus la requête qui compte le nombre d'enregistrements total doit être paramétrée comme la requête qui retourne les enregistrements, sinon tu auras un décalage ça c'est certain.

Je pense que je t'ai un peu mélangé les pinceaux avec ma remarque
Comme je te l'ai dit tu peux très bien retourner directement un objet Paginator prêt à l'emploi depuis ta DbTable si une séparation stricte des couches n'est pas obligatoire dans ton application.

Sinon, voilà à quoi je pensais, 2 possibilités :

# DbTable
public function fetchAll($page = 1, $rowCount = 50, &$totalItemCount)
{
    // $select = ....;  // avec limit
    // $countSelect = ....; // un COUNT avec mêmes jointures, mêmes conditions sans limit
    
    $rows = $this->fetchAll($select);
    $totalItemCount = $this->fetchRow($countSelect)->num;

    return $rows;
}

2ème solution, tu peux utiliser Zend_Paginator pour t'aider à générer la requête count :

# DbTable
public function fetchAll($page = 1, $rowCount = 50, &$totalItemCount)
{
    // $select = ....;  // sans limit

    $paginator = Zend_Paginator::factory($select);
    $paginator->setItemCountPerPage($rowCount);
              ->setCurrentPageNumber($page);

    $rows = $paginator->getCurrentItems();
    $totalItemCount = $paginator->getTotalItemCount();

    return $rows;
}

Tu invoques la méthode comme ceci :

$page = $this->_getParam('page', 1);

$totalItemCount = 0;
$emails = $mailCollection->fetchAll($page, 50, $totalItemCount);

$paginator = Zend_Paginator::factory($totalItemCount);
$paginator->setItemCountPerPage(50);
          ->setCurrentPageNumber($page);

$this->view->emails = $emails;
$this->view->paginator = $paginator;

Attention, certains seraient prêts à me sauter à la gorge là :p

La pagination est un élément difficile à représenter dans la séparation des responsabilités de chaque couche. Il y a les requêtes d'un côté dans la couche modèle et l'affichage de l'autre dans la couche présentation (vue). Et dans les deux tu as finalement besoin du paginateur. Dans la première pour t'aider à construire les requêtes automatiquement, dans l'autre pour t'aider à générer la numérotation des pages


A+ benjamin.