Comments

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new469 bytes

Attached patch fixes the PHP notice.

cvining’s picture

Yup. I can confirm this patch clears up notices I've been getting in my dblog.

Thx!

-- Cronin

BeaPower’s picture

worked for me too

heatherwoz’s picture

Status: Needs review » Reviewed & tested by the community

Fixed for me too.

nehajn’s picture

Its working for me too..
Thanks

richardma’s picture

I am new in drupal and Linux , can anyone help ?

I have the same problem. But, I cannot use the patch "1257690-googlecse-php-notice.patch". I got the error message on the Linux server .

"
patching file google_cse.module
Hunk #1 falled at 174.
1 out of 1 hunk failed -- saving rejects to file google_cse.module.rej"

Thanks

VirtualNorman’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
StatusFileSize
new629 bytes

I guess this patch hasn't made it into the latest 7.x-2.x-dev yet. Same correction, now applied to line 222 instead of 174. I re-rolled the patch and attached.

VirtualNorman’s picture

StatusFileSize
new585 bytes

Sorry #7 is probably not applying, as my git root is one directory higher than the drupal code root, so there is an extra directory level in my #7 patch that you probably don't have.
Here is the same patch with the paths adjusted. Hope this works.

heatherwoz’s picture

Status: Reviewed & tested by the community » Needs work

Needs a new patch, it now applies to line 247.

flaviovs’s picture

Status: Needs work » Reviewed & tested by the community
StatusFileSize
new469 bytes

@heatherwoz, patch in #8 still apply to 7.x-2.x (patch/git will take care of the small line offset). But in case you insist (<grin>), here a re-rolled patch against current 7.x-2.x.

heatherwoz’s picture

Sorry, I didn't realize git would take care of the offset. That's good to know.

malcomio’s picture

Status: Reviewed & tested by the community » Needs review

Setting to needs review so the bot can test the patch.

  • malcomio committed 1f21ef3 on 7.x-2.x
    Issue #1257690 by VirtualNorman, Dave Reid, flaviovs: Notice: Undefined...
malcomio’s picture

Status: Needs review » Fixed

Applied patch to 7.x-2.x-dev

Status: Fixed » Closed (fixed)

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