On the d.o D7 dev site, when we clear the cache and reload a project issue page, we're getting a lot of PHP notice-level warnings from this module:

Notice: Undefined index: object_base_table in sampler_views_data() (line 36 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_base_table in sampler_views_data() (line 38 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_primary_key in sampler_views_data() (line 39 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_base_table in sampler_views_data() (line 36 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_base_table in sampler_views_data() (line 38 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_primary_key in sampler_views_data() (line 39 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_base_table in sampler_views_data() (line 36 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_base_table in sampler_views_data() (line 38 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).
Notice: Undefined index: object_primary_key in sampler_views_data() (line 39 of /var/www/dev/project-drupal_7.redesign.devdrupal.org/htdocs/sites/all/modules/sampler/views/sampler.views.inc).

I'll see if I can make a patch...

CommentFileSizeAuthor
#1 1706716-sampler-php-warning.patch637 bytesjhodgdon

Comments

jhodgdon’s picture

Assigned: jhodgdon » Unassigned
Status: Active » Needs review
StatusFileSize
new637 bytes

Here's a patch. Unfortunately created with bzr since that is what is on the dev sites. You should be able to apply it using patch -p4 (or manually apply since it is small).

hunmonk’s picture

Status: Needs review » Fixed

looks good, committed and pushed to 6.x-1.x and 7.x-1.x, thanks!

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