Merge pull request #229 from Deutscher-Tischfussballbund/sportsmanager-issue225

administrators can't see player profiles of inactive players

src/structure/components/com_sportsmanager/admin.php

@@ -959,6 +959,7 @@ function adminSpieler($admin_uebersicht): void
     else if ($beschraenkter_zugriff == 2)
         $query .= "\n INNER JOIN (SELECT spieler_id FROM #__sportsmanager_mitglied_von_verein INNER JOIN #__sportsmanager_verein USING (verein_id) WHERE NOT #__sportsmanager_mitglied_von_verein.ausgetreten AND NOT #__sportsmanager_verein.ausgetreten AND mitgliedsstatus = 1 AND" . vereinFilter("#__sportsmanager_verein.verein_id IN") . "GROUP BY spieler_id ORDER BY spieler_id, verein_id) AS mitgliedschaft USING (spieler_id)";
     $anzahlSpieler["aktiv"] = loadResult($db, $query);
+
     $jahr = date("Y");
     $anzahlSpieler["J aktiv"] = loadResult($db, $query . "\n WHERE " . ($jahr - 18) . " <= geburtsjahr");
     $anzahlSpieler["S aktiv"] = loadResult($db, $query . "\n WHERE " . ($jahr - 50) . " > geburtsjahr");
@@ -985,6 +986,9 @@ function adminSpieler($admin_uebersicht): void
         $query .= "\n INNER JOIN (SELECT spieler_id FROM #__sportsmanager_mitglied_von_verein INNER JOIN #__sportsmanager_verein USING (verein_id) WHERE NOT #__sportsmanager_mitglied_von_verein.ausgetreten AND NOT #__sportsmanager_verein.ausgetreten AND mitgliedsstatus = 3 AND" . vereinFilter("#__sportsmanager_verein.verein_id IN") . "GROUP BY spieler_id ORDER BY spieler_id, verein_id) AS mitgliedschaft USING (spieler_id)";
     $anzahlSpieler["passiv"] = loadResult($db, $query);
 
