Seite 2 von 2
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 11:16
von Kirk
Ich hab´s auch getestet (Version 1.2.2 und php 8.1) wie Talk19zehn schrieb ist z.B. jenes:
in der oben genannten Version in Zeile 274, selbst wenn ich diese Änderungen mache kann ich den img BBCode ohne Fehlermeldung benutzen.
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 11:46
von eumelchen
Ich habe jetzt bei mir noch einmal alles überprüft.
In den Template-html-Dateien der Extension habe ich umfangreiche Änderungen vorgenommen, so u.A. ein festes Bild im Header der Extension und ich habe die Befehle für pic_1 und pic_2 in der daily.html entfernt.
Bilder setze ich nur noch mit IMG ein.
In der daily_module.php habe ich jetzt mal zusätzlich zu den IMG-Änderungen noch die Zeilen 91 - 151 entfernt
Code: Alles auswählen
$form_name = 'pic_1';
$pic_1_file = $request->file($form_name);
unset($pic_1_file['local_mode']);
$pic_1 = $upload->handle_upload('files.types.form', $form_name);
$pic_1_name = $pic_1_file['name'];
$pic_1_ext = str_replace('.', '', trim(strrchr(strtolower($pic_1_name), '.')));
if ($pic_1_name && in_array($pic_1_ext, $allowed_imagetypes))
{
$error_count = sizeof($pic_1->error);
if ($error_count > 1 && $pic_1->uploadname)
{
$pic_1->remove();
trigger_error(implode('<br />', $pic_1->error), E_USER_ERROR);
}
$pic_1->error = array();
$pic_1_filename = $save_day . '_1.' . $pic_1_ext;
@unlink($ext_path_images . $old_pic_1);
$pic_1_file['name'] = $pic_1_filename;
$pic_1->set_upload_ary($pic_1_file);
$pic_1->move_file(str_replace($phpbb_root_path, '', $ext_path_images), false, false, CHMOD_ALL);
}
else
{
$pic_1->remove();
$pic_1_filename = '';
}
$form_name = 'pic_2';
$pic_2_file = $request->file($form_name);
unset($pic_2_file['local_mode']);
$pic_2 = $upload->handle_upload('files.types.form', $form_name);
$pic_2_name = $pic_2_file['name'];
$pic_2_ext = str_replace('.', '', trim(strrchr(strtolower($pic_2_name), '.')));
if ($pic_2_name && in_array($pic_2_ext, $allowed_imagetypes))
{
$error_count = sizeof($pic_2->error);
if ($error_count > 1 && $pic_2->uploadname)
{
$pic_2->remove();
trigger_error(implode('<br />', $pic_2->error), E_USER_ERROR);
}
$pic_2->error = array();
$pic_2_filename = $save_day . '_2.' . $pic_2_ext;
@unlink($ext_path_images . $old_pic_2);
$pic_2_file['name'] = $pic_2_filename;
$pic_2->set_upload_ary($pic_2_file);
$pic_2->move_file(str_replace($phpbb_root_path, '', $ext_path_images), false, false, CHMOD_ALL);
}
else
{
$pic_2->remove();
$pic_2_filename = '';
}
Jetzt scheint alles zu funktionieren, es kommt keine Fehlermeldung mehr.
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 12:14
von Kirk
Warum so kompliziert, von der Anwendung für den Endverbraucher hat oxpus es eigentlich sehr gut gemacht. Man müsste eigentlich an den Dateien insb. PHP Dateien nichts ändern oder löschen.
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 12:26
von oxpus
Wenn an Templates Änderungen durchgeführt werden, ist es schwer mit dem Support. Besonders ohne dessen Erwähnung.
Daher wie Kirk schon schrieb: Warum etwas ändern, was prima funktioniert?
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 12:48
von eumelchen
Erst einmal entschuldigt das ich nicht sofort an meine Änderungen in den html-Dateien gedacht habe.
Natürlich habt Ihr recht, die Extension funktioniert prima und ich wollte ansich nur ein paar kleine Änderungen vornehmen.
Wenn ich mich richtig erinnere war der Hauptgrund das mir die Bilder etwas zu klein waren.
Als ich einmal angefangen hatte kam dann eins zum anderen und jetzt ist es so wie es mir und meinen Usern 100%-tig gefällt.
Es ist wirklich eine sehr schöne und funktionale Extension und ich bin froh das es sie gibt.
Wenn irgendwann wegen Updates "meine" Version nicht mehr funktionieren sollte kann ich immer wieder auf die Orginal-Version wechseln.
Auf jeden Fall vielen Dank für Eure Hilfe.
LG Rolf (eumelchen)
Re: Problem mit Advent Inducement Calender
Verfasst: 05.11.2022 15:47
von eumelchen
Hier noch 2 Bilder wie ein Tagesinhalt bei mir aussieht.
Desktop:
[ externes Bild ]
Sebstverständlich ist der Tagesinhalt responsiv.
Smartphone:
[ externes Bild ]