Tool Validator - Filter List won't update

Anonymous User

Original User: sirpinkee

I am trying to make my geoprocessing script update DEVICE and ADDRESS field based on the selection of a LOCATION, so that when a user selects a location (MHA or City), it lists the devices and addresses common to those locations. So far so good, have been able to list either but not both. For example: if a user selects location(City), the device list and address list (which are fields both common to the location layer) are displayed. On the other hand, if the user then decides to change the selection to location(MHA), it doesn't update the list, instead I still have values from the previous list (City). Here is my code in the ToolValidator:

 def initializeParameters(self):
    """Refine the properties of a tool's parameters.  This method is
    called when the tool is opened."""
    import arcpy

    address = []
    device  = []
    rows = arcpy.SearchCursor(r"Database Connections\Test_GDB_SQL2005.sde\Test_GDB.GISADMIN.MHAedit")
    for row in rows:
      if row.Device_Typ not in device:
        device.append(row.Device_Typ)
      if row.Site_Addre not in address:
        address.append(row.Site_Addre)

    del row, rows

    address = sorted(address)
    device  = sorted(device)

    address1 = []
    device1  = []
    rows = arcpy.SearchCursor(r"Database Connections\Test_GDB_SQL2005.sde\Test_GDB.GISADMIN.Citydbf2")
    for row in rows:
      if row.Device_Typ not in device1:
        device1.append(row.Device_Typ)
      if row.Site_Addre not in address1:
        address1.append(row.Site_Addre)

    del row, rows

    address1 = sorted(address1)
    device1  = sorted(device1)
    
    self.params[0].filter.list = ["City", "MHA"]
    if self.params[0].filter.list == "City":
      self.params[1].filter.list = device1
      self.params[2].filter.list = address1
    else:
         self.params[1].filter.list = device
         self.params[2].filter.list = address
      
    if self.params[0].filter.list == "MHA":
      self.params[1].filter.list = device
      self.params[2].filter.list = address
    else:
      self.params[1].filter.list = device1
      self.params[2].filter.list = address1
    return

N.B - The first screenshot which has the "City" selected in the Location field is correct but the second screenshot which has the "MHA" selected in the Location field still displays the address list for the "City" whereas the address list that should have been displayed is actually lesser than what is currently being displayed.

[ATTACH=CONFIG]16940[/ATTACH][ATTACH=CONFIG]16941[/ATTACH]

MHA.jpg

city.jpg

DominicReed_0-1719803863216.png

WengNg4_0-1719802369847.png

RichardFairhurst

Try moving the parameter[0] value testing part of your code from the initializeParameters section to the updateParameters section.  The behavior you are experiencing is consistent with the descriptions of the behavior of the initializeParameters section.  It only looks at the input parameter when the tool first opens and then ignores anything that happens afterward.  The updateParameters section is intended to be triggered any time a parameter value changes.  To keep track of changes in a particular parameter (so that changes in other parameters are ignored), create a self. variable in the __init__ section. 

For example, assume that there are two date fields in the tool that define a date range.  You want the tool to open with an initial pair of dates that are set 3 years apart relative to the first day of the month of today's date whenever the tool is run.  You also want the tool to detect whenever the user changes either of the dates, but only if he has kept both dates assigned.  The tool will run the change code when the initial values are assigned and then anytime the user changes one of the dates without blanking it out:

from datetime import date, timedelta

class ToolValidator(object):
  """Class for validating a tool's parameter values and controlling
  the behavior of the tool's dialog."""
  def __init__(self):
    """Setup arcpy and the list of tool parameters."""
    self.params = arcpy.GetParameterInfo()
    self.collisiondates = (None, None) # a date pair defining a range.

  def initializeParameters(self):
    """Refine the properties of a tool's parameters.  This method is
    called when the tool is opened."""
    # Set an initial date range relative to the day the user starts the tool
    today = date.today()
    from_date = str(date(today.year - 3, today.month, 1)) # 3 years back from the 1st of today's month
    self.params[0].value = from_date
    to_date = str(date(today.year, today.month, 1)-timedelta(days=1)) # the ending of the month before today's month
    self.params[1].value = to_date

  def updateParameters(self):
    """Modify the values and properties of parameters before internal
    validation is performed.  This method is called whenever a parameter
    has been changed."""
 
    frdate = str(self.params[0].value)
    tdate = str(self.params[1].value)
    if self.collisiondates != (frdate, tdate) and frdate and tdate:
      self.collisiondates = (frdate, tdate)
      # code at this indent level only activates when both dates are assigned and the user changed one of the dates
    else:
      # code at this indent level only activates when one or both the dates of the range are unassigned or they are both unchanged

  def updateMessages(self):
    """Modify the messages created by internal validation for each tool
    parameter.  This method is called after internal validation."""
    return

Applying this to your code I would say you want 4 self. variables in the __init__ section to store your lists and make them accessible to the other methods.
self.address = []
self.device = []
self.address1 = []
self.device1 = []

You should assign to these variables (add the self. prefix) and perform the SearchCursors in the initializeParameters section as you have done (assuming they are static once read and never need to be updated while the tool is being used).  Then you should move the actual assignment of the lists to the updateParameters section (you don't need to test for both "City" and "MHA", just one).

    self.params[0].filter.list = ["City", "MHA"]
    if self.params[0].value == "City":
      self.params[1].filter.list = self.device1
      self.params[2].filter.list = self.address1
    else:
      self.params[1].filter.list = self.device
      self.params[2].filter.list = self.address