Started running a debugger for hostmaster, found a few missing 'isset's so I patched it.
diff --git a/modules/hosting/hosting.features.inc b/modules/hosting/hosting.features.inc
index 112e951..7fc7716 100644
--- a/modules/hosting/hosting.features.inc
+++ b/modules/hosting/hosting.features.inc
@@ -34,7 +34,7 @@ function hosting_feature($feature) {
if (!sizeof($features)) {
$features = hosting_get_features();
}
- if ($features[$feature]['module']) {
+ if (isset($features[$feature]['module'])) {
$return = (module_exists($features[$feature]['module'])) ? HOSTING_FEATURE_ENABLED : HOSTING_FEATURE_DISABLED;
}
else {
diff --git a/modules/hosting/task/hosting_task.module b/modules/hosting/task/hosting_task.module
index 3772b81..5c7f695 100644
--- a/modules/hosting/task/hosting_task.module
+++ b/modules/hosting/task/hosting_task.module
@@ -1143,10 +1143,10 @@ function hosting_task_fetch_tasks($rid) {
$task = array_merge($return[$type], $hook_task);
$allowed = (isset($task['exists']) && !in_array($task['task_status'], array(HOSTING_TASK_QUEUED, HOSTING_TASK_PROCESSING))) ||
- if ($allowed && !$task['hidden'] && $access_callback($node, $type)) {
+ if ($allowed && !isset($task['hidden']) && $access_callback($node, $type)) {
$task['task_permitted'] = TRUE;
}
- $task['class'] = hosting_task_status_class($task['task_status']);
+ $task['class'] = hosting_task_status_class(isset($task['task_status'])? $task['task_status']: null);
$return[$type] = $task;
}
Comments
Comment #1
jon pughattached
Comment #2
decipheredComment #3
jon pughFound many more.
Comment #4
decipheredDoesn't meet Drupal coding standards
Comment #5
steven jones commented@Jon Pugh Thanks very much for the patch, applied and cleanup up in 6.x-2.x.
Comment #6
steven jones commentedActually this required some extra work in this patch
Comment #7.0
(not verified) commentedUpdating patch code