Running the CCK beta tests, I get this notice several times:

Unexpected PHP error [Undefined index: created] severity [E_NOTICE] in [/sites/all/modules/simpletest/drupal_test_case.php line 57]

This code there looks wrong, since it's looking for a key in $defaults that is never created. I think the attached patch is a reasonable fix.

CommentFileSizeAuthor
#2 exception-line57-270053-2.patch1.95 KBpwolanin
exception-line57.patch840 bytespwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cwgordon7’s picture

cwgordon7 CreditAttribution: cwgordon7 commented

This looks like it alters the original code's intent: we're supposed to use $defaults['created'], if it exists, rather than $settings, which will be merged later, right?

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
FileSize
exception-line57-270053-2.patch1.95 KB

Since $defaults['created'] can never be set, I don't think that is correct.

Also looks like the code in the other base class needs to be fixed to match

boombatower’s picture

boombatower CreditAttribution: boombatower commented
Status: Needs review » Fixed

Committed.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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