When I enable the Storm Ticket module I get this error on some pages with Views: "warning: Illegal offset type in isset or empty in /opt/prototype/includes/common.inc on line 916."

I am only experiencing this on certain Views on Panels pages, but it could be more or less widespread.

After poking around I found this is because of these sections at line 206 in stormticket.views.inc:

 $data['stormtask']['billable'] = array(
    'title' => t('Task Billable'),
    'help' => 'Storm Task Billable',
    'field' => array(
      'click sortable' => TRUE,
    ),
    'sort' => array(
      'handler' => 'views_handler_sort',
    ),
    'filter' => array(
      'handler' => 'views_handler_filter_numeric',
    ),
  );

  $data['stormtask']['billed'] = array(
    'title' => t('Tasl Billed'),
    'help' => 'Storm Task Billed',
    'field' => array(
      'click sortable' => TRUE,
    ),
    'sort' => array(
      'handler' => 'views_handler_sort',
    ),
    'filter' => array(
      'handler' => 'views_handler_filter_numeric',
    ),
  );

They should read 'stormticket' instead of 'stormtask'. Also, unrelated to the above error, there's a typo of "tasl" instead of "task".

The included patch has fixed this on my setup (Fedora 13, PHP 5.3.3, Drupal 6.19).

Comments

juliangb’s picture

StatusFileSize
new1.32 KB

Good spot.

I noticed also that the titles are wrong too - they should say ticket too. Updated patch attached.

mlove_drupal’s picture

Doh!

tchurch’s picture

You can say that again. I should say this too. I missed it when I developed it. Well spotted both.

juliangb’s picture

Status: Needs review » Fixed

Committed. This will be in 1.35.

Status: Fixed » Closed (fixed)

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