Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#1017 closed defect (fixed)

Javascript error in existing entries when adding a google map field to a form

Reported by: reto.kohli Owned by: riesen
Priority: normal Milestone: Contrexx 3.0 SP5
Component: MediaDir Version: 3.0.0.1
Severity: normal Keywords:
Cc:

Description

To reproduce the problem, create a form without any google map type fields (e.g. a text field only), then add an entry for that form. Edit the form again, and add the google map type field now. Now open the entry for editing; now the google map will not show, and the location search won't work either. This is caused by the map values (lat./lon./zoom) being empty, thus causing a javascript error in the function initialize():


[...] map.setCenter(new GLatLng(, ), ); [...]


instead of, i.e.,


[...] map.setCenter(new GLatLng(0, 0), 0); [...]


in order to provide valid js syntax at the least. Better yet, the default values should be used.

Change History (7)

comment:1 Changed 7 years ago by thomas.daeppen

  • Milestone changed from unknown to Contrexx 3.0 SP3

comment:2 Changed 7 years ago by riesen

  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed 7 years ago by riesen

(In https://github.com/Cloudrexx/cloudrexx/commit/35711086175077aea94d295d02bec74f8e32b19f) mediadir (fixed #1017): set default value of 0 if variables are empty.

comment:4 Changed 7 years ago by riesen

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:5 Changed 7 years ago by riesen

  • Owner changed from comvation to riesen
  • Status changed from reopened to new

comment:6 Changed 7 years ago by riesen

  • Resolution set to fixed
  • Status changed from new to closed

comment:7 Changed 7 years ago by kramer

(In https://github.com/Cloudrexx/cloudrexx/commit/3a7724a94b50b7766a7d8bc67e0dfefbb196290c) Merged revision(s) 26853 from trunk: Mediadir (fixed #1017): the map still didn't work on the page with section mediadir and cmd map, added fix to this view too

Note: See TracTickets for help on using tickets.