Skipping column-less mapped elements

Author: ssweber

examples/journal_internal.py

@@ -75,10 +75,21 @@
 # Requery the data since we made changes to the sort order
 frm['Journal'].requery()
 
-# To edit-protect a Calendar button (or any button), call map_elment()
+# ------------------------------------------
+# How to Edit Protect your sg.CalendarButton
+# ------------------------------------------
+
+# The sg.CalendarButton was not auto-mapped during Form creation.
+# We need to do it manually, using `frm.map_element`.
+# Typically, to manually map an sg.Input, you would pass the sg.Element, DataSet, and column it refers to,
+# but when edit-protecting a Calendar button (or any button), pass None as the column.
+
 frm.map_element(win['datepicker'], frm['Journal'], column=None)
 #               ['Key of Button']                          ^ must be None
-frm.edit_protect() # disable edit-protect to start
+
+# By default, action() includes an edit_protect() call, that disables edits in the window.
+# You can toggle it off with:
+frm.edit_protect() # toggle on/off
 
 # ---------
 # MAIN LOOP
@@ -108,4 +119,5 @@
 - using the label keyword argument to Form.record() to define a custom label
 - using Tables as Form.selector() element types
 - changing the sort order of database dataset
-"""
+- Edit-Protecting a sg.CalendarButton
+"""

pysimplesql/pysimplesql.py

@@ -652,6 +652,10 @@ def records_changed(self, column: str = None, recursive=True) -> bool:
         dirty = False
         # First check the current record to see if it's dirty
         for mapped in self.frm.element_map:
+            # skip mapped items with no column
+            if mapped.column is None:
+                continue
+
             # Compare the DB version to the GUI version
             if mapped.table == self.table:
                 # if passed custom column name
@@ -1220,6 +1224,10 @@ def save_record(self, display_message: bool = True, update_elements: bool = True
 
         # Propagate GUI data back to the stored current_row
         for mapped in self.frm.element_map:
+            # skip mapped items with no column
+            if mapped.column is None:
+                continue
+            
             if mapped.dataset == self:
 
                 # convert the data into the correct data type using the domain in ColumnInfo
@@ -2380,15 +2388,17 @@ def update_elements(self, target_data_key: str = None, edit_protect_only: bool =
         # Render GUI Elements
         # d= dictionary (the element map dictionary)
         for mapped in self.element_map:
+            # if a column-less element is added
+            if mapped.column is None:
+                continue
+
             # If the optional target_data_key parameter was passed, we will only update elements bound to that table
             if target_data_key is not None:
                 if mapped.table != self[target_data_key].table:
                     continue
 
             # skip updating this element if requested
             if mapped.element in omit_elements: continue
-            # if a column-less element is added
-            if mapped.column is None: continue
 
             # Show the Required Record marker if the column has notnull set and this is a virtual row
             marker_key = mapped.element.key + ':marker'