quillsql 2.2.1__py3-none-any.whl → 2.2.3__py3-none-any.whl

quillsql/core.py

@@ -113,8 +113,9 @@ class Quill:
                     **metadata,
                     "tenants": tenants,
                     "flags": flags,
-                    "adminEnabled": admin_enabled,
                 }
+                if admin_enabled is not None:
+                    pivot_payload["adminEnabled"] = admin_enabled
                 if filters is not None:
                     pivot_payload["sdkFilters"] = [
                         convert_custom_filter(f) for f in filters
@@ -212,17 +213,16 @@ class Quill:
                 tenants[0] != SINGLE_TENANT
             ):
                 
-                response = self.post_quill(
-                    'tenant-mapped-flags',
-                    {
-                        'reportId': metadata.get('reportId') or metadata.get('dashboardItemId'),
-                        'clientId': metadata.get('clientId'),
-                        'dashboardName': metadata.get('name'),
-                        'tenants': tenants,
-                        'flags': flags,
-                        'adminEnabled': admin_enabled,
-                    },
-                )
+                tenant_flags_payload = {
+                    'reportId': metadata.get('reportId') or metadata.get('dashboardItemId'),
+                    'clientId': metadata.get('clientId'),
+                    'dashboardName': metadata.get('name'),
+                    'tenants': tenants,
+                    'flags': flags,
+                }
+                if admin_enabled is not None:
+                    tenant_flags_payload['adminEnabled'] = admin_enabled
+                response = self.post_quill('tenant-mapped-flags', tenant_flags_payload)
                 
                 if response.get('error'):
                     return {
@@ -236,16 +236,22 @@ class Quill:
                     self.target_connection.database_type,
                 )
                 
-                tenant_flags = [
-                    {
-                        'tenantField': tenant_field,
-                        'flags': list(set(row['quill_flag'] for row in query_result['rows']))
+                tenant_flags = []
+                query_order = response.get('metadata', {}).get('queryOrder') or []
+                query_results = (flag_query_results or {}).get('queryResults') or []
+                for tenant_field, query_result in zip(query_order, query_results):
+                    rows = []
+                    if isinstance(query_result, dict):
+                        rows = query_result.get('rows') or []
+                    flags = {
+                        row.get('quill_flag')
+                        for row in rows
+                        if isinstance(row, dict) and row.get('quill_flag') is not None
                     }
-                    for tenant_field, query_result in zip(
-                        response['metadata']['queryOrder'],
-                        flag_query_results['queryResults']
-                    )
-                ]
+                    tenant_flags.append({
+                        'tenantField': tenant_field,
+                        'flags': list(flags),
+                    })
             elif tenants[0] == SINGLE_TENANT and flags:
                 if flags and isinstance(flags[0], dict):
                     tenant_flags = [{'tenantField': SINGLE_TENANT, 'flags': flags}]
@@ -277,14 +283,15 @@ class Quill:
                 and metadata.get("runQueryConfig").get("getColumns")
                 else None
             )
-            payload = {
+            payload: dict = {
                 **metadata,
                 "tenants": tenants,
                 "flags": tenant_flags,
                 "viewQuery": view_query,
                 "preQueryResultsColumns": pre_query_columns,
-                "adminEnabled": admin_enabled,
             }
+            if admin_enabled is not None:
+                payload["adminEnabled"] = admin_enabled
             if filters is not None:
                 payload["sdkFilters"] = [convert_custom_filter(f) for f in filters]
             quill_results = self.post_quill(metadata.get("task"), payload)
@@ -329,17 +336,18 @@ class Quill:
                     normalized_results.get("queryResults") or []
                 )
 
-            if (
-                normalized_results.get("mapped_array")
-                and metadata.get("runQueryConfig", {}).get("arrayToMap")
-            ):
-                array_to_map = metadata["runQueryConfig"]["arrayToMap"]
-                for array, index in zip(
-                    normalized_results["mapped_array"],
-                    range(len(normalized_results["mapped_array"])),
-                ):
-                    responseMetadata[array_to_map["arrayName"]][index][array_to_map["field"]] = array
-                del normalized_results["mapped_array"]
+            run_query_config = responseMetadata.get("runQueryConfig") or {}
+            array_to_map = run_query_config.get("arrayToMap")
+            if normalized_results.get("mapped_array") and array_to_map:
+                target_collection = responseMetadata.get(array_to_map.get("arrayName"))
+                if isinstance(target_collection, list):
+                    for index, mapped_rows in enumerate(normalized_results["mapped_array"]):
+                        if index >= len(target_collection):
+                            continue
+                        target_entry = target_collection[index]
+                        if isinstance(target_entry, dict):
+                            target_entry[array_to_map.get("field")] = mapped_rows
+                normalized_results.pop("mapped_array", None)
 
             query_results_list = normalized_results.get("queryResults") or []
             if len(query_results_list) == 1:
