Changeset 11668

Timestamp:

03/04/11 00:32:39 (2 years ago)

Author:

p_lindheimer

Message:

fixes #4884 allows FreePBX to set various options and ARI settings will preserve but hide what is in FreePBX

Files:

• freepbx/branches/2.9/amp_conf/htdocs/recordings/modules/settings.module (modified) (12 diffs)

freepbx/branches/2.9/amp_conf/htdocs/recordings/modules/settings.module

@@ -48 +48 @@
-    ),
-/* until some parsing of available formats is enabled, hide this option
+*/
-  "attachfmt" => array(
-    "type" => "email",
-    "label" => _("Email attachment format"),
-    "tooltip" => _("Audio format of message to attach to email."),
-text
+hidden
-    "input_size" => 20
-    ),
-*/
-  "backupdeleted" => array(
-    "type" => "files",
@@ -64 +64 @@
-    ),
-/* allowing users to specify context is security issue, hide this option
+*/
-  "callback" => array(
-    "type" => "phone",
-    "label" => _("Call back context"),
-    "tooltip" => _("Dialing context used for calling back the caller that left a message."),
-text
+hidden
-    "input_size" => 80
-    ),
-*/
-  "delete" => array(
-    "type" => "email",
@@ -80 +80 @@
-    ),
-/* allowing users to specify context is security issue, hide this option
+*/
-  "dialout" => array(
-    "type" => "phone",
-    "label" => _("Dial out context"),
-    "tooltip" => _("Dialing context used to dial out from voicemail."),
-text
+hidden
-    "input_size" => 80
-    ),
-*/
-  "envelope" => array(
-    "type" => "phone",
@@ -96 +96 @@
-    ),
-/* allowing users to specify context is security issue, hide this option
+*/
-  "exitcontext" => array(
-    "type" => "phone",
-    "label" => _("Operator context"),
-    "tooltip" => _("Dialing context for handling * or 0 calls to operator from voicemail."),
-text
+hidden
-    "input_size" => 80
-    ),
+/* forcegreetings and forcename should only be modified by voicemail admin, hide these options
-*/
-/* forcegreetings and forcename should only be modified by voicemail admin, hide these options
-  "forcegreetings" => array(
-    "type" => "phone",
-    "label" => _("Force greeting recording"),
-    "tooltip" => _("Forces a new user to record a greeting. A new user is determined by the password being the same as the mailbox number."),
-checkbox
+hidden
-    "input_size" => null
-    ),
@@ -116 +117 @@
-    "label" => _("Force name recording"),
-    "tooltip" => _("Forces a new user to record their name. A new user is determined by the password being the same as the mailbox number."),
-checkbox
+hidden
-    "input_size" => null
-    ),
-*/
-/* this option is not used by app_voicemail.c, but by app_directory.c
-   FreePBX does not use app_directory.c, so hide this option
@@ -132 +132 @@
-/* imap options are only effective if using imap voicemail storage
-   FreePBX does not support imap voiceemail storage, so hide these options
+*/
-  "imappassword" => array(
-    "type" => "imap",
-    "label" => _("IMAP email account password"),
-    "tooltip" => _("Password for the IMAP email account used to store voicemail messages."),
-text
+hidden
-    "input_size" => 80
-    ),
@@ -143 +144 @@
-    "label" => _("IMAP email user name"),
-    "tooltip" => _("User name for the IMAP email account used to store voicemail messages."),
-text
+hidden
-    "input_size" => 80
-    ),
-*/
-/* user can break voicemail for self by switching to a language the server
-   is not configured for, so hide this option
+*/
-  "language" => array(
-    "type" => "phone",
-    "label" => _("Voice prompt language"),
-    "tooltip" => _("Enter language code to use for voice prompts. Before non-English languages can be used, the voice prompts must be installed by the system administrator."),
-text
+hidden
-    "input_size" => 20
-    ),
-*/
-  "maxmsg" => array(
-    "type" => "files",
@@ -180 +180 @@
-    ),
-/* operator destination may not be available on a system, so hide this option
+*/
-  "operator" => array(
-    "type" => "caller",
-    "label" => _("Dial 0 for Operator"),
-    "tooltip" => _("Allow caller to reach an operator by pressing 0 before, after, or while leaving a voicemail."),
-checkbox
+hidden
-    "input_size" => null
-    ),
-*/
-  "review" => array(
-    "type" => "caller",
@@ -217 +217 @@
-    ),
-/* administrator may not want user to be able to change this, so hide this option
+*/
-  "sendvoicemail" => array(
-    "type" => "phone",
-    "label" => _("Send internal voicemail"),
-    "tooltip" => _("Allow a user logged into voicemail to compose and send voicemail messages to other voicemail users."),
-checkbox
+hidden
-    "input_size" => null
-    ),
-*/
-  "serveremail" => array(
-    "type" => "email",
@@ -240 +240 @@
-    ),
-/* timezone names are custom per installtion, so hide this option
+*/
-  "tz" => array(
-    "type" => "phone",
-    "label" => _("Timezone"),
-    "tooltip" => _("Timezone from zonemessages context. Irrelevant if Envelope is disabled."),
-text
+hidden
-    "input_size" => 80
-    ),
-*/
-  "volgain" => array(
-    "type" => "email",
@@ -308 +308 @@
-  function is_textbox($opt) {
-    if ($this->voicemail_options_table[$opt]["input_type"] == "text") {
+      return true;
+    } else {
+      return false;
+    }
+  }
+  function is_hidden($opt) {
+    if ($this->voicemail_options_table[$opt]["input_type"] == "hidden") {
-      return true;
-    } else {
@@ -905 +912 @@
-          }
-        }
+        else if ($this->is_hidden($key)) {
+          $current_option_text_input = "input type=hidden name='" . $var . "' value='" . $value . "' ";
+          $other_opts_text .= "<tr><td colspan='3'><" . $current_option_text_input . "></td></tr>";
+        }
-        else if ($this->is_checkbox($key)) {
-          $current_option_text_input = "input " . $$var_enable . " " . $$var_text_box_options . " type=checkbox name='" . $var . "' value='yes'";