remove some old code
This commit is contained in:
parent
3e1a2de3dc
commit
060b03ec24
@ -92,48 +92,6 @@ class ComicPressComicPost {
|
|||||||
}
|
}
|
||||||
// @codeCoverageIgnoreEnd
|
// @codeCoverageIgnoreEnd
|
||||||
|
|
||||||
/**
|
|
||||||
* Change the ordering of comic images in the associated post.
|
|
||||||
* @param array $order The new requested order.
|
|
||||||
*/
|
|
||||||
function change_comic_image_ordering($requested_new_order) {
|
|
||||||
$orderings = get_post_meta($this->post->ID, 'comic_ordering', true);
|
|
||||||
if (!is_array($orderings)) { $orderings = array(); }
|
|
||||||
|
|
||||||
$new_order = array();
|
|
||||||
|
|
||||||
$requested_new_order = (array)$requested_new_order;
|
|
||||||
|
|
||||||
foreach ($orderings as $type => $current_order) {
|
|
||||||
$new_order[$type] = array();
|
|
||||||
$sort_by_position = array();
|
|
||||||
$position = 0;
|
|
||||||
foreach ($requested_new_order[$type] as $id) {
|
|
||||||
if (!isset($sort_by_position[$position])) {
|
|
||||||
$sort_by_position[$position] = array();
|
|
||||||
}
|
|
||||||
$sort_by_position[$position][] = $id;
|
|
||||||
$position++;
|
|
||||||
}
|
|
||||||
ksort($sort_by_position);
|
|
||||||
$requested_order = array();
|
|
||||||
foreach ($sort_by_position as $position => $ids) {
|
|
||||||
sort($ids);
|
|
||||||
$requested_order = array_merge($requested_order, $ids);
|
|
||||||
}
|
|
||||||
|
|
||||||
$requested_order = array_merge($requested_order, array_diff($current_order, $requested_order));
|
|
||||||
|
|
||||||
foreach ($requested_order as $requested_comic) {
|
|
||||||
if (in_array($requested_comic, $current_order)) {
|
|
||||||
$new_order[$type][] = $requested_comic;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
update_post_meta($this->post->ID, 'comic_ordering', $new_order);
|
|
||||||
}
|
|
||||||
|
|
||||||
function find_parents() {
|
function find_parents() {
|
||||||
$parents = array();
|
$parents = array();
|
||||||
$post_categories = wp_get_post_categories($this->post->ID);
|
$post_categories = wp_get_post_categories($this->post->ID);
|
||||||
|
@ -70,45 +70,6 @@ class ComicPressComicPostTest extends PHPUnit_Framework_TestCase {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
function providerTestChangeComicImageOrdering() {
|
|
||||||
return array(
|
|
||||||
array(
|
|
||||||
array('comic' => array(1,2,3)),
|
|
||||||
array(
|
|
||||||
'comic' => array(2,3,1)
|
|
||||||
),
|
|
||||||
array('comic' => array(2,3,1))
|
|
||||||
),
|
|
||||||
array(
|
|
||||||
array('comic' => array(1,2,3)),
|
|
||||||
array(
|
|
||||||
'comic' => array(3,1,2)
|
|
||||||
),
|
|
||||||
array('comic' => array(3,1,2))
|
|
||||||
),
|
|
||||||
array(
|
|
||||||
array('comic' => array(1,2,3)),
|
|
||||||
array(
|
|
||||||
'comic' => array(1,2)
|
|
||||||
),
|
|
||||||
array('comic' => array(1,2,3))
|
|
||||||
),
|
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @dataProvider providerTestChangeComicImageOrdering
|
|
||||||
* @covers ComicPressComicPost::change_comic_image_ordering
|
|
||||||
*/
|
|
||||||
function testChangeComicImageOrdering($current_ordering, $revised_ordering, $expected_result) {
|
|
||||||
update_post_meta(1, 'comic_ordering', $current_ordering);
|
|
||||||
|
|
||||||
$this->p->post = (object)array('ID' => 1);
|
|
||||||
$this->p->change_comic_image_ordering($revised_ordering);
|
|
||||||
|
|
||||||
$this->assertEquals($expected_result, get_post_meta(1, 'comic_ordering', true));
|
|
||||||
}
|
|
||||||
|
|
||||||
function providerTestUpdatePostMediaData() {
|
function providerTestUpdatePostMediaData() {
|
||||||
return array(
|
return array(
|
||||||
array(
|
array(
|
||||||
|
Loading…
Reference in New Issue
Block a user