Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing issue #160 #164

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixing issue #160 #164

wants to merge 1 commit into from

Conversation

emesika
Copy link
Contributor

@emesika emesika commented Dec 4, 2023

code :

if parse_type[0].startswith("ns"):
cls._cls_suds_map[local_name]["cls"] = globals()[parse_type[1]] cls._cls_suds_map[local_name]["enum_id"] = getattr(cft, "enum_id", None) cls._cls_suds_map[local_name]["is_custom"] = True
cls._cls_suds_map[local_name]["control"] = cls._wi_type

Error :

File "/home/emesika/.local/lib/python3.9/site-packages/pylero/work_item.py", line 1464, in get_custom_fields print(f"Global = {globals()[parse_type[1]]}")
KeyError: 'duration'

Happens on :

local name = sprint_estimate
parse_type = duration

should skip non existing parse_type[1] in globals() :

if parse_type[0].startswith("ns"):
if parse_type[1] not in globals():
continue
cls._cls_suds_map[local_name]["cls"] = globals()[parse_type[1]] cls._cls_suds_map[local_name]["enum_id"] = getattr(cft, "enum_id", None) cls._cls_suds_map[local_name]["is_custom"] = True
cls._cls_suds_map[local_name]["control"] = cls._wi_type

code :

if parse_type[0].startswith("ns"):
cls._cls_suds_map[local_name]["cls"] = globals()[parse_type[1]]
cls._cls_suds_map[local_name]["enum_id"] = getattr(cft, "enum_id", None)
cls._cls_suds_map[local_name]["is_custom"] = True
cls._cls_suds_map[local_name]["control"] = cls._wi_type

Error :

File "/home/emesika/.local/lib/python3.9/site-packages/pylero/work_item.py", line 1464, in get_custom_fields
print(f"Global = {globals()[parse_type[1]]}")
KeyError: 'duration'

Happens on :

local name = sprint_estimate
parse_type = duration

should skip non existing parse_type[1] in globals() :

if parse_type[0].startswith("ns"):
if parse_type[1] not in globals():
continue
cls._cls_suds_map[local_name]["cls"] = globals()[parse_type[1]]
cls._cls_suds_map[local_name]["enum_id"] = getattr(cft, "enum_id", None)
cls._cls_suds_map[local_name]["is_custom"] = True
cls._cls_suds_map[local_name]["control"] = cls._wi_type

Signed-off-by: Eli Mesika <emesika@redhat.com>
@leelavg
Copy link
Collaborator

leelavg commented Feb 16, 2024

Unfortunately the fix seems like a workaround, maybe the original assumption that all nsX types are object but not native polarion types is blurred in recent releases and duration is the first case we hit due to that 🤔

@leelavg
Copy link
Collaborator

leelavg commented Feb 16, 2024

to the projects that I've access I don't see a custom field with duration as ns object, if possible could you pls ping me internally and reach a conclusion. thanks.

@waynesun09
Copy link
Collaborator

@emesika any info about what Leela asked in the comment?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants