From cf1ced6db157d045f4fc57e5928de541366bd11a Mon Sep 17 00:00:00 2001
From: pierrepomes <pierre.pomes@interface-tech.com>
Date: Mon, 8 Mar 2010 21:20:11 -0500
Subject: Handle empty OVPN client array correctly. Ticket #397

---
 usr/local/www/status_openvpn.php | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

(limited to 'usr/local/www')

diff --git a/usr/local/www/status_openvpn.php b/usr/local/www/status_openvpn.php
index 1132fd8..2ef71ce 100644
--- a/usr/local/www/status_openvpn.php
+++ b/usr/local/www/status_openvpn.php
@@ -89,6 +89,7 @@ function kill_client($port, $remipp) {
 }
 
 $servers = array();
+$clients = array();
 
 if (is_array($config['openvpn']['openvpn-server'])) {
 	foreach ($config['openvpn']['openvpn-server'] as & $settings) {
@@ -342,6 +343,7 @@ include("head.inc"); ?>
 <br>
 
 
+<?php if (!empty($clients)) { ?>
 <table style="padding-top:0px; padding-bottom:0px; padding-left:0px; padding-right:0px" width="100%" border="0" cellpadding="0" cellspacing="0">
 	<tr>
 		<td colspan="6" class="listtopic">
@@ -389,12 +391,14 @@ include("head.inc"); ?>
 	</tr>
 </table>
 
-<?php if ($DisplayNote) {
+<?php 
+}
+
+if ($DisplayNote) {
 	echo "<br/><b>NOTE:</b> You need to bind each OpenVPN client to enable its management daemon: use 'Local port' setting in the OpenVPN client screen";
 }
-?>
 
-<? if ((!isset($clients)) && (!isset($servers))) {
+if ((empty($clients)) && (empty($servers))) {
 	echo "No OpenVPN instance defined";
 }
 ?>
-- 
cgit v1.1