Cleanup

beer-garden · Dec 13, 2023 · 68b684b · 68b684b
1 parent 96c0214
commit 68b684b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -6,7 +6,7 @@ Brewtils Changelog
 TBD
 
 - Added new KWARG input to @command for tag/tags. This can be utilized for filtering commands.
-- Adding defaul topic for PublishClient to Plugins {Namespace}.{System}.{Version}.{Instance}
+- Adding default topic for PublishClient to Plugins {Namespace}.{System}.{Version}.{Instance}
 - Removed Python 12 support until we upgrade Marshmallow dependency to 3.15 or greater
 
 3.21.0

diff --git a/brewtils/rest/publish_client.py b/brewtils/rest/publish_client.py
@@ -78,7 +78,7 @@ def publish(
 
         Args:
             _topic (str): The topic to publish to, default is Plugin level topic
-                (Namespace.System.Version.Instance)
+                {Namespace}.{System}.{Version}.{Instance}
             _regex_only (bool): If the request will be resolved against only annotated topics
             from the @subscribe command
             _propagate (bool): If the request will be pushed up to the parent to be resolved.
@@ -92,9 +92,10 @@ def publish(
                 brewtils.plugin.CONFIG.name
                 and brewtils.plugin.CONFIG.version
                 and brewtils.plugin.CONFIG.instance_name
+                and brewtils.plugin.CONFIG.namespace
             ):
                 _topic = "{0}.{1}.{2}.{3}".format(
-                    brewtils.plugin.CONFIG.namespace or "",
+                    brewtils.plugin.CONFIG.namespace,
                     brewtils.plugin.CONFIG.name,
                     brewtils.plugin.CONFIG.version,
                     brewtils.plugin.CONFIG.instance_name,