We are getting a PHP Notice, which has a simple fix. Patch to follow.

CommentFileSizeAuthor
#1 1418036-fix-php-warning-2.patch660 bytestravist
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

travist’s picture

Here is the patch.

travist’s picture

Status: Active » Needs review
dawehner’s picture

You know such kind of fixes are easy in theory but isn't there maybe a real bug involved here?

Couldn't validate_type be removed by some kind of other problem. So it would be somehow cool if you could explain whether there is a way to reproduce the problem.

djbobbydrake’s picture

Issue summary: View changes

This is relatively straightforward. We should check for existence of the array item before trying to evaluate it. Surprised this hasn't gotten more attention. It's breaking some of our simpletests with php notices enabled.

Chris Matthews’s picture

Status: Needs review » Closed (outdated)

The Drupal 6 branch is no longer supported, please check with the D6LTS project if you need further support. For more information as to why this issue was closed, please see issue #3030347: Plan to clean process issue queue