[Asterisk-Dev] Chan_VPB - Brazilian CallerID patch

[Jorge Merlino]

Hi Daniel,

That patch didn't work for me.  I'm in Uruguay but the ID is basically 
the same only that here it starts with a D instead of an A.
Your callerid detection works fine but Welber's code to copy the data 
into the asterisk structure is in the wrong place. I could manage to 
make it work inserting the code in the VPB_RING option of the notowned 
handler.
I'll clean up my debug code and send another patch soon.

Regards,
    Jorge


Daniel Bichara wrote:

> Hi all,
>
> Attached patched to implement Brazilian CallerID detection feature to 
> chan_vpb.c
>
> Daniel
>
>------------------------------------------------------------------------
>
>--- asterisk-1.0-RC2/channels/chan_vpb.c	2004-08-22 21:00:08.000000000 -0300
>+++ asterisk.new/channels/chan_vpb.c	2004-08-29 11:36:18.000000000 -0300
>@@ -8,6 +8,11 @@
>  * Copyright (C) 2004, Ben Kramer
>  * Ben Kramer <ben at voicetronix.com.au>
>  *
>+ * Daniel Bichara <daniel at bichara.com.br> - Brazilian CallerID detection (c)2004
>+ *
>+ * Welber Silveira - welberms at magiclink.com.br - (c)2004
>+ * Copying CLID string to propper structure after detection
>+ *
>  * This program is free software, distributed under the terms of
>  * the GNU General Public License
>  */
>@@ -237,6 +241,8 @@
> 
> 	struct vpb_pvt *next;			/* Next channel in list */
> 
>+	int brcallerpos;				/* Brazilian CallerID detection */
>+
> } *iflist = NULL;
> 
> static struct ast_channel *vpb_new(struct vpb_pvt *i, int state, char *context);
>@@ -486,6 +492,10 @@
> 		ast_verbose(VERBOSE_PREFIX_4 "%s: handle_owned: got event: [%d=>%d]\n",
> 			p->dev, e->type, e->data);
> 
>+	/* Welber Silveira - welberms at magiclink.com.br - (c)2004
>+	 * Copying CLID string to propper structure after detection */
>+	p->owner->callerid = strdup(p->callerid);
>+
> 	f.src = type;
> 	switch (e->type) {
> 		case VPB_RING:
>@@ -749,6 +759,26 @@
> 		case VPB_DTMF:
> 			if (p->state == VPB_STATE_ONHOOK){
> 				/* DTMF's being passed while on-hook maybe Caller ID */
>+
>+				/* Daniel Bichara - daniel at bichara.com.br - (c)2004
>+				 * Brazilian CallerID detection */
>+				if ( p->mode == MODE_FXO ) { /* FXO and DTMF - brazilian callerid */
>+					if ( e->data == 'A' ) { /* CallerID Start signal */
>+						p->brcallerpos = -1;
>+						memset(p->callerid,0,AST_MAX_EXTENSION);
>+					}
>+					else if ( e->data == 'C' ) { /* CallerID End signal */
>+						p->callerid[p->brcallerpos] = '\0';
>+						if (option_verbose > 2)
>+							ast_verbose(VERBOSE_PREFIX_3 " %s: CallerID %s\n",p->dev,p->callerid);
>+
>+					} else if ( p->brcallerpos < AST_MAX_EXTENSION ) {
>+						if ( p->brcallerpos >= 0 )
>+							p->callerid[p->brcallerpos] = e->data;
>+						p->brcallerpos++;
>+					}
>+				}
>+
> 				break;
> 			}
> 			if (p->wantdtmf == 1) {
>@@ -895,11 +925,13 @@
> 			*/
> 		}
> 		/* Two scenarios: Are you owned or not. */
>+
> 		if (p->owner) {
> 			monitor_handle_owned(p, &e);
> 		} else {
> 			monitor_handle_notowned(p, &e);
> 		}
>+
> 		/* if ((!p->owner)&&(p->golock)){
> 			ast_mutex_unlock(&p->owner->lock);
> 			ast_mutex_unlock(&p->lock);
>  
>
>------------------------------------------------------------------------
>
>_______________________________________________
>Asterisk-Dev mailing list
>Asterisk-Dev at lists.digium.com
>http://lists.digium.com/mailman/listinfo/asterisk-dev
>To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>