With views 3, I notices an issue:

Missing argument 3 for views_include_handler(), called in /var/www/vhosts/123.com/httpdocs/sites/all/modules/contrib/views_periodic_execution/views_periodic_execution.report.inc on line 30 and defined in /var/www/vhosts/123.com/httpdocs/sites/all/modules/contrib/views/includes/handlers.inc on line 48.

I went into that file, and changed:
if (views_include_handler($display_plugin['handler'], 'display', 'handler') && is_subclass_of($display_plugin['handler'], $display_type)) {

into

if ($display_plugin['handler'] == 'views_build_menu_plugin_display') {

Which fixed it.

CommentFileSizeAuthor
#6 views_periodic_execution.patch2.2 KBdanny englander

Comments

steven jones’s picture

steven jones commented
Status: Active » Needs work

Needs a patch, and a views 3 version of this module.

dm66’s picture

dm66 commented

To work Views periodic execution in Views 3 need:

replace string in views_periodic_execution.module in line 87
if (views_include_handler($display_plugin, 'display') && is_subclass_of($display_plugin['handler'], $display_type)) {

if (views_include_handler($display_plugin['handler'], $display_plugin, 'display') && is_subclass_of($display_plugin['handler'], $display_type)) {

and replace string in views_periodic_execution.module in line 150
$view->execute_display();
$view->execute_display($view_data['display']);

and replace string in views_periodic_execution.report.inc in line 30
if (views_include_handler($display_plugin, 'display') && is_subclass_of($display_plugin['handler'], $display_type)) {

if (views_include_handler($display_plugin['handler'], $display_plugin, 'display') && is_subclass_of($display_plugin['handler'], $display_type)) {

Work fine in Views 3.

brycesenz’s picture

brycesenz commented

Subscribing. The comments in #3 seem to work for me, though I've only just started using this module.

udvranto’s picture

udvranto commented

subscribing

udvranto’s picture

udvranto commented

#2 works.

danny englander’s picture

danny englander
(he/him)
commented
StatusFileSize
new views_periodic_execution.patch2.2 KB

I've rolled a patch based on #2 and tested locally, it seemed to be fine. However, I have never rolled a patch that does more than one file before so I hope it's ok. (BTW, these fixes solved my issue as well and I was able to send email out via rules which was previously being stopped becuase of this error.)

steven jones’s picture

steven jones commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Assigned: Unassigned » steven jones

Views 3 support will be in the 7.x-1.x branch.

steven jones’s picture

steven jones commented
Status: Needs work » Fixed

Working with views 3 in the 7.x branch now.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

heyyo’s picture

heyyo commented

Any chance to have a views 3 version for Drupal 6.x ?