[asterisk-bugs] [Asterisk 0012248]: [patch] Add option to return all available channels on app_chanisavail

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12248 
====================================================================== 
Reported By:                dagmoller
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12248
Category:                   Applications/NewFeature
Reproducibility:            N/A
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 109355 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-18-2008 07:18 CDT
Last Modified:              05-23-2008 07:07 CDT
====================================================================== 
Summary:                    [patch] Add option to return all available channels
on app_chanisavail
Description: 
Added option 'a' that check and return all available channels, not only the
first one.

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0012619 added feature
====================================================================== 

---------------------------------------------------------------------- 
 russell - 05-23-08 07:07  
---------------------------------------------------------------------- 
One last suggestion ...

+		if (tmp_availchan->used) {
+			ast_str_append(&tmp_availchan, 0, "%s", "&");
+		}
+		ast_str_append(&tmp_availchan, 0, "%s", tempchan->name);

This could be ....

ast_str_append(&tmp_availchan, 0, "%s%s", tmp_availchan->used ? "&" : "",
tempchan->name); 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-23-08 07:07  russell        Note Added: 0087242                          
======================================================================