Merge pull request #51 from AttorneyOnline/fix-showname

Fix shownames being set to a single space when the user does not specify one
AttorneyOnline · Apr 3, 2021 · c6bc249 · c6bc249
2 parents 737f2f1 + 57b3281
commit c6bc249
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/packets.cpp b/src/packets.cpp
@@ -512,7 +512,8 @@ AOPacket AOClient::validateIcPacket(AOPacket packet)
  if (incoming_args.length() > 15) {
  // showname
  QString incoming_showname = dezalgo(incoming_args[15].toString().trimmed());
- if (incoming_showname.length() == 0)
+ // if the raw input is not empty but the trimmed input is, use a single space
+ if (incoming_showname.isEmpty() && !incoming_args[15].toString().isEmpty())
  incoming_showname = " ";
  args.append(incoming_showname);
  showname = incoming_showname;