Jumping to /admin/build/trigger with no actions defined

jefkin - April 25, 2008 - 18:59
Project:Actions
Version:5.x-2.3
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:duplicate
Description

You get a php error on php5 with 5.x-2.3

line 769: Illegal argument to foreach

simple fix, wrap foreach in an if:

if (is_array($ops))
{
foreach ($ops as $op => $description) {
$forms['actions_'. $hook .'_'. $op .'_assign_form'] = array ( 'callback' => 'actions_assign_form' );
}
}

#1

jvandyk - April 25, 2008 - 19:11
Status:needs review» duplicate

http://drupal.org/node/236908

#2

jefkin - April 21, 2009 - 20:44
Project:Actions» Devel
Version:5.x-2.3» 6.x-1.16
Component:Code» devel_themer
Status:duplicate» active

This is probably already reported, but I'd didn't spot it.

Conflicts exist between devel_themer in the Devel module(which appears to includes a few versioned jquery ui files) and the Jquery_ui module, which allows a site to use specific (often newer) versions of the jquery_ui.

I propose that devel_themer ads some small logic on the order of:

<?php
if (module_exists('jquery_ui'))
{
 
jquery_ui_add('ui.core');
 
jquery_ui_add('ui.mouse');
 
jquery_ui_add('ui.draggable');
}
else
{
 
drual_add_js('...'); // what currently is: ui.mouse.js, and ui.draggable.js
}
?>

My option has been to expect my jquery ui stuff to fail whil I have the themer turned on.

Thanks for your consideration.

#3

jefkin - April 21, 2009 - 20:47
Project:Devel» Actions
Version:6.x-1.16» 5.x-2.3
Component:devel_themer» Code
Status:active» duplicate

Sorry, ignore above this, I pasted in the wrong box

 
 

Drupal is a registered trademark of Dries Buytaert.