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

Inventory and evaluate merge of refactoring done for haiti instance #1

Open
springmeyer opened this issue Nov 22, 2010 · 0 comments
Open

Comments

@springmeyer
Copy link
Member

At http://bitbucket.org/springmeyer/ogcserver the ogcserver code was refactored in a hurry and unfortunately many changes, although improvements to previously broken behavior, are brittle and need cleaning up and review.

All these changes were merged in 25f4111

They include:

  1. Extended the wsgi.py to support reading directly from an xml (skipping config), as well as other subtle options (these need ported to mod_python and cgi/fcgi impl) like default HTML response.

  2. Better GetCapabilities support for various clients, even broken ones (we need more tests for these)

  3. Switched to using WMS 1.1.1 as default if no version was requested (this avoided breaking GetCaps for old clients that never report version because they only support 1.1.x)

  4. Added more thorough unicode support to GetCaps

  5. loosened strictness of parameters such as format, transparent, and exceptions in terms of case and existence

  6. added preliminary support for Mapnik2 if installed, otherwise we default to mapnik1

  7. added strict dependency on lxml, because otherwise GetCaps will not work

  8. added png8 (aka png245) request type, needs cleanup

  9. switched ordering of layers to be correct to the WMS standard in terms of rendering order (this breaks many clients that do not request the right order in GetCaps and needs more work/testing) - it was originally done to support QGIS 1.5

  10. added draft support for "meta layers" where groups of layers are constructed (and can be requested as one) if the styles rules are named (this was a quick hack and needs to be either removed or abstracted into a config file)

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

1 participant