@@ -354,15 +362,14 @@ class Quill:
 
         except Exception as err:
             if task == "update-view":
-                self.post_quill(
-                    "set-broken-view",
-                    {
-                        "table": metadata.get("name"),
-                        "clientId": metadata.get("clientId"),
-                        "error": str(err),
-                        "adminEnabled": admin_enabled,
-                    },
-                )
+                broken_view_payload = {
+                    "table": metadata.get("name"),
+                    "clientId": metadata.get("clientId"),
+                    "error": str(err),
+                }
+                if admin_enabled is not None:
+                    broken_view_payload["adminEnabled"] = admin_enabled
+                self.post_quill("set-broken-view", broken_view_payload)
             return {
                 "error": str(err).splitlines()[0],
                 "status": "error",

{quillsql-2.2.1.dist-info → quillsql-2.2.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quillsql
-Version: 2.2.1
+Version: 2.2.3
 Summary: Quill SDK for Python.
 Home-page: https://github.com/quill-sql/quill-python
 Author: Quill

{quillsql-2.2.1.dist-info → quillsql-2.2.3.dist-info}/RECORD

@@ -1,5 +1,5 @@
 quillsql/__init__.py,sha256=FiuoxaNZveKXOPB0hkpfGNlpZKmSn3pRwcqm9HKYbCQ,180
-quillsql/core.py,sha256=cWWu_ME3kapzv1fBhoJLmkA7QIlbVCAOMHyce_VE3BU,22464
+quillsql/core.py,sha256=gUyM_To8Gdw30KFLqFLnvXykxCFoC7NE-A61cToEdXQ,23285
 quillsql/error.py,sha256=n9VKHw4FAgg7ZEAz2YQ8L_8FdRG_1shwGngf2iWhUSM,175
 quillsql/assets/__init__.py,sha256=oXQ2ZS5XDXkXTYjADxNfGt55cIn_rqfgWL2EDqjTyoI,45
 quillsql/assets/pgtypes.py,sha256=-B_2wUaoAsdX7_HnJhUlx4ptZQ6x-cXwuST9ACgGFdE,33820
@@ -14,7 +14,7 @@ quillsql/utils/pivot_template.py,sha256=NzHO7Ux8GVAKY-DUtsOmE7TUls2q6FJG2kgJxVWq
 quillsql/utils/run_query_processes.py,sha256=FRmNvjTDLUBr7MqDKQmivdC0anwybMXUyzQbKnaZx70,698
 quillsql/utils/schema_conversion.py,sha256=TFfMibN9nOsxNRhHw5YIFl3jGTvipG81bxX4LFDulUY,314
 quillsql/utils/tenants.py,sha256=ZD2FuKz0gjBVSsThHDv1P8PU6EL8E009NWihE5hAH-Q,2022
-quillsql-2.2.1.dist-info/METADATA,sha256=AImLKL4nUys0Yz6i_12iPMHLxi4q5KHnxZJWRoIyw7U,1786
-quillsql-2.2.1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-quillsql-2.2.1.dist-info/top_level.txt,sha256=eU2vHnVqwpYQJ3ADl1Q-DIBzbYejZRUhcMdN_4zMCz8,9
-quillsql-2.2.1.dist-info/RECORD,,
+quillsql-2.2.3.dist-info/METADATA,sha256=m_GfCogVSbSTxw0MGKq-scyOyeipyRbcoU40BHE0yq4,1786
+quillsql-2.2.3.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+quillsql-2.2.3.dist-info/top_level.txt,sha256=eU2vHnVqwpYQJ3ADl1Q-DIBzbYejZRUhcMdN_4zMCz8,9
+quillsql-2.2.3.dist-info/RECORD,,