kickern/com_sportsmanager
1
0
Fork 0
mirror of https://github.com/Deutscher-Tischfussballbund/com_sportsmanager.git synced 2026-06-10 06:27:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #113 from Deutscher-Tischfussballbund/sportsmanager2-issue059

Browse Source 
Sportsmanager2 issue059
This commit is contained in:
MarvinF
2025-08-11 17:58:26 +02:00
committed by GitHub
parent b1f67d1d13 8479691824
commit e0575dc125
4 changed files with 60 additions and 51 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/structure/components/com_sportsmanager/admin.php
+41 -40
View File
@@ -20,8 +20,6 @@ require_once JPATH_SITE . '/components/com_sportsmanager/database/init.php';
function UhrzeitWandlerDE($Zeitstempel): mixed
{
//Jürgen Meyer 07.03.2025
$Temp = explode(' ', $Zeitstempel);
$Datum = $Temp[0];
$Uhrzeit = $Temp[1];
@@ -36,7 +34,6 @@ function UhrzeitWandlerDE($Zeitstempel): mixed
function validateDate($date, $format = 'Y-m-d H:i:s'): bool
{
//Jürgen Meyer 07.03.2025
$d = DateTime::createFromFormat($format, $date);
return $d && $d->format($format) == $date;
}
@@ -2242,14 +2239,9 @@ function adminImportSpielerDetailsForm(): void
$rows = $db->loadObjectList();
foreach ($rows as $row) {
$query = "DELETE FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '$row->session_id'";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
adminDeleteSpielerImport($row->session_id);
}
if (empty($_FILES["daten"]["tmp_name"]) || !is_uploaded_file($_FILES["daten"]["tmp_name"])) {
redirectSportsManagerURL('&task=admin_spieler', Text::_('COM_SPORTSMANAGER_NO_FILE_SPECIFIED'));
}
@@ -2507,12 +2499,7 @@ function adminImportSpielerDetailsForm(): void
die($db->stderr(true));
}
$query = "DELETE FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '" . $db->escape($session_id) . "'";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
adminDeleteSpielerImport($session_id);
HTML_sportsmanager_admin::adminImportSpielerBericht($lizenzen_aktualisiert, array(), 0, array(), array(), array(), false, redirectSessionIdEmpty() ? null : redirectSessionIdSetzen());
@@ -2629,8 +2616,19 @@ function adminImportSpielerDetailsForm(): void
HTML_sportsmanager_admin::adminImportSpielerDetails($beschraenkter_zugriff, $ansprechpartner_importieren, $veranstalter, $veranstalter_import, $veranstalter_zugehoerigkeit, $session_id, implode(",", $spalten), $verein_import, $vereine, einstellungswert("spielerimport_persoenliche_daten_vorauswahl"));
}
function adminDeleteSpielerImport($session_id){
$db = getDatabase();
$query = "DELETE FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '" . $db->escape($session_id) . "'";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
}
function adminImportSpielerVorschau(): void
{
$db = getDatabase();
global $_FILES;
$jInput = Factory::getContainer()->get(SiteApplication::class)->input;
@@ -2677,6 +2675,24 @@ function adminImportSpielerVorschau(): void
$n++;
}
//check Passnummer auf gueltiges Format
$query = "SELECT nachname, vorname, spielernr, spielernr_alt"
. "\n FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '" . $db->escape($session_id) . "'"
. "\n AND ((spielernr NOT REGEXP '^[0-9]{2}-[0-9]{4,6}$' AND spielernr <> '')"
. "\n OR (spielernr_alt NOT REGEXP '^[0-9]{2}-[0-9]{4,6}$' AND spielernr_alt <> ''));";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
$spielerfehler = $db->loadObjectList();
if (count($spielerfehler) > 0) {
adminDeleteSpielerImport($session_id);
HTML_sportsmanager_admin::adminImportSpielerFehler($spielerfehler, $fehler="Passnummer");
return;
}
$spielernr_aendern = false;
if (isset($spalten["spielernr_alt"])) {
$query = "SELECT nachname, vorname, spielernr, spielernr_alt"
@@ -2713,9 +2729,8 @@ function adminImportSpielerVorschau(): void
$spielernr_aendern = false;
}
}
if ($spielernr_aendern) {
$query = "SELECT nachname, vorname"
$query = "SELECT nachname, vorname, bestand.spielernr"
. "\n FROM"
. "\n (SELECT nachname, vorname, spielernr, spielernr_alt"
. "\n FROM #__sportsmanager_spieler_import"
@@ -2727,15 +2742,15 @@ function adminImportSpielerVorschau(): void
. "\n ON bestand.spielernr = import.spielernr"
. "\n WHERE NOT EXISTS(SELECT * FROM #__sportsmanager_spieler_import WHERE spielernr_alt = bestand.spielernr AND spielernr != spielernr_alt AND NOT ISNULL(spielernr) AND NOT ISNULL(spielernr_alt) AND spielernr != '' AND spielernr_alt != '')"
. "\n ORDER BY nachname, vorname";
$db->setQuery($query);
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
$konflikte = $db->loadObjectList();
if (count($konflikte) > 0) {
HTML_sportsmanager_admin::adminImportSpielerFehler($konflikte);
adminDeleteSpielerImport($session_id);
HTML_sportsmanager_admin::adminImportSpielerFehler($konflikte, $fehler="konflikt");
return;
}
}
@@ -2755,7 +2770,6 @@ function adminImportSpielerVorschau(): void
die($db->stderr(true));
}
$spieler_import = $db->loadObjectList();
$import_verweigern = false;
if ($beschraenkter_zugriff) {
foreach ($spieler_import as $s) {
@@ -2785,7 +2799,7 @@ function adminImportSpielerVorschau(): void
. "\n FROM #__sportsmanager_spieler_import"
. "\n LEFT JOIN #__sportsmanager_spieler ON #__sportsmanager_spieler_import.spielernr != '' AND " . ($spielernr_aendern ? "IF(NOT ISNULL(#__sportsmanager_spieler_import.spielernr_alt) AND #__sportsmanager_spieler_import.spielernr_alt != '', #__sportsmanager_spieler_import.spielernr_alt = #__sportsmanager_spieler.spielernr, #__sportsmanager_spieler_import.spielernr = #__sportsmanager_spieler.spielernr)" : "#__sportsmanager_spieler_import.spielernr = #__sportsmanager_spieler.spielernr")
. "\n WHERE session_id = '" . $db->escape($session_id) . "'";
$db->setQuery($query);
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
@@ -2797,14 +2811,8 @@ function adminImportSpielerVorschau(): void
}
if ($import_verweigern) {
$query = "DELETE FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '" . $db->escape($session_id) . "'";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
adminDeleteSpielerImport($session_id);
}
HTML_sportsmanager_admin::adminImportSpielerVorschau($import_verweigern, $spieler_import, $veranstalter, $session_id, $jInput->get('persoenliche_daten', 0, 'INT'), $jInput->get('lizenznr_beibehalten', 0, 'INT'), $jInput->get('spalten', '', 'RAW'));
}
@@ -2880,7 +2888,7 @@ function adminImportSpieler(): void
* sonst Spielernr. übernehmen
*/
if (isset($spalten["spielernr_alt"])) {
$query = "SELECT nachname, vorname"
$query = "SELECT nachname, vorname, bestand.spielernr"
. "\n FROM"
. "\n (SELECT nachname, vorname, spielernr, spielernr_alt"
. "\n FROM #__sportsmanager_spieler_import"
@@ -2897,10 +2905,8 @@ function adminImportSpieler(): void
die($db->stderr(true));
}
$konflikte = $db->loadObjectList();
if (count($konflikte) > 0) {
HTML_sportsmanager_admin::adminImportSpielerFehler($konflikte);
HTML_sportsmanager_admin::adminImportSpielerFehler($konflikte, $fehler="konflikt");
return;
}
@@ -3342,12 +3348,7 @@ function adminImportSpieler(): void
}
}
$query = "DELETE FROM #__sportsmanager_spieler_import"
. "\n WHERE session_id = '" . $db->escape($session_id) . "'";
$db->setQuery($query);
if (!$db->execute()) {
die($db->stderr(true));
}
adminDeleteSpielerImport($session_id);
sort($vereineManuell);
sort($ausgetreteneVereineMitAktivenMitgliedern);
src/structure/components/com_sportsmanager/views/sportsmanager/view_admin.php
+15 -9
View File
@@ -2293,23 +2293,21 @@ class HTML_sportsmanager_admin
<?php
}
static function adminImportSpielerFehler($spieler): void
static function adminImportSpielerFehler($spieler, $fehler): void
{
global $params;
$spieler_anzahl = count($spieler);
?>
<div
class="componentheading<?php echo $params->get('pageclass_sfx'); ?>"><?php echo Text::_('COM_SPORTSMANAGER_PLAYERS_IMPORT2'); ?></div>
<div class="componentheading<?php echo $params->get('pageclass_sfx'); ?>">
<?php echo Text::_('COM_SPORTSMANAGER_PLAYERS_IMPORT2'); ?>
</div>
<table class="contentpaneopen<?php echo $params->get('pageclass_sfx'); ?>">
<tr>
<td>
<?php
if ($spieler_anzahl > 0) {
?>
<?php echo Text::_('COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE'); ?>
<?php
echo Text::_('COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE');
}
?>
</td>
@@ -2322,12 +2320,20 @@ class HTML_sportsmanager_admin
<table class="contentpaneopen<?php echo $params->get('pageclass_sfx'); ?>">
<tr>
<td nowrap class="contentheading<?php echo $params->get('pageclass_sfx'); ?>"
style="width: 100%"><?php echo Text::_('COM_SPORTSMANAGER_IMPORT_DUPLICATE_MESSAGE'); ?></td>
style="width: 100%">
<?php
if ($fehler == "konflikt")
echo Text::_('COM_SPORTSMANAGER_IMPORT_DUPLICATE_MESSAGE');
if ($fehler == "Passnummer")
echo Text::_('COM_SPORTSMANAGER_IMPORT_WRONG_FORMAT_PLAYERNUMBER');
?>
</td>
</tr>
</table>
<table class="contentpaneopen<?php echo $params->get('pageclass_sfx'); ?>">
<tr class="sectiontableheader<?php echo $params->get('pageclass_sfx'); ?>">
<th nowrap><strong><?php echo Text::_('COM_SPORTSMANAGER_PLAYER_NUMBER_SHORT'); ?></strong></th>
<th nowrap><strong><?php echo Text::_('COM_SPORTSMANAGER_NAME'); ?></strong></th>
</tr>
<?php
@@ -2337,6 +2343,7 @@ class HTML_sportsmanager_admin
?>
<tr class="sectiontableentry<?php echo $k + 1;
$k = ($k + 1) % 2; ?><?php echo $params->get('pageclass_sfx'); ?>">
<td nowrap><?php echo htmlentities_utf8($s->spielernr); ?></td>
<td nowrap><?php echo htmlentities_utf8($s->nachname . ", " . $s->vorname); ?></td>
</tr>
<?php
@@ -2359,7 +2366,6 @@ class HTML_sportsmanager_admin
static function adminImportSpielerVorschau($import_verweigern, $spieler, $veranstalter, $session_id, $persoenliche_daten, $lizenznr_beibehalten, $spalten): void
{
global $params;
$spieler_anzahl = count($spieler);
?>
src/structure/language/de-DE/de-DE.com_sportsmanager.ini
+2 -1
View File
@@ -404,8 +404,9 @@ COM_SPORTSMANAGER_COUNTRY_CODE="Landeskennung"
COM_SPORTSMANAGER_IMPORT="Importieren"
COM_SPORTSMANAGER_IMPORT_MESSAGE="Im Import sind ausschlie&szlig;lich Spielerdaten zum Verein %s enthalten. Soll ausschlie&szlig;lich der Spielerbestand des einen Vereins aktualisiert werden, muss der zugeh&ouml;rige Verein unten ausgew&auml;hlt werden. Beinhaltet der Import den gesamten Spielerbestand einer Organisation, muss die zugeh&ouml;rige Organisation gew&auml;hlt werden."
COM_SPORTSMANAGER_CHECK="Pr&uuml;fen"
COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE="Im Import sind Konflikte enthalten, die im Vorfeld manuell beseitigt werden müssen."
COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE="Im Import sind Fehler oder Konflikte enthalten, die im Vorfeld manuell beseitigt werden müssen."
COM_SPORTSMANAGER_IMPORT_DUPLICATE_MESSAGE="Versuch, Spielernr. auf eine bereits für einen anderen Spieler vergebene Spielernr. zu ändern"
COM_SPORTSMANAGER_IMPORT_WRONG_FORMAT_PLAYERNUMBER="Eine oder mehrere Spielernummer enthalten ein ung&uuml;ltiges Format"
COM_SPORTSMANAGER_NAME2="Name"
COM_SPORTSMANAGER_DATA_IMPORT_ABORT_MESSAGE="Der Import wird abgebrochen, da Konflikte bei den zu importierenden Spielerdaten bestehen. Bitte kontaktiere einen Moderator und sende dabei die Importdatei mit!"
COM_SPORTSMANAGER_DATA_IMPORT_NO_CONFLICTS="Es bestehen keine Konflikte bei den zu importierenden Spielerdaten."
src/structure/language/en-GB/en-GB.com_sportsmanager.ini
+2 -1
View File
@@ -404,8 +404,9 @@ COM_SPORTSMANAGER_COUNTRY_CODE="Country code"
COM_SPORTSMANAGER_IMPORT="Import"
COM_SPORTSMANAGER_IMPORT_MESSAGE="In the import there are only player information about club %s present. Shall only the members of that one club be updated, the associated club has to be selected down here. If the import contains all members of the organisation then the organisation must be selected."
COM_SPORTSMANAGER_CHECK="Check"
COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE="There are conflicts in the import which have to be fixed manually first."
COM_SPORTSMANAGER_IMPORT_CONFLICTS_MESSAGE="There are faults or conflicts in the import which have to be fixed manually first."
COM_SPORTSMANAGER_IMPORT_DUPLICATE_MESSAGE="Attempt to change player number into one that is already assigned to another player."
COM_SPORTSMANAGER_IMPORT_WRONG_FORMAT_PLAYERNUMBER="One or more player numbers contain an invalid format"
COM_SPORTSMANAGER_NAME2="Name"
COM_SPORTSMANAGER_DATA_IMPORT_ABORT_MESSAGE="The import has been aborted because there are conflicts in the containing player information. Please contact a moderator and attach the import!"
COM_SPORTSMANAGER_DATA_IMPORT_NO_CONFLICTS="There are conflicts in the containing player information."