From f1de543acf1d3f5fa4aeef6f704cae8a9010ec86 Mon Sep 17 00:00:00 2001
From: "Eric A. Miller" <emiller@cpan.org>
Date: Fri, 8 Nov 2013 19:50:50 -0500
Subject: [PATCH] Don't try to convert negative mW to dBm in AP power report

---
 Netdisco/Changes                                                | 1 +
 .../lib/App/Netdisco/Web/Plugin/Report/ApRadioChannelPower.pm   | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/Netdisco/Changes b/Netdisco/Changes
index 4b3d4db9..ddc6d1c0 100644
--- a/Netdisco/Changes
+++ b/Netdisco/Changes
@@ -21,6 +21,7 @@
 
   * Set of Port Name (description) fixed to assume OK dialog response
   * Default class of a PoE port is 0 if not otherwise defined
+  * Don't try to convert negative mW to dBm in AP power report
 
 2.019003 - 2013-10-27
 
diff --git a/Netdisco/lib/App/Netdisco/Web/Plugin/Report/ApRadioChannelPower.pm b/Netdisco/lib/App/Netdisco/Web/Plugin/Report/ApRadioChannelPower.pm
index 57d64e50..95cc7b67 100644
--- a/Netdisco/lib/App/Netdisco/Web/Plugin/Report/ApRadioChannelPower.pm
+++ b/Netdisco/lib/App/Netdisco/Web/Plugin/Report/ApRadioChannelPower.pm
@@ -22,7 +22,7 @@ sub port_tree {
     foreach my $device (@$devices) {
         my $power2;
 
-        if ( defined( $device->power ) && $device->power ) {
+        if ( defined( $device->power ) && $device->power > 0 ) {
             $power2 = sprintf( "%.1f",
                 10.0 * CORE::log( $device->power ) / CORE::log(10) );
         }