Core/Chat: fix RBAC_PERM_COMMANDS_NOTIFY_COMMAND_NOT_FOUND_ERROR for subcommands

Source/Framework/Constants/SharedConst.cs

@@ -967,7 +967,6 @@ namespace Framework.Constants
         AhbotUpdateInterval,
         AllTaxiPaths,
         AllowGmGroup,
-        AllowPlayerCommands,
         AllowTrackBothResources,
         AllowTwoSideInteractionAuction,
         AllowTwoSideInteractionCalendar,

Source/Game/Chat/CommandHandler.cs

@@ -107,6 +107,9 @@ namespace Game.Chat
                     string arg = args.NextString("");
                     if (!ExecuteCommandInTable(command.ChildCommands, arg, fullcmd))
                     {
+                        if (_session != null && !_session.HasPermission(RBACPermissions.CommandsNotifyCommandNotFoundError))
+                            return false;
+
                         if (!arg.IsEmpty())
                             SendSysMessage(CypherStrings.NoSubcmd);
                         else

Source/Game/Entities/Unit/Unit.Combat.cs

@@ -2668,10 +2668,10 @@ namespace Game.Entities
             float DoneTotalMod = 1.0f;
 
             // Some spells don't benefit from pct done mods
-            if (spellProto != null)
+            if (spellProto != null && !spellProto.HasAttribute(SpellAttr6.NoDonePctDamageMods))
             {
                 // mods for SPELL_SCHOOL_MASK_NORMAL are already factored in base melee damage calculation
-                if (!spellProto.HasAttribute(SpellAttr6.NoDonePctDamageMods) && !spellProto.GetSchoolMask().HasAnyFlag(SpellSchoolMask.Normal))
+                if (!spellProto.GetSchoolMask().HasAnyFlag(SpellSchoolMask.Normal))
                 {
                     float maxModDamagePercentSchool = 0.0f;
                     for (var i = SpellSchools.Holy; i < SpellSchools.Max; ++i)