From 4dcb73247412125d5e94c18251d30a3f90ce974c Mon Sep 17 00:00:00 2001
From: Paul Vincent Craven <paul@cravenfamily.com>
Date: Wed, 19 Jun 2019 10:03:12 -0500
Subject: [PATCH] Add support for properties element in a tile. Plus, fix a
 couple typos in comments.

---
 pytiled_parser/objects.py    | 13 +++++++++++++
 pytiled_parser/xml_parser.py | 20 ++++++++++++++++----
 2 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/pytiled_parser/objects.py b/pytiled_parser/objects.py
index a805a7b..8031ac4 100644
--- a/pytiled_parser/objects.py
+++ b/pytiled_parser/objects.py
@@ -42,6 +42,18 @@ class OrderedPair(NamedTuple):
     y: Union[int, float]
 
 
+class Property(NamedTuple):
+    """OrderedPair NamedTuple.
+
+    Attributes:
+        name str: Name of property
+        value str: Value of property
+    """
+
+    name: str
+    value: str
+
+
 class Size(NamedTuple):
     """Size NamedTuple.
 
@@ -471,6 +483,7 @@ class Tile:
     animation: Optional[List[Frame]]
     image: Optional[Image]
     hitboxes: Optional[List[TiledObject]]
+    properties: Optional[List[Property]]
 
 
 @dataclasses.dataclass
diff --git a/pytiled_parser/xml_parser.py b/pytiled_parser/xml_parser.py
index 7c01243..77fb731 100644
--- a/pytiled_parser/xml_parser.py
+++ b/pytiled_parser/xml_parser.py
@@ -451,7 +451,7 @@ def _parse_tiles(
             terrain_list_attrib = re.split(",", tile_terrain_attrib)
             # terrain_list is list of indexes of Tileset.terrain_types
             terrain_list: List[Optional[int]] = []
-            # each index in terrain_list_attrib reffers to a corner
+            # each index in terrain_list_attrib refers to a corner
             for corner in terrain_list_attrib:
                 if corner == "":
                     terrain_list.append(None)
@@ -459,6 +459,18 @@ def _parse_tiles(
                     terrain_list.append(int(corner))
             tile_terrain = objects.TileTerrain(*terrain_list)
 
+        # tile element optional sub-elements
+        properties: Optional[List[objects.Property]] = None
+        tile_properties_element = tile_element.find("./properties")
+        if tile_properties_element:
+            properties = []
+            property_list = tile_properties_element.findall("./property")
+            for property in property_list:
+                name = property.attrib["name"]
+                value = property.attrib["value"]
+                obj = objects.Property(name, value)
+                properties.append(obj)
+
         # tile element optional sub-elements
         animation: Optional[List[objects.Frame]] = None
         tile_animation_element = tile_element.find("./animation")
@@ -466,7 +478,7 @@ def _parse_tiles(
             animation = []
             frames = tile_animation_element.findall("./frame")
             for frame in frames:
-                # tileid reffers to the Tile.id of the animation frame
+                # tileid refers to the Tile.id of the animation frame
                 tile_id = int(frame.attrib["tileid"])
                 # duration is in MS. Should perhaps be converted to seconds.
                 # FIXME: make decision
@@ -485,7 +497,7 @@ def _parse_tiles(
             hitboxes = _parse_hitboxes(tile_hitboxes_element)
 
         tiles[id] = objects.Tile(
-            id, tile_type, tile_terrain, animation, tile_image, hitboxes
+            id, tile_type, tile_terrain, animation, tile_image, hitboxes, properties
         )
 
     return tiles
@@ -495,7 +507,7 @@ def _parse_image_element(image_element: etree.Element) -> objects.Image:
     """Parse image element given.
 
     Returns:
-        : Color in Arcade's preffered format.
+        : Color in Arcade's preferred format.
     """
     image = objects.Image(image_element.attrib["source"])