Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redundant googlemap settings in wt_module_setting #42

Closed
ddrury opened this issue Nov 23, 2013 · 2 comments
Closed

Redundant googlemap settings in wt_module_setting #42

ddrury opened this issue Nov 23, 2013 · 2 comments

Comments

@ddrury
Copy link
Contributor

ddrury commented Nov 23, 2013

Hi Greg,

Have just noticed this. The following settings (I think) are now unused
GM_API_KEY
GM_DEFAULT_TOP_VALUE
GM_DISP_COUNT
GM_MAX_NOF_LEVELS
GM_PH_CONTROLS
GM_PH_WHEEL
GM_PRE_POST_MODE_1
GM_PRE_POST_MODE_2
GM_PRE_POST_MODE_3
GM_PRE_POST_MODE_4
GM_PRE_POST_MODE_5
GM_PRE_POST_MODE_6
GM_PRE_POST_MODE_7
GM_PRE_POST_MODE_8
GM_PRE_POST_MODE_9

I determined this list by disabling the googlemap module, dumping & then deleting all its settings and then re-enabled it and saved its settings. The above list were all absent.

Dave

@ddrury
Copy link
Contributor Author

ddrury commented Nov 24, 2013

most of these setting were included in db_schema_4_5.php but I don't think the version number in googlemaps/defaultconfig.php was updated

@fisharebest
Copy link
Owner

This cleanup was written during the last major GM changes - but obviously never finished.

fisharebest added a commit that referenced this issue Dec 2, 2013
Conflicts:
	modules_v3/googlemap/defaultconfig.php
reenx pushed a commit to reenx/webtrees that referenced this issue Dec 14, 2016
Conflicts:
	modules_v3/googlemap/defaultconfig.php
@ricklach ricklach mentioned this issue Jul 11, 2019
@makitso makitso mentioned this issue Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants