reduce notices
greggles - September 4, 2009 - 08:10
| Project: | Geo |
| Version: | 6.x-1.x-dev |
| Component: | Geo API |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
When I enabled the module I got a bunch of php notices related to some missing single quotes. If you look at core like comment.module you'll see a slightly different notation that matches what I do in the patch.
| Attachment | Size |
|---|---|
| geo_notices.patch | 2.14 KB |

#1
I found some more when adding a field:
Looking at line 24, a variable is assigned and never used. This next patch removes that.
I then tried creating some fields and creating nodes using those fields.
When creating nodes I got the error:
Which seems to be a typo. Also fixed.
For some of these (especially the changes to the geo_field_widget_info and geo_set_value) I'm not sure it's the right decision, but it does get rid of notices without introducing any side effects that I could see in my quick testing.
#2
Applied - thanks!
#3
Automatically closed -- issue fixed for 2 weeks with no activity.