Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Largo_Related tests and bugfixes #842

Merged
merged 16 commits into from
Aug 17, 2015
Merged

Largo_Related tests and bugfixes #842

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
088a505
Notes for #782 :(
benlk Aug 7, 2015
00e0f12
Nope
benlk Aug 7, 2015
9a06c65
Working tests for __construct
benlk Aug 10, 2015
2b12139
Test for LR's Popularity Sort function, using objects instead of actu…
benlk Aug 10, 2015
11384ff
Partially functional tests for series custom ordering and featured as…
benlk Aug 11, 2015
46a4c76
Well at least I know *how* it's going wrong.
benlk Aug 11, 2015
efef9ce
Some other stuff is broken, but this is a fix for posts not being ret…
benlk Aug 14, 2015
f6b9b89
Make custom order output the first post first
benlk Aug 14, 2015
7e402e3
remove debugging statement
benlk Aug 14, 2015
036bd16
Notes on series_custom and featured orderby parameters, variable fix
benlk Aug 14, 2015
a456811
Fixing dates. Tests now working for asc, desc, series_custom, but not…
benlk Aug 14, 2015
5bb0a76
Giving up on test_series_featured_* because they're returning in reve…
benlk Aug 15, 2015
cdc4cb8
Working category test
benlk Aug 17, 2015
612faa4
Notes for future test-writers
benlk Aug 17, 2015
24edd00
Gotta initialize Largo_Related with a count and a post
benlk Aug 17, 2015
1037d67
Cleaning up the comments
benlk Aug 17, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions inc/related-content.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -460,6 +460,7 @@ protected function cleanup_ids() {
* Fetches posts contained within the series(es) this post resides in. Feeds them into $this->post_ids array
*
* @access protected
* @see largo_series_custom_order
*/
protected function get_series_posts() {

Expand Down Expand Up @@ -488,7 +489,9 @@ protected function get_series_posts() {
// see if there's a post that has the sort order info for this series
$cftl_query = new WP_Query( array(
'post_type' => 'cftl-tax-landing',
'series' => $term->slug,
'tax_query' => array (
'series' => $term->slug,
),
'posts_per_page' => 1
));

Expand All @@ -500,12 +503,13 @@ protected function get_series_posts() {
case 'ASC':
$args['order'] = 'ASC';
break;
// 'series_custom' and 'featured' are custom ones, caught with largo_series_custom_order in inc/wp-taxonomy-landing/functions/cftl-series-order.php
case 'custom':
$args['orderby'] = 'series_custom';
break;
case 'featured, DESC':
case 'featured, ASC':
$args['orderby'] = $opt['post_order'];
$args['orderby'] = $has_order;
break;
}
}
Expand Down
Loading