Cyclone-Team/gramps
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

0006802: In Narrative Web report most Family Maps are missing. Problem due to single apostrophe (closing single quotation mark) which needs to be escaped.

Browse Source 
svn: r23187
This commit is contained in:
Tim G L Lyons 2013-09-24 11:29:26 +00:00
parent 40085e924c
commit a8cd8958c8
1 changed files with 25 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
src/plugins/webreport/NarrativeWeb.py
View File

@ -3533,7 +3533,8 @@ class PlacePages(BasePage):
head += jsc
# Google adds Latitude/ Longitude to its maps...
jsc += google_jsc % (latitude, longitude, placetitle)
jsc += google_jsc % (latitude, longitude,
placetitle.replace("'", "\\'"))
else:
# OpenStreetMap (OSM) adds Longitude/ Latitude to its maps,
@ -5505,7 +5506,7 @@ class PersonPages(BasePage):
individualdetail += self.display_ind_associations(assocs)
# for use in family map pages...
if len(place_lat_long):
if len(place_lat_long) > 0:
if self.report.options["familymappages"]:
# save of, string_io and cur_fname before creating a new page
sof = of
@ -5640,13 +5641,17 @@ class PersonPages(BasePage):
# are we creating Drop Markers or Markers?
elif self.googleopts in ["Drop", "Markers"]:
tracelife += """
['%s', %s, %s, %d],""" % (placetitle, latitude, longitude, seq_)
['%s', %s, %s, %d],""" % (placetitle.replace("'", "\\'"), latitude, longitude, seq_)
# are we using OpenStreetMap?
else:
tracelife += """
[%s, %s],""" % (longitude, latitude)
seq_ += 1
# FIXME: The last element in the place_lat_long list is treated
# specially, and the code above is apparently repeated so as to
# avoid a comma at the end, and get the right closing. This is very
# ugly.
latitude, longitude, placetitle, handle ,date, etype = place_lat_long[-1]
# are we using Google?
@ -5662,7 +5667,7 @@ class PersonPages(BasePage):
elif self.googleopts in ["Drop", "Markers"]:
tracelife += """
['%s', %s, %s, %d]
];""" % (placetitle, latitude, longitude, seq_)
];""" % (placetitle.replace("'", "\\'"), latitude, longitude, seq_)
# are we using OpenStreetMap?
elif self.mapservice == "OpenStreetMap":
@ -5701,11 +5706,25 @@ class PersonPages(BasePage):
# are we using Google?
if self.mapservice == "Google":
jsc += google_jsc % (latitude, longitude, placetitle)
# FIXME: Horrible hack, because when there is only a
# single marker, the javascript for place is used,
# which has a 'place_canvas' division, instead of a
# 'map_canvas' division.
jsc += google_jsc.replace("place_canvas",
"map_canvas") % \
(latitude, longitude,
placetitle.replace("'", "\\'"))
# we are using OpenStreetMap?
else:
jsc += openstreetmap_jsc % (xml_lang()[3:5].lower(), longitude, latitude)
# FIXME: Horrible hack, because when there is only a
# single marker, the javascript for place is used,
# which has a 'place_canvas' division, instead of a
# 'map_canvas' division.
jsc += openstreetmap_jsc.replace("place_canvas",
"map_canvas") % \
(xml_lang()[3:5].lower(),
longitude, latitude)
# there is more than one marker...
else: