MessageSend, for Asterisk >= 11

Tried and seems to work: message are sent and received.
Here the "specify":
https://wiki.asterisk.org/wiki/display/AST/Asterisk+11+ManagerAction_MessageSend
I think it need some logic.
Marcelo,
I'm sorry but in the last year I haven't had the time to work on it and I
think I will not have in the near future because I no longer work for that
"VoIP" client of mine.

But I really thank you for your work because it's really useful and high
quality.

Thanks again,
--
Paolo

On Fri, Jun 7, 2013 at 5:18 PM, Marcelo Gornstein <notifications@github.com>
wrote:

> In src/mg/PAMI/Message/Action/MessageSendAction.php:
>
> > + * @subpackage Action
> > + * @author     Marcelo Gornstein <marcelog@gmail.com>
> > + * @license    http://marcelog.github.com/PAMI/ Apache License 2.0
> > + * @link       http://marcelog.github.com/PAMI/
> > + */
> > +class MessageSendAction extends ActionMessage
> > +{
> > +    /**
> > +     * Constructor.
> > +     *
> > +     * @param string $channel Channel to send message to.
> > +     * @param string $message Message to send.
> > +     *
> > +     * @return void
> > +     */
> > +    public function __construct($to, $from, $body, $base64body)
>
> hey :) let's work together on it, i'll be happy to accept a pr once we are
> ok with the feature. would you care to update the code with what we're
> discussing and also to add some unit tests in this suite
> <https://github.com/marcelog/PAMI/blob/master/test/actions/Test_Actions.php>?
> what would be needed is to:
>
>    1. Change constructor prototype, leaving just *$body* (this value will
>    be used in the *Base64Body* key in the action)
>    2. Let's add another argument to the constructor, specifying the
>    technology (subclassing might be too much). The value for this new argument
>    would be a constant inside this same class, something like
>    *MessageSendAction::XMPP_MESSAGE* or *MessageSendAction::SIP_MESSAGE*.
>    This would automagically add "xmpp:" or "sip:" prefixes to the "to"
>    argument.
>    3. Write the unit tests in the suite mentioned above.
>
> What do you think? Does it make sense?
>
> Cheers!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marcelog/PAMI/pull/43/files#r4590295>.
>