[Home]

Summary:ASTERISK-16238: [patch] [regression] MusicOnHold don't play if MeetMe Room is stored in mysql db (realtime)
Reporter:Daniel Knoll (kenji)Labels:
Date Opened:2010-06-12 15:22:13Date Closed:2010-08-11 12:28:50
Priority:MinorRegression?Yes
Status:Closed/CompleteComponents:Addons/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20100720__issue17502.diff.txt
( 1) issue_0017502_full_log_case1
( 2) issue_0017502_full_log_case2
Description:If an Meetme Room is stored in mysql database and using realtime, the musiconhold don't play

Testet with an room in mysql and one in the file meetme.conf

****** ADDITIONAL INFORMATION ******

also impact:

if orginate a call that will play a sound into a room (stored in mysql)
and using the Option 'q' for silence entry, that will be ignored. You hear the entry/leave beep sound although the option 'q' is set.
Comments:By: Paul Belanger (pabelanger) 2010-06-12 21:23:49

We require a complete debug log to help triage the issue.

This document will provide instructions on how to collect debugging logs from an Asterisk machine for the purpose of helping bug marshals troubleshoot an issue:

http://svn.digium.com/svn/asterisk/trunk/doc/HOWTO_collect_debug_information.txt
By: Daniel Knoll (kenji) 2010-06-13 03:06:23

i added 2 cases
Case1: using meetme play MusicOnhold with room stored in mysql (room 999)
      it will playing no Musiconhold.
      with room 998 stored in file, it will play MusicOnHold

Case2: beeing in Room and Orginate  a silince Call (over AMI) playing some Audio
      room 999 (mysql) play entry/leave sounds
      room 998 (file) don't play entry/leave sounds (this is good)

Hope that can helps you,
By: Daniel Knoll (kenji) 2010-06-13 03:17:20

in my compiled version

asterisk-1.6.2.6
asterisk-addons-1.6.2.0

all works fine. first i've seen this bug in  

asterisk-1.6.2.7
asterisk-addons-1.6.2.1
By: Daniel Knoll (kenji) 2010-07-08 13:06:04

Found an additional Bug, relating to this one

If stored some Text in column "opts" and and a pin is set, you got the voicepromt "vm-rec-name".
If delete text and redial, you don't get this voicepromt.

I Found this, because I add the actual table structure described in meetme.sql before that, i had a minimal set of important columns.
Sorry for non existing full bug report, just reporting it, if you like, i open a separate Issue.
By: Daniel Knoll (kenji) 2010-07-20 11:15:10

Hi tilghman,
thanx for your patch, but unfortunately the compile breaks at app_meetme.c with

[CC] app_meetme.c -> app_meetme.o
app_meetme.c: In function ‘find_conf_realtime’:
app_meetme.c:3504: warning: passing argument 4 of ‘ast_app_parse_options’ discards qualifiers from pointer target type
app_meetme.c: In function ‘conf_exec’:
app_meetme.c:3811: error: ‘catg’ undeclared (first use in this function)
app_meetme.c:3811: error: (Each undeclared identifier is reported only once
app_meetme.c:3811: error: for each function it appears in.)
app_meetme.c:3823: error: ‘found’ undeclared (first use in this function)
make[1]: *** [app_meetme.o] Error 1
make: *** [apps] Error 2

testet with:
asterisk-1.6.2.7
asterisk-1.6.2.8
asterisk-1.6.2.9
asterisk-1.6.2.10-rc2
By: Tilghman Lesher (tilghman) 2010-07-20 15:01:56

Aha.  Patch fixed.
By: Daniel Knoll (kenji) 2010-07-21 10:35:05

great. all 2 cases i complained works fine.

- moh ist now started again
- quiet entry into conf. is now realy quiet

Thank you for fixing.
By: Digium Subversion (svnbot) 2010-07-21 10:56:04

Repository: asterisk
Revision: 278463

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r278463 | tilghman | 2010-07-21 10:56:03 -0500 (Wed, 21 Jul 2010) | 11 lines

Ensure realtime conferences are treated the same as static conferences when trying to find an empty one.

Also, parse the useropts properly, when retrieving from realtime, and add them
to the existing flags.

(closes issue ASTERISK-16238)
Reported by: kenji
Patches:
      20100720__issue17502.diff.txt uploaded by tilghman (license 14)
Tested by: kenji

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=278463
By: Digium Subversion (svnbot) 2010-07-21 10:58:19

Repository: asterisk
Revision: 278464

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_meetme.c

------------------------------------------------------------------------
r278464 | tilghman | 2010-07-21 10:58:19 -0500 (Wed, 21 Jul 2010) | 18 lines

Merged revisions 278463 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r278463 | tilghman | 2010-07-21 10:56:05 -0500 (Wed, 21 Jul 2010) | 11 lines
 
 Ensure realtime conferences are treated the same as static conferences when trying to find an empty one.
 
 Also, parse the useropts properly, when retrieving from realtime, and add them
 to the existing flags.
 
 (closes issue ASTERISK-16238)
  Reported by: kenji
  Patches:
        20100720__issue17502.diff.txt uploaded by tilghman (license 14)
  Tested by: kenji
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=278464