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 #8853] Syntax Highlighting: host.address vs host.add #2819

Closed
icinga-migration opened this issue Mar 24, 2015 · 6 comments
Closed
Labels
area/configuration DSL, parser, compiler, error handling bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

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

Created by mfriedrich on 2015-03-24 09:13:28 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2015-04-27 12:10:04 +00:00)
Target Version: 2.3.5
Last Update: 2015-04-27 12:15:27 +00:00 (in Redmine)

Icinga Version: 2.3.2
Backport?: Already backported
Include in Changelog: 1

There's a mismatch with host.address and the object method ".add" being highlighted. This was introduced with array methods in #8660

Auswahl_119.png

Attachments

Changesets

2015-04-27 11:03:06 +00:00 by (unknown) 5d92e8f

Fix syntax highlighting: host.address vs host.add

Fix syntax highlighting: host.address vs host.add

refs #8853

2015-04-27 12:07:29 +00:00 by (unknown) 0f0d60c

Fix syntax highlighting: host.address vs host.add

fixes #8853

2015-04-27 12:15:40 +00:00 by (unknown) 51ab5c6

Fix syntax highlighting: host.address vs host.add

fixes #8853

Relations:

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-03-24 09:13:52 +00:00

  • Relates set to 8660

@icinga-migration
Copy link
Author

Updated by gbeutner on 2015-04-27 06:12:10 +00:00

@ccesario: Can you please update this issue?

@icinga-migration
Copy link
Author

Updated by ccesario on 2015-04-27 11:14:37 +00:00

gunnarbeutner wrote:

@ccesario: Can you please update this issue?

Hello @gunnarbeutner,

Branch fix/syntax-highlighting-host-add-8853 - https://git.icinga.org/?p=icinga2.git;a=tree;h=refs/heads/fix/syntax-highlighting-host-add-8853;hb=refs/heads/fix/syntax-highlighting-host-add-8853;js=1 contain the proposed solution for this.

Could you please check it!?

Best regards

Carlos

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-04-27 12:05:39 +00:00

  • Assigned to changed from ccesario to mfriedrich
  • Target Version set to 2.4.0

Thanks, I'll test that :)

@icinga-migration
Copy link
Author

Updated by Anonymous on 2015-04-27 12:10:04 +00:00

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

Applied in changeset 0f0d60c.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2015-04-27 12:15:27 +00:00

  • Target Version changed from 2.4.0 to 2.3.5
  • Backport? changed from TBD to Yes

@icinga-migration icinga-migration added bug Something isn't working area/configuration DSL, parser, compiler, error handling labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.3.5 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/configuration DSL, parser, compiler, error handling bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant