Additional text to show above the static block image does not work

Yes, I just recently identified this myself. Out of interest, were you using gmaplocation in Drupal 6, because my quick initial testing suggested that perhaps that code was already broken in D6... though I need to test that further.

I get the same error when saving my block. My block never shows up at all.

Just tried the module and get the same warning. This is a new install no upgrade. THXS

Same error here. Clean 7 install.
Saving the block configuration generates the error, even if you don't add text to be shown above.
But at least the title and path to display at are saved. It appears to be just the value of the text to be shown above that's not saved and generates the error.

Same problem as comment #4
@attachement - text you see above the image is not from the locationmap block

Given other commitments, it is unlikely I'll be fixing this before the May-July timeframe. Patches welcome.

here is a patch that fix this bug and add block image width and height configuration on top.

Great work!
But I have a problem ...
I set the width to 900px but the image is not larger than 640px.
Does anyone know why?

xlyz, thank you for your patch. I have literally not seen this till now, due to commitments that have kept me away from d.o completely. Apologies for the delay; I hope to test and commit this in the next couple of days.

no prob. attribution appreciated.

Pass the following option to git commit to attribute authorship to this user:
--author="xlyz <xlyz@310584.no-reply.drupal.org>"

I am getting this error also.

Notice: Undefined index: locationmap_text_top in locationmap_block_save() (line 355 of /home8/kpcprodu/public_html/americanwrc/sites/all/modules/locationmap/locationmap.module).

I don't have any text in the block "Additional text to show above the image " area...
I do have a block title and I also have a "WYSIWYG CK editor" in the "Additional text"

Do I need to do anything about this Notice? The map and all seems to be functioning properly, and I don't need any additional text to show above the image for now... maybe I'll just wait for the next module release and hope it fixes the problem?

Yay I just installed my first drupal module patch ever!

You know what, I just went ahead and forced myself to just do the patch.

Problem is fixed and I can also change the size of the map in the block area.

Yay I did it I'm so proud of myself. :-D

nathancollins44: well done. next step, write a patch! :)

Watch this space...

Committed in fdd7c38.

Attribution given as requested xlyz. Always happy to do this, though I may forget. I suggest you use git format-patch so that your authorship metadata is included in your patch files and thus automatically applied. See http://drupal.org/node/1146430

And we're done.

xlyz: just a point. When introducing a new configuration option, it's important to make sure that the default value is the same as the previous hard-coded value, so that the upgrade doesn't break previous site layouts. I'll correct that in the next commit.

you use git format-patch so that your authorship metadata is included in your patch files

yep. i did take note of that already, but too late for this patch. ty btw.

Patch applied and tested here too.
Works great, thanks!

Franck Horlaville
TAM Software