PHP Notice: Trying to get property of non-object

kenorb - January 17, 2009 - 21:26
Project:Browscap
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

This can be improved:

// $Id: browscap.module,v 1.6.2.3.2.4 2008/08/13 02:49:43 robloach Exp $
PHP Notice:  Trying to get property of non-object in /home/sites/co.uk/public_html/sites/all/modules/contributions/browscap/browscap.module on line 257
PHP Notice:  Trying to get property of non-object in /home/sites/co.uk/public_html/sites/all/modules/contributions/browscap/browscap.module on line 259
PHP Notice:  Undefined index:  parent in /home/sites/co.uk/public_html/sites/all/modules/contributions/browscap/browscap.module on line 90
PHP Notice:  Undefined index:  crawler in /home/sites/co.uk/public_html/sites/all/modules/contributions/browscap/browscap.module on line 95

#2

VeryMisunderstood - May 18, 2009 - 05:38
Category:feature request» task

#3

Rob Loach - June 8, 2009 - 11:48
Status:active» needs review
AttachmentSize
blah.patch 2.65 KB

#5

VeryMisunderstood - June 8, 2009 - 17:58
Title:Trying to get property of non-object» PHP Notice: Trying to get property of non-object
Category:task» bug report

#6

greggles - September 16, 2009 - 22:48

Here's the same patch merged with some other php notices I found.

However, I was never able to reproduce the original issue here. What are the steps to repeat?

AttachmentSize
360347_6_browscap_notices.patch 3.77 KB

#7

greggles - September 17, 2009 - 15:05
Status:needs review» fixed

I looked at the code some more and it seems reasonable - http://drupal.org/cvs?commit=264084

(Also, I didn't want to have to merge a bunch of stuff as I work on other patches ;) )

#8

System Message - October 1, 2009 - 15:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.