Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

Fix Mapping Service Settings handling#56

Open
GGoetzelmann wants to merge 1 commit intodevelopmentfrom
fix_mapping-service-ui_settingsHandling
Open

Fix Mapping Service Settings handling#56
GGoetzelmann wants to merge 1 commit intodevelopmentfrom
fix_mapping-service-ui_settingsHandling

Conversation

@GGoetzelmann
Copy link

This PR addresses the following bugs:

  • setting showServiceURL to true is overwritten by faulty and obsolete applyConfig call (where serviceURL is always undefined).
  • setting the ajaxBaseURL via settings file is not reflected in the service URL input field. The input field always shows the hard-coded base-url "http://localhost:8095/" instead.

This PR contains the following improvements:

  • URL setting is now always done via setURL() function instead of directly via DOM element. Not sure if this is the most elegant solution, feel free to change.

This PR does NOT address the following bugs:

  • Reload of the service input URL only results in changes of the plugins if the backend call is successful. Otherwise the plugin list stays the same but the rest of the page stops working properly. I assume this may be an issue in the com_mapping-service-input module rather than the frontend.

unified URL handling
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant