[asterisk-bugs] [Asterisk-GUI 0011882]: [patch] Zap/BRI channels reported to have 30 channels

[noreply at bugs.digium.com]

The following issue has been CLOSED 
====================================================================== 
http://bugs.digium.com/view.php?id=11882 
====================================================================== 
Reported By:                tzafrir
Assigned To:                bkruse
====================================================================== 
Project:                    Asterisk-GUI
Issue ID:                   11882
Category:                   Card Detection
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 2155 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 open
Fixed in Version:           
====================================================================== 
Date Submitted:             01-30-2008 15:05 CST
Last Modified:              04-28-2008 09:29 CDT
====================================================================== 
Summary:                    [patch] Zap/BRI channels reported to have 30
channels
Description: 
The asterisk-gui assumes that a digital span is either E1 or T1. Hence the
line:

  SPANS[l]['reserved_ch'] = Number(n[l]['basechan']) + (
(Number(n[l]['totchans'])==24)? 23 : 15 );

A span may also be BRI, which has 2 B channels and a D channel.

A simpler way would be to just set the number to "number of channels" - 1.
 Asterisk uses this in chan_zap and I use this calculation in
genzaptelconf. Is there any place where it would fail?
====================================================================== 

---------------------------------------------------------------------- 
 bkruse - 04-28-08 09:29  
---------------------------------------------------------------------- 
Hey Tzafrir.


Since BRI/MISDN stuff is on another page now, there will be no need for
this patch.

Giving some karma for the effort and find, thank you :]

-bk 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-28-08 09:29  bkruse         Note Added: 0086095                          
======================================================================