BIND package code clean up

Code cleanup to reflect coding style guidelines and consistency
Tabs only, always uses braces, spacing
C/C++ style comments
Use array_key_exists instead of key_exists
$config[] keys are always enclosed in quotes and use single quotes for literals

1 files changed, 69 insertions, 59 deletions

diff --git a/config/bind/bind_views.xml b/config/bind/bind_views.xml
index 505f2b0d..7d38f481 100644
--- a/config/bind/bind_views.xml
+++ b/config/bind/bind_views.xml
@@ -2,8 +2,8 @@
 <!DOCTYPE packagegui SYSTEM "./schema/packages.dtd">
 <?xml-stylesheet type="text/xsl" href="./xsl/package.xsl"?>
 <packagegui>
-        <copyright>
-        <![CDATA[
+	<copyright>
+		<![CDATA[
 /* $Id$ */
 /* ========================================================================== */
 /*
@@ -42,10 +42,10 @@
                                                                               */
 /* ========================================================================== */
         ]]>
-        </copyright>
-    <description>Describe your package here</description>
-    <requirements>Describe your package requirements here</requirements>
-    <faq>Currently there are no FAQ items provided.</faq>
+	</copyright>
+	<description>Describe your package here</description>
+	<requirements>Describe your package requirements here</requirements>
+	<faq>Currently there are no FAQ items provided.</faq>
 	<name>bindviews</name>
 	<version>0.1.0</version>
 	<title>Bind: Views Settings</title>
@@ -56,29 +56,29 @@
 		<section>Services</section>
 		<configfile>bind.xml</configfile>
 	</menu>
-	        <tabs>
-                <tab>
-                        <text>Settings</text>
-                        <url>/pkg_edit.php?xml=bind.xml</url>
-                </tab>
-                <tab>
-                        <text>ACLs</text>
-                        <url>/pkg.php?xml=bind_acls.xml</url>
-                </tab>
-                <tab>
-                        <text>Views</text>
-                        <url>/pkg.php?xml=bind_views.xml</url>
-                        <active/>
-                </tab>
-                <tab>
-                        <text>Zones</text>
-                        <url>/pkg.php?xml=bind_zones.xml</url>
-                </tab>
-                <tab>
-                        <text>Sync</text>
-                         <url>/pkg_edit.php?xml=bind_sync.xml</url>
-                </tab>
-        </tabs>
+	<tabs>
+		<tab>
+			<text>Settings</text>
+			<url>/pkg_edit.php?xml=bind.xml</url>
+		</tab>
+		<tab>
+			<text>ACLs</text>
+			<url>/pkg.php?xml=bind_acls.xml</url>
+		</tab>
+		<tab>
+			<text>Views</text>
+			<url>/pkg.php?xml=bind_views.xml</url>
+			<active/>
+		</tab>
+		<tab>
+			<text>Zones</text>
+			<url>/pkg.php?xml=bind_zones.xml</url>
+		</tab>
+		<tab>
+			<text>Sync</text>
+			<url>/pkg_edit.php?xml=bind_sync.xml</url>
+		</tab>
+	</tabs>
 	<configpath>['installedpackages']['bindviews']['config']</configpath>
 	<adddeleteeditpagefields>
 		<columnitem>
@@ -106,19 +106,26 @@
 			<type>input</type>
 		</field>
 		<field>
-           <fielddescr>Recursion</fielddescr>
-           <fieldname>recursion</fieldname>
-           <description>A recursive query occurs when your DNS server is queried for a domain that it currently knows nothing about, in which case it will try to resolve the given host by performing further queries (eg by starting at the root servers and working out, or by simply passing the request to yet another DNS server).</description>
-           <type>select</type>
-	       <options>
-			<option><name>No</name><value>no</value></option>
-			<option><name>Yes</name><value>yes</value></option>
-		   </options>
+			<fielddescr>Recursion</fielddescr>
+			<fieldname>recursion</fieldname>
+			<description>A recursive query occurs when your DNS server is queried for a domain that it
+					currently knows nothing about, in which case it will try to resolve the given host by
+					performing further queries (eg by starting at the root servers and working out, or by
+					simply passing the request to yet another DNS server).
+			</description>
+			<type>select</type>
+			<options>
+				<option><name>No</name><value>no</value></option>
+				<option><name>Yes</name><value>yes</value></option>
+			</options>
 		</field>
 		<field>
 			<fielddescr>Match-clients</fielddescr>
 			<fieldname>match-clients</fieldname>
-			<description>If either or both of match-clients are missing they default to any (all hosts match). The match-clients statement defines the address_match_list for the source IP address of the incoming messages.</description>
+			<description>If either or both of match-clients are missing they default to any (all hosts
+					match). The match-clients statement defines the address_match_list for the source IP
+					address of the incoming messages.
+			</description>
 			<type>select_source</type>
 			<source><![CDATA[$config['installedpackages']['bindacls']['config']]]></source>
 			<source_name>name</source_name>
@@ -129,8 +136,10 @@
 		<field>
 			<fielddescr>Allow-recursion</fielddescr>
 			<fieldname>allow-recursion</fieldname>
-			<description>For example, if you have one DNS server serving your local network, you may want all of your local computers to use your DNS server.</description>
-	    	<type>select_source</type>
+			<description>For example, if you have one DNS server serving your local network, you may want
+					all of your local computers to use your DNS server.
+			</description>
+			<type>select_source</type>
 			<source><![CDATA[$config['installedpackages']['bindacls']['config']]]></source>
 			<source_name>name</source_name>
 			<source_value>name</source_value>
@@ -138,25 +147,26 @@
 			<size>03</size>
 		</field>
 		<field>
-                        <type>listtopic</type>
-                        <name>Custom Views </name>
-                        <fieldname>temp</fieldname>
-                </field>
-	        <field>
-                        <fielddescr>Custom Options</fielddescr>
-                        <fieldname>bind_custom_options</fieldname>
-                        <description>You can put your own custom options here, separated by semi-colons (;).</description>
-                        <type>textarea</type>
-                        <cols>65</cols>
-                        <rows>8</rows>
-                        <encoding>base64</encoding>
-                </field>
+			<type>listtopic</type>
+			<name>Custom Views</name>
+			<fieldname>temp</fieldname>
+		</field>
+		<field>
+			<fielddescr>Custom Options</fielddescr>
+			<fieldname>bind_custom_options</fieldname>
+			<description>You can put your own custom options here, separated by semi-colons (;).
+			</description>
+			<type>textarea</type>
+			<cols>65</cols>
+			<rows>8</rows>
+			<encoding>base64</encoding>
+		</field>
 	</fields>
-        <custom_php_command_before_form>
-        </custom_php_command_before_form>
+	<custom_php_command_before_form>
+	</custom_php_command_before_form>
 	<custom_delete_php_command>
-	</custom_delete_php_command>	
-        <custom_php_resync_config_command>
-	        bind_sync();
-        </custom_php_resync_config_command>
+	</custom_delete_php_command>
+	<custom_php_resync_config_command>
+		bind_sync();
+	</custom_php_resync_config_command>
 </packagegui>