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

[dev.icinga.com #8363] Updating dashlets corrupts their URLs #1368

Closed
icinga-migration opened this issue Feb 5, 2015 · 4 comments
Closed
Labels
area/dashboard Affects the dashboard bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/8363

Created by greatexpectations on 2015-02-05 08:26:13 +00:00

Assignee: elippmann
Status: Resolved (closed on 2015-04-07 11:10:04 +00:00)
Target Version: 2.0.0-rc1
Last Update: 2015-04-07 11:10:04 +00:00 (in Redmine)


The "Edit Settings" dialog in the dashboard view corrupts the dashboard URL by inserting superfluous HTML-entities like & and the like.

  1. In the dashboard view, select the "Edit Settings" item from the "+" menu
  2. Select a dashboard entry
  3. Change the name
  4. The operation completes, but an invalid URL has been generated for the dashboard view, for example:
    "monitoring/list/services?service_state=0&limit=10&sort=service_last_state_change&dir=desc"

Changesets

2015-04-07 11:05:12 +00:00 by elippmann cdd5da8

Use Url::getRelativeUrl() when updating dashlets

fixes #8363
@icinga-migration
Copy link
Author

Updated by astrandb on 2015-02-19 20:30:11 +00:00

The url is changed to:
"monitoring/list/services?service_state=0&limit=10&sort=service_last_state_change&amp:dir=desc"
instead of the original
"monitoring/list/services?service_state=0&limit=10&sort=service_last_state_change&dir=desc"

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-31 14:36:12 +00:00

  • Subject changed from Dashboard "edit settings" corrupts URLs to Updating dashlets corrupts their URLs
  • Target Version set to 2.0.0-rc1

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-04-07 11:05:39 +00:00

  • Status changed from New to Assigned
  • Assigned to set to elippmann

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-04-07 11:10:04 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset cdd5da8.

@icinga-migration icinga-migration added bug Something isn't working area/dashboard Affects the dashboard labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.0.0-rc1 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dashboard Affects the dashboard bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant