From 48edd028fc3d2c72866fbee9d1f1bc91ddf11568 Mon Sep 17 00:00:00 2001
From: Matteo Pasotti <matteo.pasotti@gmail.com>
Date: Thu, 8 Jan 2015 12:50:02 +0100
Subject: added brief comment explaining how to read shorewall conf from now on

---
 lib/AdminPanel/Shared/Shorewall.pm | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'lib')

diff --git a/lib/AdminPanel/Shared/Shorewall.pm b/lib/AdminPanel/Shared/Shorewall.pm
index af18d60e..f043e86d 100644
--- a/lib/AdminPanel/Shared/Shorewall.pm
+++ b/lib/AdminPanel/Shared/Shorewall.pm
@@ -108,8 +108,15 @@ sub read_ {
 
     return \%conf;
     
+    # get_zones has been moved to AdminPanel::Module::Firewall cause it requires
+    # user interaction thus it should be logically separated by shorewall
     # get_zones(\%conf);
     # get_config_file('zones', $ver) && \%conf;
+    # consequently, to read shorewall conf
+    # you have to do something like this now (within Module::Firewall)
+    # my $conf = AdminPanel::Shared::Shorewall::read_();
+    # OPTIONAL: my $self->get_zones(\$conf)
+    # my $shorewall = AdminPanel::Shared::Shorewall::get_config_file('zones', '') && $conf;
 }
 
 sub ports_by_proto {
-- 
cgit v1.2.1