Merge pull request #27 from jbrooksuk/api-improvements

Incidents API

app/controllers/ApiController.php

@@ -19,4 +19,16 @@
 			return $component->incidents;
 		}
 
+		public function getIncidents() {
+			return Incident::all();
+		}
+
+		public function getIncident($id) {
+			if ($incident = Incident::find($id)) {
+				return $incident;
+			} else {
+				App::abort(404, 'Incident not found');
+			}
+		}
+
 	}

app/models/Component.php

@@ -20,7 +20,7 @@
 		/**
 		 * Get the transformer instance.
 		 *
-		 * @return mixed
+		 * @return ComponentTransformer
 		 */
 		public function getTransformer() {
 			return new ComponentTransformer();

app/models/Incident.php

@@ -1,6 +1,6 @@
 <?php
 
-	class Incident extends Eloquent {
+	class Incident extends Eloquent implements Dingo\Api\Transformer\TransformableInterface {
 		/**
 		 * An incident belongs to a component.
 		 * @return Illuminate\Database\Eloquent\Relations\BelongsTo
@@ -29,4 +29,13 @@
 				case 4: return 'glyphicon-ok';
 			}
 		}
+
+		/**
+		 * Get the transformer instance.
+		 *
+		 * @return IncidentTransformer
+		 */
+		public function getTransformer() {
+			return new IncidentTransformer();
+		}
 	}

app/routes/api.php

@@ -5,5 +5,7 @@
 		Route::get('components', 'ApiController@getComponents');
 		Route::get('components/{id}', 'ApiController@getComponent');
 		Route::get('components/{id}/incidents', 'ApiController@getComponentIncidents');
+		Route::get('incidents', 'ApiController@getIncidents');
+		Route::get('incidents/{id}', 'ApiController@getIncident');
 
 	});

app/transformers/ComponentTransformer.php

@@ -8,8 +8,9 @@
 				'description' => $component->description,
 				'status_id' => (int) $component->status,
 				'status' => $component->getHumanStatusAttribute(),
-				'incident_count' => $component->incidents()->count()
+				'incident_count' => $component->incidents()->count(),
+				'created_at' => $component->created_at->timestamp,
+				'updated_at' => $component->updated_at->timestamp,
 			];
 		}
-
 	}

app/transformers/IncidentTransformer.php

@@ -0,0 +1,19 @@
+<?php
+
+	class IncidentTransformer extends \League\Fractal\TransformerAbstract {
+		public function transform(Incident $incident) {
+			$component = $incident->parent;
+			$transformer = $component->getTransformer();
+
+			return [
+				'id' => (int) $incident->id,
+				'name' => $incident->name,
+				'message' => $incident->message,
+				'status_id' => (int) $incident->status,
+				'status' => $incident->getHumanStatusAttribute(),
+				'component' => $transformer->transform($component),
+				'created_at' => $incident->created_at->timestamp,
+				'updated_at' => $incident->updated_at->timestamp,
+			];
+		}
+	}