PageRepository find by category

[matudelatower]

When find pages by category.

must be add ->andWhere('page.enabled = true') like:

public function findByCategory(Category $category, $order, $orderBy, $page, $limit): Paginator
    {
        $qb = $this->createQueryBuilder('page')
            ->where('page.category = :category')
            ->andWhere('page.enabled = true')
            ->orderBy('page.'.$orderBy, $order)
            ->setMaxResults($limit)
            ->setFirstResult($limit * ($page-1))
            ->setParameter('category', $category)
        ;

        return new Paginator($qb->getQuery()->useResultCache($this->cacheEnabled, $this->cacheTtl));
    }

[matudelatower]

And $orderBy param comes with createdAt value from CategoryController->indexAction() and throws:

in current($validOrderFields) gives createdAt if i use id instead works fine, or remove ->orderBy('page.'.$orderBy, $order) from de function above

[Pierstoval]

Thanks for the report, I'll investigate on this 👍

[Gyvastis]

Any fix for this yet? I'm using this hack for now.

[matudelatower]

@Gyvastis im solve overriding the repository

namespace WebBundle\Repository;

use Doctrine\ORM\Tools\Pagination\Paginator;
use Orbitale\Bundle\CmsBundle\Repository\PageRepository;
use Orbitale\Bundle\CmsBundle\Entity\Category;
use Orbitale\Bundle\CmsBundle\Entity\Page;

/**
 * PaginaRepository
 *
 * This class was generated by the Doctrine ORM. Add your own custom
 * repository methods below.
 */
class PaginaRepository extends PageRepository {

	/**
	 * @param Category $category
	 * @param string $order
	 * @param string $orderBy
	 * @param int $page
	 * @param int $limit
	 *
	 * @return Paginator
	 */
	public function findByCategory( Category $category, $order, $orderBy, $page, $limit ): Paginator {
		$qb = $this->createQueryBuilder( 'page' )
		           ->where( 'page.category = :category' )
		           ->andWhere( 'page.enabled = true' )
//		           ->orderBy( 'page.' . $orderBy, $order )
                   ->setMaxResults( $limit )
		           ->setFirstResult( $limit * ( $page - 1 ) )
		           ->setParameter( 'category', $category );

		return new Paginator( $qb->getQuery()->useResultCache( $this->cacheEnabled, $this->cacheTtl ) );
	}

and changing the repository mapping in entity

 * @ORM\Entity(repositoryClass="WebBundle\Repository\PaginaRepository")
 */
class Pagina extends BasePage implements AmpInterface {

[Gyvastis]

@matudelatower but you're messing up the order of page items inside a category page, no?

[matudelatower]

@Gyvastis yes, but they always came in order by id by default