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

Fix bug #2114 and fix Open Street Map sp in i18n message #131

Closed
wants to merge 6 commits into from
Closed

Fix bug #2114 and fix Open Street Map sp in i18n message #131

wants to merge 6 commits into from

Conversation

filbertkm
Copy link
Contributor

These changes:

  1. fix the spelling of Open Street Maps -> OpenStreetMap in the map_provider_description i18n (en) message that appears on the advanced map installer page.
  2. fix bug #2114 -- on the admin/settings map tab, the country dropdown list overflows the right column on smaller screens. Changed substr param from 35 to 30 which fixes this.

@filbertkm
Copy link
Contributor Author

  1. fixed a bug with the timezones. my timezone was somehow set to 12 -- a number and not a string. date_default_timezone_set requires a string, so I got an error that the timezone was invalid. Adding this check, fixed this and allowed me to go back to the settings page and set the timezone. There might be a better way, but this worked.

@rjmackay
Copy link
Contributor

Hi Katie,
Sorry this has been ignored for so long. Your patch is now out of date and I suspect fixed already. If you think there's still an issue, could you update your code and resubmit? Thanks
Robbie

@rjmackay rjmackay closed this Feb 13, 2012
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

Successfully merging this pull request may close these issues.

None yet

2 participants