Fluid interface

Permalink
It appears that none of these PageList filter or sort methods have a return value. It would be very convenient if they returned $this so that you could chain methods together. For example:

$pageList = new PageList();
$pages = $pageList->filterByIsApproved($cvIsApproved)
         ->filterByUserID($uID)
         ->sortByPublicDate()
         ->get();

Tags:

, page list, page search, page listing, collection filter, listing pages
View Replies:
ijessup replied on at Permalink Reply
ijessup
I believe you can just do this:
$pageList = new PageList();
$pageList->filterByIsApproved($cvIsApproved);
$pageList->filterByUserID($uID);
$pageList->sortByPublicDate();
$pages = $pageList->get();
From a coding elegance point of view, I agree with you. However, it's not entirely necessary.
sparkymeister replied on at Permalink Reply
Yes, you can do it that way. Actually, you have to. I'm just saying that a fluent interface would be nice and it would not break backwards compatibility.
ijessup replied on at Permalink Reply
ijessup
Again, I certainly agree.

You could always mod the core and submit a diff file to the c5 guys. I know Andrew has uploaded a number of mod's I've made to c5's Github account.