Skip to content

Commit

Permalink
make envvars cpython compatible
Browse files Browse the repository at this point in the history
first step towards fixing pyrevitlabs#1749
  • Loading branch information
sanzoghenzo authored Jul 6, 2023
1 parent 79d2a76 commit 67c4a65
Showing 1 changed file with 4 additions and 16 deletions.
20 changes: 4 additions & 16 deletions pyrevitlib/pyrevit/coreutils/envvars.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,15 +97,8 @@ def get_pyrevit_env_var(param_name):
# This function returns None if it can not find the parameter.
# Thus value of None should not be used for params

data_dict = AppDomain.CurrentDomain.GetData(ENV_VAR_DICT_NAME)

if data_dict:
try:
return data_dict[param_name]
except KeyError:
return None
else:
return None
data_dict = get_pyrevit_env_vars()
return data_dict.get(param_name) if data_dict else None


def set_pyrevit_env_var(param_name, param_value):
Expand All @@ -117,11 +110,6 @@ def set_pyrevit_env_var(param_name, param_value):
"""
# Get function returns None if it can not find the parameter.
# Thus value of None should not be used for params
data_dict = AppDomain.CurrentDomain.GetData(ENV_VAR_DICT_NAME)

if data_dict:
data_dict[param_name] = param_value
else:
data_dict = {param_name: param_value}

data_dict = get_pyrevit_env_vars() or {}
data_dict[param_name] = param_value
AppDomain.CurrentDomain.SetData(ENV_VAR_DICT_NAME, data_dict)

0 comments on commit 67c4a65

Please sign in to comment.