Merge pull request #81 from passren/0.8.1

0.8.1 - fixed something

Author: passren

.github/workflows/run-test.yaml

@@ -40,6 +40,7 @@ jobs:
           pip install pyparsing
           pip install sqlean.py
       - name: black
+        if: matrix.python-version != '3.9'
         run: |
           pip install black
           black --check pydynamodb

pydynamodb/connection.py

@@ -10,7 +10,6 @@
 from .error import NotSupportedError
 from .util import RetryConfig, retry_api_call
 
-
 if TYPE_CHECKING:
     from botocore.client import BaseClient
 

pydynamodb/sql/ddl_alter.py

@@ -109,6 +109,7 @@
 SSESpecification.KMSMasterKeyId $$$$$$$$
 TableClass STANDARD_INFREQUENT_ACCESS
 """
+
 import logging
 from .ddl_sql import DdlBase
 from .common import KeyWords, Tokens

pydynamodb/sql/ddl_drop.py

@@ -11,6 +11,7 @@
 DROP GLOBAL TABLE Issues
     ReplicationGroup (us-east-1, us-west-2)
 """
+
 import logging
 from .ddl_sql import DdlBase
 from .common import KeyWords, Tokens

pydynamodb/sql/dml_delete.py

@@ -13,6 +13,7 @@
 
 DELETE FROM "Music" WHERE "Artist" = 'Acme Band' AND "SongTitle" = 'PartiQL Rocks' RETURNING ALL OLD *
 """
+
 import logging
 from .dml_sql import DmlBase
 from .common import KeyWords, Tokens

pydynamodb/sql/dml_insert.py

@@ -9,6 +9,7 @@
 ------------------------
 INSERT INTO "Music" value {'Artist' : 'Acme Band','SongTitle' : 'PartiQL Rocks'}
 """
+
 import logging
 from .dml_sql import DmlBase
 from .common import KeyWords, Tokens

pydynamodb/sql/dml_select.py

@@ -31,6 +31,7 @@
 ConsistentRead False
 ReturnConsumedCapacity NONE
 """
+
 import logging
 import re
 from abc import ABCMeta

pydynamodb/sql/dml_sql.py

@@ -135,14 +135,14 @@ def __escape_column(self, token: Any) -> str:
 
     def __init__(self, statement: str) -> None:
         super().__init__(statement)
-        self._where_conditions = []
+        self._parsed_where_conditions = []
         self._limit = None
         self._consistent_read = False
         self._return_consumed_capacity = "NONE"
 
     @property
     def where_conditions(self) -> List[Optional[str]]:
-        return self._where_conditions
+        return self._parsed_where_conditions
 
     @property
     def limit(self) -> int:
@@ -166,18 +166,18 @@ def transform(self) -> Dict[str, Any]:
     def _construct_where_conditions(self, where_conditions: List[Any]) -> str:
         for condition in where_conditions:
             if not isinstance(condition, ParseResults):
-                self._where_conditions.append(str(condition))
+                self._parsed_where_conditions.append(str(condition))
             else:
                 function_ = condition.get("function", None)
                 function_with_op_ = condition.get("function_with_op", None)
                 if function_:
                     flatted_func_params = ",".join(condition["function_params"])
-                    self._where_conditions.append(
+                    self._parsed_where_conditions.append(
                         "%s(%s)" % (function_, flatted_func_params)
                     )
                 elif function_with_op_:
                     flatted_func_params = ",".join(condition["function_params"])
-                    self._where_conditions.append(
+                    self._parsed_where_conditions.append(
                         "%s(%s) %s %s"
                         % (
                             function_with_op_,
@@ -191,7 +191,7 @@ def _construct_where_conditions(self, where_conditions: List[Any]) -> str:
                     flatted_where = " ".join(
                         str(c) for c in flatten_list(where_conditions_)
                     )
-                    self._where_conditions.append(flatted_where)
+                    self._parsed_where_conditions.append(flatted_where)
 
         return "WHERE %s" % " ".join(self.where_conditions)
 

pydynamodb/sql/dml_update.py

@@ -20,6 +20,7 @@
 REMOVE AwardDetail.Grammys[2]
 WHERE Artist='Acme Band' AND SongTitle='PartiQL Rocks'
 """
+
 import logging
 from .dml_sql import DmlBase
 from .json_parser import jsonArray, jsonObject

pydynamodb/sql/json_parser.py

@@ -9,31 +9,6 @@
 #   members optional in array and object collections
 #
 # Updated 9 Aug 2016 - use more current pyparsing constructs/idioms
-#
-'''
-json_bnf = """
-object
-    { members }
-    {}
-members
-    string : value
-    members , string : value
-array
-    [ elements ]
-    []
-elements
-    value
-    elements , value
-value
-    string
-    number
-    object
-    array
-    true
-    false
-    null
-"""
-'''
 
 import pyparsing as pp
 from pyparsing import pyparsing_common as ppc
@@ -61,11 +36,6 @@ def make_keyword(kwd_str, kwd_value):
 jsonValue = pp.Forward().set_name("jsonValue")
 
 jsonElements = pp.DelimitedList(jsonValue).set_name(None)
-# jsonArray = pp.Group(LBRACK + pp.Optional(jsonElements, []) + RBRACK)
-# jsonValue << (
-#     jsonString | jsonNumber | pp.Group(jsonObject) | jsonArray | TRUE | FALSE | NULL
-# )
-# memberDef = pp.Group(jsonString + COLON + jsonValue).set_name("jsonMember")
 
 jsonArray = pp.Group(
     LBRACK + pp.Optional(jsonElements) + RBRACK, aslist=RETURN_PYTHON_COLLECTIONS