Skip to content

Commit daf21c6

Browse files
authored
Merge pull request #49 from jmkerloch/fix-plugin-id-qdt-profile
fix(qdt profile): plugin_id should be an int
2 parents b4e4ff5 + 8529f53 commit daf21c6

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

profile_manager/profiles/utils.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ class PluginInformation:
133133
name: str
134134
folder_name: str
135135
official_repository: bool
136-
plugin_id: str
136+
plugin_id: Optional[int]
137137
version: str
138138

139139

@@ -193,7 +193,11 @@ def get_profile_plugin_information(
193193
name=manager_metadata["name"],
194194
folder_name=plugin_slug_name,
195195
official_repository=True, # For now we only support official repository
196-
plugin_id=manager_metadata["plugin_id"],
196+
plugin_id=(
197+
int(manager_metadata["plugin_id"])
198+
if manager_metadata["plugin_id"]
199+
else None
200+
),
197201
version=define_plugin_version_from_metadata(
198202
manager_metadata=manager_metadata,
199203
plugin_metadata=plugin_metadata,
@@ -221,7 +225,7 @@ def get_profile_plugin_list_information(
221225

222226
for plugin_name in plugin_list:
223227
plugin_info = get_profile_plugin_information(profile_name, plugin_name)
224-
if plugin_info and plugin_info.plugin_id != "":
228+
if plugin_info and plugin_info.plugin_id:
225229
profile_plugin_list.append(plugin_info)
226230

227231
return profile_plugin_list

0 commit comments

Comments
 (0)