1. onSipReply has changed recently - now it has two more parameters

2. you can do all this in DSM, much easier (see request event DSM examples
and quizconnect tutorial). the only thing you need to add is calendar
functions, and - if you dont want to use mod_ conference to connect two
legs) audio connector.

stefan

2010/9/28 Raphael Coeffic <[email protected]>

>  On 28.09.10 14:59, Антон Загорский wrote:
>
>> Hello Raphael.
>>
>> Please take a look on an attached archive. Now I'm working on 1.2.1
>>
>>
>>  In fact, deriving directly from AmSession should be just fine.
>>>
>> But this way I'll lose AmB2ABSessionCaller class benefits?..
>> Also, I've looked on AmB2ABSessionCaller sources and was surprised that
>> onB2ABEvent lost 'virtual' qualification, but in a parent class
>> AmB2ABSession method is virtual. Why?
>>
>>
>>
> Just as you whish! I thought you were asking me whether it would be ok to
> derive directly from AmSession, which is perfectly ok. Deriving from
> AmB2ABCallerSession is also perfectly ok.
>
> Regarding the reason why onB2BABEvent has no "virtual" keyword: it does not
> matter, "once virtual, always virtual".
> Here are some thoughts from other people on the topic:
>
> http://stackoverflow.com/questions/2963965/why-is-virtual-optional-for-overridden-methods-in-derived-classes
>
> Regarding the onSipReply() and onSipEvent() method, you will notice that
> those functions are defined in the AmSession class, and not in its factory.
> This means that you should move those method from the factory to the session
> class itself (CallCenterCallerSession in your case).
>
> Cheers
> Raphael.
>
> PS: please remember to CC the mailing list as well, so that other people
> can profit from the discussion.
>
> _______________________________________________
> Semsdev mailing list
> [email protected]
> http://lists.iptel.org/mailman/listinfo/semsdev
>
_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev

Reply via email to