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

using correct variable data types instead of using generic Object type #3

Merged
merged 3 commits into from Apr 25, 2013
Merged

using correct variable data types instead of using generic Object type #3

merged 3 commits into from Apr 25, 2013

Conversation

ghost
Copy link

@ghost ghost commented Apr 24, 2013

Using correct data type for the variables instead of using generic Object type and down casting it to appropriate type.
ternary operator is used where ever possible as a replacement to the if condition to made the code look concise.

RangerRick added a commit that referenced this pull request Apr 25, 2013
using correct variable data types instead of using generic Object type
@RangerRick RangerRick merged commit 5f19787 into OpenNMS:1.12 Apr 25, 2013
@RangerRick
Copy link
Contributor

Actually, since this code is compiled to javascript to GWT, I'm not confident this patch reliably solves the problem, and will additionally be very difficult to test (since testing in Eclipse isn't enough to know it's working). Since we're trying to go to a release candidate for 1.12 right now, I've gone ahead and reverted it.

@ghost
Copy link
Author

ghost commented Apr 25, 2013

You mean to say you have reverted my pull request #5 and not this pull request ( #3 ). It's fine, we will merge again #5 pull request and test them once the 1.12 release is done.

RangerRick pushed a commit that referenced this pull request May 1, 2013
This reverts commit 5f19787, reversing
changes made to 6288396.
@ghost
Copy link
Author

ghost commented May 2, 2013

We have merged the changes with master branch as 1.12 branch is very close to the release.

DerTak added a commit that referenced this pull request Aug 10, 2014
Missing stuff and cleanup
indigo423 added a commit that referenced this pull request Jan 24, 2015
Fixed and enhanced
- I have merged this pull request make sense
indigo423 pushed a commit that referenced this pull request Jul 6, 2015
NMS-5272: Removed unnecessary spaces and formatted file.
j-white pushed a commit that referenced this pull request Dec 10, 2015
deejgregor pushed a commit that referenced this pull request Oct 10, 2022
We published an image for ALEC in Docker Hub, so the Chart is fully functional.
GitHub won't be used anymore.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant