Deprecate legacy binary parser data attributes in message models

Author: clue

src/Model/HeaderBag.php

@@ -4,8 +4,6 @@
 
 class HeaderBag
 {
-    public $data = '';
-
     public $attributes = array(
         'qdCount'   => 0,
         'anCount'   => 0,
@@ -21,6 +19,11 @@ class HeaderBag
         'rcode'     => Message::RCODE_OK,
     );
 
+    /**
+     * @deprecated unused, exists for BC only
+     */
+    public $data = '';
+
     public function get($name)
     {
         return isset($this->attributes[$name]) ? $this->attributes[$name] : null;

src/Model/Message.php

@@ -78,14 +78,20 @@ private static function generateId()
         return mt_rand(0, 0xffff);
     }
 
-    public $data = '';
-
     public $header;
     public $questions = array();
     public $answers = array();
     public $authority = array();
     public $additional = array();
 
+    /**
+     * @deprecated still used internally for BC reasons, should not be used externally.
+     */
+    public $data = '';
+
+    /**
+     * @deprecated still used internally for BC reasons, should not be used externally.
+     */
     public $consumed = 0;
 
     public function __construct()