Closed (fixed)
Project:
Background Images
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
18 May 2012 at 14:22 UTC
Updated:
21 Jul 2012 at 04:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
drclaw commentedHmm... I'll take a look at this... looks like a spelling error =/
Thanks for the report!
Comment #2
drclaw commentedI can't reproduce the error. The spelling error maybe existed in an older version of bg_image... try downloading the latest recommended release and see if you're still getting the error.
Thanks!
drclaw
PS: PHP should be no problem if you're coming from some flavour of C!
Comment #3
jspaens commentedOK. I'll give it a try, but I downloaded very recently so I guess it WAS the most recent version. For the moment the issue has no priority, struggling to get upto speed with the (rather challenging) new environment.
Keep IT up!
Comment #4
bobup commentedI also am getting an error very similar to the summary above, with a slight spelling change. Here is what I'm getting:
Notice: Undefined variable: node_options in bg_image_node_options() (line 397 of /Users/bobup/Desktop/drupal-7.14/sites/all/modules/bg_image/bg_image.module).
(--> note that the undefined variable is "node_options", not "note_options" as stated in the summary above.)
See attached image.
I'm running D7 and Background Image 7.x-1.0-beta3+1-dev. When clicking on the "Background Images" tab on the Content page I get the above error, and even though I set up a background image node, the selector on this page is empty.
For context, line 397 of the module says:
if (!$node_options) {
I have the following modules installed and enabled: Background Image, Background Image Context, Background Image UI, Context and Context UI.
Is there something else I should install/enable? Thanks for your help!
Comment #5
drclaw commentedHi, Have you created any actual background image nodes?
Regardless, it looks like an error that still needs to be fixed. I'll take a look and report back.
Comment #6
bobup commented>>> Hi, Have you created any actual background image nodes?
Yes. Here is what I did:
Created content type "bgimage". Created a field for that type named "bgimage_field", type "image".
In Configuration >> Background Image I set the Node Type to "bgimage" and the Node Field to "bgimage_field".
Then on the Content page I click on the "Background Images" tab and get the error.
Comment #7
drclaw commentedThat all sounds correct as far as the configuration is concerned, but have you actually created any bgimage content?
Comment #8
bobup commentedAhhh - that was the clue I was missing. I knew it was something stupid on my part. Thanks!
Comment #9
drclaw commentedNo worries. The error message still needs to be fixed anyway. Maybe I'll print a message that suggests adding at least one node before you can assign a background image.
Comment #10
drclaw commentedI've fixed the error message and committed. There should be a new release candidate release out sometime in the near future!