[pylint.message] Symbol seems to never be None anymore

1 files changed, 5 insertions, 12 deletions

diff --git a/pylint/message/message_definition.py b/pylint/message/message_definition.py
index 2d8cd2997..cd33cd575 100644
--- a/pylint/message/message_definition.py
+++ b/pylint/message/message_definition.py
@@ -29,12 +29,9 @@ class MessageDefinition:
         if not msgid[0] in MSG_TYPES:
             raise InvalidMessageError("Bad message type %s in %r" % (msgid[0], msgid))
         self.msgid = msgid
+        self.symbol = symbol
         self.msg = msg
         self.description = description
-        if not symbol:
-            # backward compatibility, message may not have a symbol
-            symbol = msgid
-        self.symbol = symbol
         self.scope = scope
         self.minversion = minversion
         self.maxversion = maxversion
@@ -60,10 +57,6 @@ class MessageDefinition:
         if checkerref:
             desc += " This message belongs to the %s checker." % self.checker.name
         title = self.msg
-        if self.symbol:
-            msgid = "%s (%s)" % (self.symbol, self.msgid)
-        else:
-            msgid = self.msgid
         if self.minversion or self.maxversion:
             restr = []
             if self.minversion:
@@ -75,9 +68,9 @@ class MessageDefinition:
                 desc += " It can't be emitted when using Python %s." % restr
             else:
                 desc += " This message can't be emitted when using Python %s." % restr
-        desc = normalize_text(" ".join(desc.split()), indent="  ")
+        msg_help = normalize_text(" ".join(desc.split()), indent="  ")
+        message_id = "%s (%s)" % (self.symbol, self.msgid)
         if title != "%s":
             title = title.splitlines()[0]
-
-            return ":%s: *%s*\n%s" % (msgid, title.rstrip(" "), desc)
-        return ":%s:\n%s" % (msgid, desc)
+            return ":%s: *%s*\n%s" % (message_id, title.rstrip(" "), msg_help)
+        return ":%s:\n%s" % (message_id, msg_help)