remove device/details

lib/App/Netdisco/Web/Plugin/Device/Details.pm

@@ -1,7 +1,6 @@
 package App::Netdisco::Web::Plugin::Device::Details;
 
 use Dancer ':syntax';
-use Dancer::Plugin::Ajax;
 use Dancer::Plugin::DBIC;
 use Dancer::Plugin::Auth::Extensible;
 use Dancer::Plugin::Swagger;
@@ -10,23 +9,29 @@ use App::Netdisco::Web::Plugin;
 
 register_device_tab({ tag => 'details', label => 'Details' });
 
-# device details table
+# forward API call to AJAX route handler
 swagger_path {
     description => 'Get properties and power details for a device.',
-    path => '/api/device/{identifier}/details',
     tags => ['Devices'],
     parameters => [
-        { name => 'identifier', in => 'path', required => 1, type => 'string' },
+        identifier => {in => 'path', required => 1, type => 'string' },
     ],
     responses => { default => { examples => {
         # TODO document fields returned
-        'application/json' => { device => {}, power => {} },
+        'application/json' => { device => {} },
     } } },
 },
+get '/api/device/:identifier' => require_login sub {
+    vars->{'is_api'} = 1;
+    forward '/ajax/content/device/details',
+      { tab => 'details', q => params->{'identifier'} };
+};
+
+# device details table
 get '/ajax/content/device/details' => require_login sub {
     my $q = param('q');
     my $device = schema('netdisco')->resultset('Device')
-      ->search_for_device($q) or send_error('Bad device', 400);
+      ->search_for_device($q) or return bang('Bad device', 400);
 
     my @results
         = schema('netdisco')->resultset('Device')
@@ -37,11 +42,12 @@ get '/ajax/content/device/details' => require_login sub {
         = schema('netdisco')->resultset('DevicePower')
         ->search( { 'me.ip' => $device->ip } )->with_poestats->hri->all;
 
+    delete $results[0]->{'snmp_comm'};
+    $results[0]->{'power'} = \@power;
+
     if (vars->{'is_api'}) {
         content_type('application/json');
-        # TODO merge power into device details
-        # TODO remove sensitive data (community)
-        to_json { device => $results[0], power => \@power };
+        to_json { device => $results[0] };
     }
     else {
         content_type('text/html');
@@ -51,4 +57,4 @@ get '/ajax/content/device/details' => require_login sub {
     }
 };
 
-1;
+true;