Ticket #4461 (closed Bugs: fixed)

Opened 3 years ago

Last modified 3 years ago

Recordings portal won't apply changes

Reported by: peterbokunet Assigned to:
Priority: minor Milestone: 2.9
Component: Other Module Version: 2.7-branch
Keywords: Cc:
Confirmation: Unreviewed Distro:
Backend Engine: Asterisk 1.6 Distro Ver:
Backend Ver: SVN Revision (if applicable):

Description

Starting with asterisk 1.6 the AMI command "Reload app_voicemail.so" will fail [silently]. Updates made through the .../recordings will apply to the voicemail.conf file but the asterisk reload won't pick up the changes.

Because the following patch is backwards compatible with 1.4.x (I tested on 1.4.30) it should be safe to apply universally. Without this change, any changes through the recordings portal will fail to apply until the module is reload through other means.

This was discovered trying to update the email address field. 

--- modules/settings.module.orig	2010-07-21 13:14:12.000000000 -0500
+++ modules/settings.module	2010-07-21 13:11:02.000000000 -0500
@@ -1349,7 +1349,7 @@ class Settings {
 
     global $asterisk_manager_interface;
 
-    $response = $asterisk_manager_interface->Command("Action: Command\r\nCommand: Reload app_voicemail.so\r\n\r\n");
+    $response = $asterisk_manager_interface->Command("Action: Command\r\nCommand: Module Reload app_voicemail.so\r\n\r\n");
   }
 
   /*

Thanks,

Change History

07/21/10 14:52:04 changed by p_lindheimer

  • status changed from new to closed.
  • resolution set to fixed.

(In [10169]) fixes #4461 adapt to 1.4+ asterisk syntax. This change will break 1.2 systems

09/15/10 20:14:00 changed by p_lindheimer

(In [10285]) Merged revisions 10163-10171,10173-10284 via svnmerge from http://www.freepbx.org/v2/svn/freepbx/branches/2.8

........

r10169 | p_lindheimer | 2010-07-21 11:52:03 -0700 (Wed, 21 Jul 2010) | 1 line

fixes #4461 adapt to 1.4+ asterisk syntax. This change will break 1.2 systems

........

r10226 | p_lindheimer | 2010-08-24 16:08:16 -0700 (Tue, 24 Aug 2010) | 1 line

fixes #4509 and re #4134 use proper path for call monitor

........

r10246 | mickecarlsson | 2010-09-06 22:25:50 -0700 (Mon, 06 Sep 2010) | 1 line

Re #3503 Add Chinese language as selection in FreePBX, it was removed in r7337 and I forgot to add it back when proper language files was submitted. This change should be merged to all supported version

........

r10268 | p_lindheimer | 2010-09-14 15:56:42 -0700 (Tue, 14 Sep 2010) | 1 line

fixes #4501 change to 1.4+ syntax (this breaks 1.2 I believe - WARNING)

........