From: Mikael Nordfeldth <mmn@hethane.se>
Date: Sun, 4 Oct 2015 07:59:01 +0000 (+0200)
Subject: Send the entire XMLStringer object in MagicEnvelope events.
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=9b461db4da665e2c0d5cfdeda53b56aaef781445;p=quix0rs-gnu-social.git

Send the entire XMLStringer object in MagicEnvelope events.
---

diff --git a/plugins/OStatus/lib/magicenvelope.php b/plugins/OStatus/lib/magicenvelope.php
index 2c54e3679b..e6b068c924 100644
--- a/plugins/OStatus/lib/magicenvelope.php
+++ b/plugins/OStatus/lib/magicenvelope.php
@@ -206,11 +206,11 @@ class MagicEnvelope
      * @return string representation of XML document
      */
     public function toXML($flavour=null) {
-        $xml = null;
-        if (Event::handle('MagicEnvelopeToXML', array($this, $flavour, &$xml))) {
+        $xs = new XMLStringer();
+        $xs->startXML();    // header, to point out it's not HTML or anything...
+        if (Event::handle('StartMagicEnvelopeToXML', array($this, $xs, $flavour))) {
             // fall back to our default, normal Magic Envelope XML.
-            $xs = new XMLStringer();
-            $xs->startXML();
+            // the $xs element _may_ have had elements added, or could get in the end event
             $xs->elementStart('me:env', array('xmlns:me' => self::NS));
             $xs->element('me:data', array('type' => $this->data_type), $this->data);
             $xs->element('me:encoding', null, $this->encoding);
@@ -218,12 +218,9 @@ class MagicEnvelope
             $xs->element('me:sig', null, $this->getSignature());
             $xs->elementEnd('me:env');
 
-            $xml = $xs->getString();
+            Event::handle('EndMagicEnvelopeToXML', array($this, $xs, $flavour));
         }
-        if (is_null($xml)) {
-            throw new ServerException('No Magic Envelope XML string was created.');
-        }
-        return $xml;
+        return $xs->getString();
     }
 
     /*