On 28.09.10 16:18, Stefan Sayer wrote:
1. onSipReply has changed recently - now it has two more parameters
Yep, but we were talking about 1.2.1 ;-)
-Raphael.
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] <mailto:[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] <mailto:[email protected]>
http://lists.iptel.org/mailman/listinfo/semsdev
_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev
