Closed (outdated)
Project:
Location
Version:
6.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
12 Apr 2010 at 15:51 UTC
Updated:
19 Jul 2016 at 12:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
yesct commentedHutch, the patch is missing...
Comment #2
hutch commentedOoops, here comes:
Comment #3
yesct commentedthanks for all the location work!
Comment #4
yesct commentedis this related to #772146: Node location module outputs a wrong closed DIV breaking page layout
?
Comment #5
hutch commentedSame bug I think, I couldn't find it ;-(
Comment #6
yesct commentedHutch, could not find what? Oh, you mean the other issue when you were searching for your issue? :) Yeah, the location module issue queue is hard to dig through :) One of the things I'm trying to help with. :)
Comment #7
eporama commentedI applied this patch against 6.x-3.x-dev and tested with multiple locations. It does indeed fix the improper nesting of locations. And removes the dangling div if you don't have any locations at all.
Comment #8
Melissamcewen commentedI tested this on Justfood.org and it's working for me.
Comment #9
bdragon commentedThanks, committed. (I hope, had to do a bunch of stuff manually because DRUPAL-5--3 was a bit out of sync)
http://drupal.org/cvs?commit=356848
http://drupal.org/cvs?commit=356850
http://drupal.org/cvs?commit=356906
Comment #11
grasmash commentedCommitted to what version? I'm using 6.x-3.x-dev and still experiencing this bug.
Exact same symptoms as in:
http://drupal.org/node/772146
*UPDATE*
This problems seems only to occur on nodes where at least 1 location is permitted, but no location information has been entered.
Comment #12
yesct commentedmadmatter23, thanks for following through on this.
Please confirm the date of the recent dev version you are using, and provide a step by step, starting with a blank test site, with how to reproduce this bug. That will really help others be able to test this out and help find a solution. Thanks.
Comment #13
podaroknot critical
Comment #14
legolasboClosing old D6 issues as D6 is end of life