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
[EUWE] Pull in the real power_state column off of service #382
[EUWE] Pull in the real power_state column off of service #382
Conversation
powerOperationInProgressState: function(item) { | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "suspending"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
|| (item.powerState !== "timeout" && item.powerStatus === "suspending"); | ||
powerOperationInProgressState: function(item) { | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
|| (item.powerState !== "timeout" && item.powerStatus === "suspending"); | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "suspending"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
|| (item.powerState !== "timeout" && item.powerStatus === "stopping") | ||
|| (item.powerState !== "timeout" && item.powerStatus === "suspending"); | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
|| (item.powerState !== "timeout" && item.powerStatus === "suspending"); | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "suspending"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
|| (item.powerState !== "timeout" && item.powerStatus === "stopping") | ||
|| (item.powerState !== "timeout" && item.powerStatus === "suspending"); | ||
return (item.power_state !== "timeout" && item.options.power_status === "starting") | ||
|| (item.power_state !== "timeout" && item.options.power_status === "stopping") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad line breaking before '||'.
It's OK to ignore those hound violations (they are not valid. I always see them only on the Euwe branch) @chessbyte This is the Euwe equivalent of the Upstream PR #377 |
@chriskacerguis please review before I merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the Hound issues, this LGTM. Good to merge
This PR pulls in the service.power_state directly from the virtual power_state column vs the cached version previously exposed in the options hash.
https://bugzilla.redhat.com/show_bug.cgi?id=1396728
This is Upstream equivalent of #377