More bug fixes for kart queue + AI choose standard karts only on the server

Addons were pickable even when explicitly forbidden by filter, because apparently I need to
call kart filter twice anyway. Happily it's quite beautiful, as one of cases is the exact inverse of the order
of another case.

Author: kimden

src/network/protocols/server_lobby.cpp

@@ -8271,7 +8271,7 @@ bool ServerLobby::areKartFiltersIgnoringKarts() const
 //-----------------------------------------------------------------------------
 std::string ServerLobby::getKartForBadKartChoice(STKPeer* peer, const std::string& username, const std::string& check_choice) const
 {
-    std::set<std::string> karts = peer->getClientAssets().first;
+    std::set<std::string> karts = (peer->isAIPeer() ? m_available_kts.first : peer->getClientAssets().first);
     applyAllKartFilters(username, karts, true);
     if (m_kart_elimination.isEliminated(username)
         && karts.count(m_kart_elimination.getKart()))

src/utils/track_filter.cpp

@@ -420,10 +420,6 @@ void KartFilter::apply(FilterContext& context) const
     //     return;
     // }
 
-
-    if (!m_ignore_players_input && !context.applied_at_selection_start)
-        return;
-
     // Ignoring means that the random of allowed and randoms will be picked afterwards
     // Not ignoring means that the player chooses from allowed
     // and one instance of each random
@@ -489,7 +485,8 @@ void KartFilter::apply(FilterContext& context) const
             result.insert(s);
         }
     }
-    else if (m_ignore_players_input && context.applied_at_selection_start)
+    else if ((m_ignore_players_input && context.applied_at_selection_start)
+        || (!m_ignore_players_input && !context.applied_at_selection_start))
     {
         for (const auto& array: m_random_stuff)
         {