fix issue # 155

Author: AxeemHaider

setup.py

@@ -16,7 +16,7 @@
     # Versions should comply with PEP440.  For a discussion on single-sourcing
     # the version across setup.py and the project code, see
     # https://packaging.python.org/en/latest/single_source_version.html
-    version='1.5.0',
+    version='1.5.1',
 
     description="FireO ORM is specifically designed for the Google's Firestore.",
     long_description=long_description,

src/fireo/queries/filter_query.py

@@ -361,7 +361,15 @@ def order(self, field_name):
         if field_name[0] == '-':
             order_direction = 'Desc'
             name = field_name[1:]  # Get the field name after dash(-) e.g -age name will be age
-        f_name = self.model._meta.get_field(name).db_column_name
+        
+        # ISSUE # 155
+        # If name is for nested field for MapField then there is not need to get field name
+        # from model because there is no such field in model
+        if "." in name:
+            f_name = name
+        else:
+            f_name = self.model._meta.get_field(name).db_column_name
+
         self.order_by.append((f_name, order_direction))
         return self
 

src/tests/v15/test_issue_155.py

@@ -0,0 +1,39 @@
+from fireo.models import Model
+from fireo.fields import TextField, NumberField, IDField
+import fireo
+
+
+def test_issue_155():
+
+    class Test(Model):
+        name = fo.fields.TextField()
+        score_data = fo.fields.MapField(default={
+            'field1': 0,
+            'field2': 0
+        })
+
+
+    test1 = Test(name='test1')
+    test1.save()
+    test1.score_data['field1'] = 100
+    test1.score_data['field2'] = 300
+    test1.upsert()
+
+    test2 = Test(name='test2')
+    test2.save()
+    test2.score_data['field1'] = 200
+    test2.score_data['field2'] = 200
+    test2.upsert()
+
+    test3 = Test(name='test3')
+    test3.save()
+    test3.score_data['field1'] = 300
+    test3.score_data['field2'] = 100
+    test3.upsert()
+
+    try:
+        Test.collection.order('score_data.field2').fetch()
+        assert True == True
+    except Exception as e:
+        assert True == False
+