Ticket #3591 (closed Bugs: fixed)

Opened 1 year ago

Last modified 11 months ago

Bug with 'Administrators' module

Reported by: Oleh Assigned to: p_lindheimer
Priority: minor Milestone: 2.6
Component: Core Version: 2.5-branch
Keywords: Cc:
Confirmation: Need testing SVN Revision (if applicable):
Backend Engine: Asterisk 1.4.x Backend Engine Version:

Description

After last update (framework 2.5.1.2, core 2.5.1.6) when open Administrators module i se this message: NOTE:AUTHTYPE is not set to 'database' in /etc/amportal.conf - note that this module is not currently providing access control, and changing passwords here or adding users will have no effect unless AUTHTYPE is set to 'database'.

But AUTHTYPE is database and everything is ok and authentication works like before update.

Change History

03/20/09 09:00:27 changed by p_lindheimer

  • confirmation changed from Unreviewed to Need testing.
  • component changed from - choose - to Core.

oops, have a look at this patch, let me know. I'll look at getting the module updated and published this weekend if this is ok: 

Index: page.ampusers.php
===================================================================
--- page.ampusers.php	(revision 7542)
+++ page.ampusers.php	(working copy)
@@ -159,7 +159,7 @@
 					<h4><?php echo _("General Settings")?></h4>
 				</td>
 			</tr>
-<?php if (($amp_conf["AUTHTYPE"] != "database") || ($amp_conf["AUTHTYPE"] != "webserver")) { ?>			
+<?php if (($amp_conf["AUTHTYPE"] != "database") && ($amp_conf["AUTHTYPE"] != "webserver")) { ?>			
 			<tr>
 				<td colspan="2">
 	<?php echo '<b>'._("NOTE:").'</b>'._("AUTHTYPE is not set to 'database' in /etc/amportal.conf - note that this module is not currently providing access control, and changing passwords here or adding users will have no effect unless AUTHTYPE is set to 'database'.") ?><br /><br />

03/23/09 08:15:18 changed by Oleh

Thanks, this patch works for me.

04/27/09 16:36:11 changed by p_lindheimer

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

r7543 - will be published shortly.

05/06/09 12:15:58 changed by p_lindheimer

(In [7647]) Merged revisions 7525-7553,7555-7612,7614-7646 via svnmerge from http://svn.freepbx.org/modules/branches/2.5

........

r7543 | mickecarlsson | 2009-03-21 05:37:55 -0700 (Sat, 21 Mar 2009) | 1 line

Fixes #3591 logic should be AND instead of OR

........

r7571 | mickecarlsson | 2009-04-06 11:18:35 -0700 (Mon, 06 Apr 2009) | 1 line

Re #3616, adds Portuguese (Brasil) to varous modules

........

r7612 | p_lindheimer | 2009-05-02 09:52:03 -0700 (Sat, 02 May 2009) | 1 line

fixes #3653 Skip Busy Agent ignored in some Follow-Me configurations under a queue

........

r7624 | p_lindheimer | 2009-05-02 11:40:34 -0700 (Sat, 02 May 2009) | 1 line

fixes #3457 don't use _. since Asterisk 1.2 will take that over more explicit matches like _s-.

........

r7626 | p_lindheimer | 2009-05-02 12:21:14 -0700 (Sat, 02 May 2009) | 1 line

fixes #3517 no need to test nodest variable, it will be set and empty if not set in the channel

........

r7627 | p_lindheimer | 2009-05-02 13:15:16 -0700 (Sat, 02 May 2009) | 1 line

fixes #3618 generate announce-position in queues.conf for Asterisk 1.6 (yes or no only for now)

........

r7644 | mickecarlsson | 2009-05-05 13:54:37 -0700 (Tue, 05 May 2009) | 1 line

Re #3657, updated Brazilian Portuguese language files

........