+	$query = "SELECT COUNT(DISTINCT spieler_id) FROM #__sportsmanager_spieler WHERE aktueller_verein_id IS NULL;";
+	$anzahlSpieler["vereinslos"] = loadResult($db, $query) - $anzahlSpieler["passiv"];
+
     $search = stripslashes($filter);
     if ($beschraenkter_zugriff == 0 && $search == "_____DOPPELT") {
         $jahr = date("Y");

src/structure/components/com_sportsmanager/sportsmanager.php

@@ -3228,7 +3228,7 @@ function spielerDetails(): void
     $statistik["elo_doppel"] = loadResult($db, $query) && $elo_anzeigen;
 
     if (isJson()) {
-        JSON_sportsmanager::    spielerDetails($spieler, $vereine, $veranstalter, $spieler_elo_verlauf_einzel, $spieler_elo_verlauf_doppel, $spielerNamen, $teamNamen, $veranstaltungBezeichnungen, $turnierdisziplinBezeichnungen, $individualwettbewerbBezeichnungen, $ranglistenplatzierungen, $turnierplatzierungen, $teams, $elo_detailliert, $statistik, $einstufungen);
+        JSON_sportsmanager::spielerDetails($spieler, $vereine, $veranstalter, $spieler_elo_verlauf_einzel, $spieler_elo_verlauf_doppel, $spielerNamen, $teamNamen, $veranstaltungBezeichnungen, $turnierdisziplinBezeichnungen, $individualwettbewerbBezeichnungen, $ranglistenplatzierungen, $turnierplatzierungen, $teams, $elo_detailliert, $statistik, $einstufungen);
     } else {
         HTML_sportsmanager::spielerDetails($spieler, $vereine, $veranstalter, $spieler_elo_verlauf_einzel, $spieler_elo_verlauf_doppel, $spielerNamen, $teamNamen, $veranstaltungBezeichnungen, $turnierdisziplinBezeichnungen, $individualwettbewerbBezeichnungen, $ranglistenplatzierungen, $turnierplatzierungen, $teams, $sortierung, $vorheriger_spieler_id, $naechster_spieler_id, $elo_detailliert, $statistik, $beginn, $kategorie, $einstufungen, $filter, $veranstaltungid, $veranstalterid, $einstufungid, $unabhaengige_ansicht, $details_anzeigen, $lizenz_anzeigen);
     }

src/structure/components/com_sportsmanager/views/sportsmanager/view.html.php

@@ -4284,7 +4284,11 @@ static function spielerDetails($spieler, $vereine, $veranstalter, $spieler_elo_v
 								$einstufungid, $unabhaengige_ansicht, $details_anzeigen, $lizenz_anzeigen):void {
 global $params;
 
-if (count($vereine) == 0){echo "Sie haben keine Berechtigung auf diesen Datensatz!!!"; die;}
+if ((count($vereine) == 0) && !benutzerZugriff("spieler_aendern") && !benutzerVeranstalterModerator() && !benutzerVereinModerator())
+{
+	echo "Sie haben keine Berechtigung auf diesen Datensatz!!!"; 
+	die;
+}
 
 ?>
 <table style="width: 100%; border: none; border-spacing: 0; padding: 3px">

src/structure/components/com_sportsmanager/views/sportsmanager/view_admin.php

@@ -1123,7 +1123,6 @@ class HTML_sportsmanager_admin
     static function adminSpieler($filter, $rows, $anzahlSpieler, $admin_uebersicht, $beschraenkter_zugriff, $ansprechpartner_importieren, $ansprechpartner_exportieren, $ansprechpartner_spielerdaten_aendern, $ansprechpartner_bilder_aendern): void
     {
         global $params;
-
         ?>
         <div
             class="componentheading<?php echo $params->get('pageclass_sfx'); ?>"><?php echo Text::_('COM_SPORTSMANAGER_PLAYERS'); ?>
@@ -1243,8 +1242,10 @@ class HTML_sportsmanager_admin
                         :</strong>
                 </td>
                 <td nowrap style="text-align: right"><?php echo $anzahlSpieler["passiv"]; ?></td>
-                <td></td>
-                <td></td>
+                <td nowrap style="vertical-align: top"><strong><?php echo Text::_('COM_SPORTSMANAGER_PLAYER_UNATTACHED'); ?>
+                        :</strong>
+                </td>
+                <td nowrap style="text-align: right"><?php echo $anzahlSpieler["vereinslos"]; ?></td>
             </tr>
             <tr>
                 <td nowrap style="vertical-align: top"><strong><?php echo Text::_('COM_SPORTSMANAGER_MEN_ACTIVE'); ?>
@@ -1400,11 +1401,11 @@ class HTML_sportsmanager_admin
         <form action="<?php echo SportsManagerURL($row != null ? ('#id' . $row->spieler_id) : ''); ?>" method="post"
               name="adminForm" id="adminForm" enctype="multipart/form-data">
             <div class="uk-overflow-auto">
-                <table class="uk-table" style="width: 100%">
+                <table style="width: 100%">
                     <tr>
                         <td nowrap style="width: 60%; vertical-align: top">
                             <div class="uk-overflow-auto">
-                                <table class="uk-table" style="width: 100%">
+                                <table style="width: 100%">
                                     <tr>
                                         <td nowrap style="width: 20%; text-align: right">
                                             <label for="lastname"><?php echo Text::_('COM_SPORTSMANAGER_LAST_NAME') ?>

src/structure/language/de-DE/de-DE.com_sportsmanager.ini

@@ -363,6 +363,7 @@ COM_SPORTSMANAGER_STATISTIK="Statistik"
 COM_SPORTSMANAGER_PLAYERS_ACTIVE="Spieler aktiv"
 COM_SPORTSMANAGER_RESTRICTED_PLAYERS="Spieler eingeschränkt"
 COM_SPORTSMANAGER_PLAYER_PASSIV="Spieler passiv"
+COM_SPORTSMANAGER_PLAYER_UNATTACHED="Spieler vereinslos"
 COM_SPORTSMANAGER_MEN_ACTIVE="Herren aktiv"
 COM_SPORTSMANAGER_WOMEN_ACTIVE="Damen aktiv"
 COM_SPORTSMANAGER_JUNIOR_ACTIVE="Junioren aktiv"

src/structure/language/en-GB/en-GB.com_sportsmanager.ini

@@ -361,8 +361,9 @@ COM_SPORTSMANAGER_PLAYERS_EXPORT_SPORT_SOFTWARE="Export players (Sport Software)
 COM_SPORTSMANAGER_CLEANUP_INACTIVE_PLAYERS="Cleanup inactive players"
 COM_SPORTSMANAGER_STATISTIK="Statistics"
 COM_SPORTSMANAGER_PLAYERS_ACTIVE="Players active"
-COM_SPORTSMANAGER_RESTRICTED_PLAYERS="Players restrited"
+COM_SPORTSMANAGER_RESTRICTED_PLAYERS="Players restricted"
 COM_SPORTSMANAGER_PLAYER_PASSIV="Players passive"
+COM_SPORTSMANAGER_PLAYER_UNATTACHED="Players unattached"
 COM_SPORTSMANAGER_MEN_ACTIVE="Male active"
 COM_SPORTSMANAGER_WOMEN_ACTIVE="Female active"
 COM_SPORTSMANAGER_JUNIOR_ACTIVE="Juniors active"