Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #3979] new filtering: rename the first condition to 'root' or similar #1108

Closed
icinga-migration opened this issue Apr 18, 2013 · 3 comments
Labels
bug

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Apr 18, 2013

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

Created by mfriedrich on 2013-04-18 08:08:59 +00:00

Assignee: (none)
Status: Rejected (closed on 2013-04-19 14:53:09 +00:00)
Target Version: (none)
Last Update: 2013-04-19 14:53:09 +00:00 (in Redmine)

Icinga Version: 1.9.0dev
Icinga Web Version: 1.9.0dev
IDO Version: 1.9.0dev
OS Version: Debian/wheezy
DB Type: MySQL
DB Version: 5.5.30
Browser Version: chromium 25

the problem is - once you drag and drop there, especially with different condition than AND (like OR) this is irritating.

see screenshot how an actual OR of 2 states is done now.

suggestion: rename to 'root' node, or similar.

icinga_web_1.9_filter_root.png

Attachments


Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 18, 2013

Updated by jmosshammer on 2013-04-18 08:29:18 +00:00

  • Status changed from Assigned to Feedback

I don't really see an issue here - the toplevel is always AND.
If I change it to root and you add another condition 'host name = "blob"' on the same level than your 'OR' group, it's

hostname = "blob" AND (status = 0 OR status = 1)

As the first level is always an AND conjunction, I don't really see any reason why to name it different.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 18, 2013

Updated by mfriedrich on 2013-04-18 12:21:02 +00:00

  • Target Version deleted 1.9

Mhhh hard to explain. I'll show it to you the other day.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 19, 2013

Updated by mfriedrich on 2013-04-19 14:53:09 +00:00

  • Status changed from Feedback to Rejected
  • Assigned to deleted jmosshammer
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.