line 1840 of .../sites/all/modules/patched/views/includes/view.inc

This is triggered by views_schema_6013 / views_update_6013 - which specify a field ('core') of type 'int' with no size.

Patch included - checks $info['size'] is set before reading it.

CommentFileSizeAuthor
#5 views-1432242-5-fix-size-notice.patch721 bytesmikeytown2
views-php5.3-size.patch683 bytesserenecloud
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Update status, not that the patch got lost.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review

Fail.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Reviewed & tested by the community

Works for me

Status: Reviewed & tested by the community » Needs work

The last submitted patch, views-php5.3-size.patch, failed testing.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Version: 6.x-2.16 » 6.x-2.x-dev
Status: Needs work » Needs review
FileSize
views-1432242-5-fix-size-notice.patch721 bytes

Patch against the latest dev

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Reviewed & tested by the community
Peter Bowey’s picture

Peter Bowey CreditAttribution: Peter Bowey commented

Refer #5:

Agreed, this patch is required for Views.
Patch tested OK here.
I am using the latest Views dev [6.x-2.x-dev]

BarisW’s picture

BarisW CreditAttribution: BarisW commented

Patch works fine. Thanks!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Reviewed & tested by the community » Needs work

If we commit this, there should be a patch for 6.x-3.x as well :(

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs work » Reviewed & tested by the community

6.x-3.x already has a fix committed.
http://drupalcode.org/project/views.git/blob/302f190e1427a7b5cd39c9ebb48...

laurentb’s picture

laurentb CreditAttribution: laurentb commented

5: views-1432242-5-fix-size-notice.patch queued for re-testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue summary: View changes
Parent issue: » #2507847: Plan for Views v6.x-2.27 release
izmeez’s picture

izmeez CreditAttribution: izmeez commented

Patch in comment #5 applies without difficulty to latest views-6.x-2.26, is already RTBC and has been committed to views-6.x-3.x

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented
Status: Reviewed & tested by the community » 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

izmeez’s picture

izmeez CreditAttribution: izmeez commented
Title: PHP 5.3 - Notice: Undefined index: size in views_db_object->export_row() » [views] PHP 5.3 - Notice: Undefined index: size in views_db_object->export_row()
Project: Views (for Drupal 7) » Drupal 6 Long Term Support
Version: 6.x-2.x-dev »
Status: Closed (outdated) » Fixed

Patch committed as noted in #3022037-4: [views] meta 2.x patches ready for release

Status: Fixed » Closed (fixed)

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