Installing a Feature that includes a CCK Computed Field can yield the following (drush output):

WD php: Notice: Undefined index: store in computed_field_field_settings() (line 107 of                                                                     [error]
/Users/me/Projects/xxx/sites/all/modules/computed_field/computed_field.module).
WD php: Notice: Undefined variable: columns in computed_field_field_settings() (line 126 of                                                                [error]
/Users/me/Projects/xxx/sites/all/modules/computed_field/computed_field.module).
CommentFileSizeAuthor
#3 undefined_vars-1128544-3.patch1.33 KBcmaltais
#1 undefined_vars-1128544-1.patch928 bytesryan_courtnage

Comments

ryan_courtnage’s picture

ryan_courtnage commented
StatusFileSize
new undefined_vars-1128544-1.patch928 bytes

Attached patch to 6.x-1.0-beta5 eliminates the errors.

git diff 6.x-1.0-beta5 > undefined_vars-1128544-1.patch

mikeytown2’s picture

mikeytown2 commented
Status: Active » Reviewed & tested by the community

patch fixes the bug

cmaltais’s picture

cmaltais commented
Version: 6.x-1.0-beta5 » 6.x-1.0
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new undefined_vars-1128544-3.patch1.33 KB

I have rerolled the patch for version 1.0.

This should probably be tested, but worked on my install of openatrium 1.x.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Due to the Drupal core life cycle policy and security support advisery, Drupal 6 is no longer supported. So issues for Drupal 6 cannot be longer maintained. The project maintainer has asked for closing all D6 issues to clean up the issue queue. Feel free to reopen the issue if required or set it to "needs to be ported" and latest D8 dev version, if the issue discusses a still missing feature which can be implemented in the D8 version.