From a0cdca9e7c568a003dc755857dc21ddd70efaffb Mon Sep 17 00:00:00 2001
From: Matt Smith <psykzz@users.noreply.github.com>
Date: Thu, 30 Nov 2017 02:55:45 +0000
Subject: [PATCH] Fix turbine computer showing broken

---
 code/modules/power/turbine.dm             | 14 +++++++++-----
 tgui/src/interfaces/turbine_computer.ract | 18 +++++++++---------
 2 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/code/modules/power/turbine.dm b/code/modules/power/turbine.dm
index ac664894de..8b88193e21 100644
--- a/code/modules/power/turbine.dm
+++ b/code/modules/power/turbine.dm
@@ -344,10 +344,10 @@
 /obj/machinery/computer/turbine_computer/ui_data(mob/user)
 	var/list/data = list()
 
-	data["working"] = (compressor.starter && compressor && compressor.turbine && !compressor.stat && !compressor.turbine.stat)
 	data["connected"] = (compressor && compressor.turbine) ? TRUE : FALSE
-	data["compressor_broke"] = (!compressor || compressor.stat) ? TRUE : FALSE
-	data["turbine_broke"] = (!compressor || compressor.turbine.stat) ? TRUE : FALSE
+	data["compressor_broke"] = (!compressor || (compressor.stat & BROKEN)) ? TRUE : FALSE
+	data["turbine_broke"] = (!compressor || !compressor.turbine || (compressor.turbine.stat & BROKEN)) ? TRUE : FALSE
+	data["broken"] = (data["compressor_broke"] || data["turbine_broke"])
 	data["online"] = compressor.starter
 
 	data["power"] = DisplayPower(compressor.turbine.lastgen)
@@ -360,9 +360,13 @@
 	if(..())
 		return
 	switch(action)
-		if("power")
+		if("power-on")
 			if(compressor && compressor.turbine)
-				compressor.starter = !compressor.starter
+				compressor.starter = TRUE
+				. = TRUE
+		if("power-off")
+			if(compressor && compressor.turbine)
+				compressor.starter = FALSE
 				. = TRUE
 		if("reconnect")
 			locate_machinery()
diff --git a/tgui/src/interfaces/turbine_computer.ract b/tgui/src/interfaces/turbine_computer.ract
index 0e24823b39..325c5b4581 100644
--- a/tgui/src/interfaces/turbine_computer.ract
+++ b/tgui/src/interfaces/turbine_computer.ract
@@ -13,20 +13,20 @@ component.exports = {
 <ui-notice>
 	<ui-section label='Reconnect'>
 		<div style='float:right'>
-			<ui-button icon='refresh' action='reconnect' state='{{data.connected}}'>Reconnect</ui-button>
+			<ui-button icon='refresh' action='reconnect'>Reconnect</ui-button>
 		</div>
 	</ui-section>
 </ui-notice>
 <ui-display title='Turbine Controller'>
 	<ui-section label='Status'>
-		{{#if data.working}}
-			<span class='{{powerState(data.online)}}'>{{data.online && !(data.compressor_broke || data.turbine_broke) ? "Online" : "Offline"}}</span>
-		{{else}}
+		{{#if data.broken}}
 			<span class='bad'>Broken</span>
+		{{else}}
+			<span class='{{powerState(data.online)}}'>{{data.online && !(data.compressor_broke || data.turbine_broke) ? "Online" : "Offline"}}</span>
 		{{/if}}
 		<div style='float:right'>
-			<ui-button icon='power-off' action='power' state='{{data.working}}' style='{{data.online ? "selected" : ""}}'>On</ui-button>
-			<ui-button icon='close' action='power' state='{{data.working}}' style='{{data.online ? "" : "selected"}}'>Off</ui-button>
+			<ui-button icon='power-off' action='power-on' state='{{data.broken}}' style='{{data.online ? "selected" : ""}}'>On</ui-button>
+			<ui-button icon='close' action='power-off' state='{{data.broken}}' style='{{data.online ? "" : "selected"}}'>Off</ui-button>
 		</div>
 		{{#if data.compressor_broke}}
 			<br>
@@ -40,12 +40,12 @@ component.exports = {
 </ui-display>
 <ui-display title='Status'>
 	<ui-section label='Turbine Speed'>
-		<span>{{data.working ? data.rpm : "--"}} RPM</span>
+		<span>{{data.broken ? "--" : data.rpm}} RPM</span>
 	</ui-section>
 	<ui-section label='Internal Temp'>
-		<span>{{data.working ? data.temp : "--"}} K</span>
+		<span>{{data.broken ? "--" : data.temp}} K</span>
 	</ui-section>
 	<ui-section label='Generated Power'>
-		<span>{{data.working ? data.power : "--"}}</span>
+		<span>{{data.broken ? "--" : data.power}}</span>
 	</ui-section>
 </ui-display>