diff --git a/test/widgets/ArchiveDropdownWidgetTest.php b/test/widgets/ArchiveDropdownWidgetTest.php index 2eb1eaa..6a4d6e4 100644 --- a/test/widgets/ArchiveDropdownWidgetTest.php +++ b/test/widgets/ArchiveDropdownWidgetTest.php @@ -58,4 +58,60 @@ class ArchiveDropdownWidgetTest extends PHPUnit_Framework_TestCase { $this->assertTrue(empty($html)); } + + function testBuildComicArchiveDropdown() { + $w = $this->getMock('ArchiveDropdownWidget', array('_new_comicpressstoryline', '_new_wp_query', 'build_dropdown')); + + $storyline = $this->getMock('ComicPressStoryline', array('read_from_options', 'build_from_restrictions')); + $storyline->expects($this->once())->method('read_from_options'); + $storyline->expects($this->once())->method('build_from_restrictions')->will($this->returnValue(array(1,2,3))); + + $w->expects($this->once())->method('_new_comicpressstoryline')->will($this->returnValue($storyline)); + + $query = $this->getMock('WP_Query', array('query', 'have_posts', 'next_post')); + $query->expects($this->once())->method('query')->with(array( + 'showposts' => -1, + 'category__in' => array(1,2,3) + )); + + wp_insert_post((object)array('ID' => 1, 'guid' => 'guid', 'post_title' => 'title')); + + $query->expects($this->at(1))->method('have_posts')->will($this->returnValue(true)); + $query->expects($this->at(2))->method('next_post')->will($this->returnValue((object)array('ID' => 1, 'guid' => 'guid', 'post_title' => 'title'))); + $query->expects($this->at(3))->method('have_posts')->will($this->returnValue(false)); + + $w->expects($this->once())->method('_new_wp_query')->will($this->returnValue($query)); + + $w->expects($this->once())->method('build_dropdown')->with(array('guid' => 'title')); + + $w->build_comic_archive_dropdown(); + } + + function providerTestUpdate() { + $w = new ArchiveDropdownWidget(); + $valid_mode = array_shift(array_keys($w->modes)); + + return array( + array(array(), array()), + array( + array('title' => 'test'), + array('title' => 'test'), + ), + array( + array('mode' => 'bad'), + array() + ), + array( + array('mode' => $valid_mode), + array('mode' => $valid_mode) + ) + ); + } + + /** + * @dataProvider providerTestUpdate + */ + function testUpdate($input, $expected_output) { + $this->assertEquals($expected_output, $this->w->update($input, array())); + } } \ No newline at end of file diff --git a/widgets/ArchiveDropdownWidget.inc b/widgets/ArchiveDropdownWidget.inc index 91057a8..da2f39a 100644 --- a/widgets/ArchiveDropdownWidget.inc +++ b/widgets/ArchiveDropdownWidget.inc @@ -3,68 +3,41 @@ Widget Name: comicpress archive dropdown Widget URI: http://comicpress.org/ Description: -Author: Philip M. Hofer (Frumph) -Version: 1.04 +Author: Philip M. Hofer (Frumph) & John Bintz +Version: 1.1 Author URI: http://frumph.net/ */ -/* -function comicpress_archive_dropdown_storyline() { - $storyline = new ComicPressStoryline(); - $storyline->create_structure(get_option('comicpress-storyline-category-order')); - $categories = array_keys($storyline->_structure); - foreach ($categories as $id) { - $post = ComicPressDBInterface::get_instance()->get_first_comic($id); - } -} -*/ - -function comicpress_archive_dropdown() { ?> -
- -
- -
- -
- 'ArchiveDropdownWidget', 'description' => __('Display a dropdown list of your archives, styled.','comicpress') ); $this->WP_Widget('archive_dropdown', __('ComicPress Archive Dropdown','comicpress'), $widget_ops); + + $this->modes = array( + 'monthly_archive' => __('Monthly archive', 'comicpress'), + 'comic_archive' => __('Comic archive', 'comicpress'), + 'storyline_order' => __('Storyline order', 'comicpress'), + ); } function _verify_nonce() { return __comicpress_verify_nonce(); } function template_redirect() { - if ($this->_verify_nonce() == 'follow-archive-dropdown') { - + if (ArchiveDropdownWidget::_verify_nonce() == 'follow-archive-dropdown') { + wp_redirect($_GET['cp']['urls']); } } + /** + * Build a dropdown geared toward quick links to posts. + */ function build_dropdown($entries) { if (is_string($entries) || is_array($entries)) { $id = 'archive-dropdown-' . md5(rand()); + $button_id = 'archive-dropdown-submit-' . md5(rand()); $nonce = wp_create_nonce('comicpress'); $action_nonce = wp_create_nonce('comicpress-follow-archive-dropdown'); @@ -88,46 +61,143 @@ class ArchiveDropdownWidget extends WP_Widget { } ?> - + build_dropdown(wp_get_archives('type=monthly&format=option&show_post_count=-1&echo=0')); + } + + function _new_comicpressstoryline() { return new ComicPressStoryline(); } + function _new_comicpressdbinterface() { return new ComicPressDBInterface(); } + function _new_wp_query() { return new WP_Query(); } + + /** + * Build the comic archive dropdown. + */ + function build_comic_archive_dropdown() { + $storyline = $this->_new_comicpressstoryline(); + $storyline->read_from_options(); + + $query = $this->_new_wp_query(); + $query->query(array( + 'showposts' => -1, + 'category__in' => $storyline->build_from_restrictions() + )); + + $results = array(); + while($query->have_posts()) { + $post = $query->next_post(); + $results[get_permalink($post)] = get_the_title($post); + } + + return $this->build_dropdown($results); + } + + /** + * Build dropdown based on storyline order. + */ + function build_storyline_order_dropdown() { + $storyline = $this->_new_comicpressstoryline(); + $storyline->read_from_options(); + + $results = array(); + if (!empty($storyline->_structure)) { + foreach ($storyline->_structure as $id => $info) { + $category = get_category($id); + if (!empty($category)) { + $results[get_category_link($id)] = str_repeat(' ', ($info['level'] - 1) * 3) . $category->name; + } + } + } + + return $this->build_dropdown($results); + } + + /** + * Render the widget. + */ function widget($args, $instance) { extract($args, EXTR_SKIP); echo $before_widget; $title = empty($instance['title']) ? '' : apply_filters('widget_title', $instance['title']); - if ( !empty( $title ) ) { echo $before_title . $title . $after_title; }; - if ($instance['showcomicposts'] == 'on') { - comicpress_archive_dropdown_comics(); - } else { - comicpress_archive_dropdown(); + if (!empty($title)) { echo $before_title . $title . $after_title; }; + + if (method_exists($this, "build_{$instance['mode']}_dropdown")) { + echo $this->{"build_{$instance['mode']}_dropdown"}(); } + echo $after_widget; } + /** + * Update widget parameters. + */ function update($new_instance, $old_instance) { - $instance = $old_instance; - $instance['title'] = strip_tags($new_instance['title']); - $instance['showcomicposts'] = $new_instance['showcomicposts']; + $instance = array(); + + foreach (array('title', 'mode') as $field) { + if (isset($new_instance[$field])) { + switch ($field) { + case 'mode': + if (isset($this->modes[$new_instance[$field]])) { + $instance[$field] = $new_instance[$field]; + } + break; + default: + $instance[$field] = strip_tags($new_instance[$field]); + break; + } + } + } + return $instance; } + /** + * Show the widget editor. + */ function form($instance) { - $instance = wp_parse_args( (array) $instance, array( 'title' => '', 'showcomicposts' => 'off' ) ); - $title = strip_tags($instance['title']); - $showcomicposts = $instance['showcomicposts']; if (empty($showcomicposts)) $showcomicposts = 'off'; - ?> -

-

  />Off

- modes)); + $instance = wp_parse_args((array)$instance, array('title' => '', 'mode' => $valid_mode)); + + foreach (array( + 'title' => __('Title:', 'comicpress'), + 'mode' => __('Show in widget:', 'comicpress') + ) as $field => $label) { ?> +

+ +
+ modes as $mode => $label) { ?> +
+ + +
+ +

+