I was getting a lot of undefined indexes in beta8 and decided to try -dev. That reduced the undefined indexes to this list.

Notice: Undefined index: file in /path/to/stuff/html/sites/all/modules/composite/composite.module on line 116
Notice: Undefined index: module in /path/to/stuff/html/sites/all/modules/composite/modules/fields/composite_fields.module on line 24
Notice: Undefined index: module in /path/to/stuff/html/sites/all/modules/composite/modules/fields/composite_fields.module on line 24
Notice: Undefined index: file in /path/to/stuff/html/sites/all/modules/composite/composite.module on line 116
Notice: Trying to get property of non-object in /path/to/stuff/html/sites/all/modules/composite/modules/fields/composite_fields.module on line 157
Notice: Undefined index: file in /path/to/stuff/html/sites/all/modules/composite/composite.module on line 116
Notice: Undefined index: task access in /path/to/stuff/html/sites/all/modules/composite/composite.module on line 274
Notice: Undefined index: task access in /path/to/stuff/html/sites/all/modules/composite/composite.module on line 274

konordo took a look at your code and I think has solved the problem. This is his proposed patch:
Replace

// Includes the 'file' parameter of a reference type, if defined. 
function composite_include_file($type) {
if ($type) {
  if ($type['file'] && is_file($type['file'])) {
    include_once $type['file'];
  }
}
}

with

// Includes the 'file' parameter of a reference type, if defined. 
function composite_include_file($type) {
  if ($type['file']) { 
   if (is_file($type['file'])) {
     include_once $type['file'];
    }
  }
}

Comments

idcm’s picture

Wait, wrong patch. Error persists. Sorry

idcm’s picture

here is an update from konordo
For this issue:

Notice: Undefined index: file in /sites/all/modules/composite/composite.module on line 116

Replace this:

// Includes the 'file' parameter of a reference type, if defined.
function composite_include_file($type) {
if ($type) {
  if ($type['file'] && is_file($type['file'])) {
    include_once $type['file'];
  }
}
}

with this

// Includes the 'file' parameter of a reference type, if defined. 
function composite_include_file($type) {
  if (isset($type['file']) && is_file($type['file'])) {
    include_once $type['file'];
  }
}
idcm’s picture

konordo share this with me for this notice

Notice: Undefined index: module in /home/idcmi/drupallayout.idcminnovations.com/html/sites/all/modules/composite/modules/fields/composite_fields.module on line 24

change this

 if (!$data[$k]['module']) {
     $data[$k]['module'] = $module;
   }

to this

 if (isset($data[$k]['module'])) {
   if (!$data[$k]['module']) {
     $data[$k]['module'] = $module;
   }
        }
bengtan’s picture

Status: Active » Fixed

Hi,

Actually, these aren't 'bugs' per se. These are just 'notice' messages.

From what I remember of the Drupal 6 requirements for PHP, you are supposed to turn off PHP 'notice' messages and ignore them.

However, I've made a couple of commits to try to fix this anyway.

See http://drupal.org/cvs?commit=445320.

If you encounter any more of these, please let me know.

Thank you.

Status: Fixed » Closed (fixed)

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

greggles’s picture