Oh, stimmt. Ich glaube das wurde schon einmal im Forum gemeldet/besprochen.lck wrote: ↑Thu Aug 11, 2022 11:04 amBei mir liefert der PHP error_log folgendes:In der Zeile stehtphp_error_log wrote:PHP Fatal error: Array and string offset access syntax with curly braces is no longer supported in D:\...\xh175\plugins\newsletter\index.php on line 581ersetze alle Vorkommen vonCode: Select all
$k[$i]=ord($ky{$i})&0x1F;
durchCode: Select all
{$i}
, es sind insgesamt 3, in den Zeilen 581, 585 und 586. Dann sollte Newsletter zumindest laufen.Code: Select all
[$i]
Soweit ich es überblicke, ist die Warnung eher harmlos, aber man sollte sich diesen Code trotzdem mal vornehmen. Zum einen sollte man den Rückgabewert von preg_match() prüfen und entsprechend behandeln; dann spart man sich die Warnungen, und auch unnötige Funktionsaufrufe. Und die beiden preg_match() Paare (Zeile 334/335 und 337/338) erscheinen unnötig; das sollte eigentlich mit jeweils einem einzigen preg_match() zu erledigen sein. Wer mag, kann ja mal folgendes ausprobieren (ungetested):lck wrote: ↑Thu Aug 11, 2022 11:04 amIm Backend des Plugins tauchen dann noch ein paar Warnings auf. Christoph, was wäre da zu ändern?Zeile 335 bis 338Debug Mode wrote:WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:335
WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:336
WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:338
WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:355
WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:356
WARNING: Undefined array key 0
D:\xampp\htdocs\Portable_XH-3.1\www\xh175_test_calendar148\plugins\newsletter\admin.php:379Code: Select all
preg_match('/newsletter\(["|\'].*["|\']\)/',$mres[0],$mres); if (trim($mres[0]=="")) { /// try {{{PLUGIN:....}}} notification preg_match('/{{{[PLUGIN:]*newsletter\(["|\'].*["|\']\).*}}}/i',$i,$mres); preg_match('/newsletter\(["|\'].*["|\']\)/',$mres[0],$mres);
Code: Select all
$page[$l[$count]]=$h[$count];
preg_match('/#cmsimple\s+.*(newsletter\(["|\'].*["|\']\)).*#/i',$i,$mres);
if (trim($mres[1]=="")) { /// try {{{PLUGIN:....}}} notification
preg_match('/{{{[PLUGIN:]*(newsletter\(["|\'].*["|\']\)).*}}}/i',$i,$mres);
/*
if (trim($mres[0]=="")) { //not found on the page, search hi_pd_scripting for newsletter calls
if (isset($plugin_cf['hi_pd_scripting']['scripts_max_on_page']) && isset($pd_router) ) { // only if pd_scripting and page_param are installed search through pagedata.php
$page_pd = $pd_router -> find_page($count); // get page
// search for newsletter in hi_pd_scripting fields in pageparam
for ($pd_i = 1; ($pd_i <= $plugin_cf['hi_pd_scripting']['scripts_max_on_page']); $pd_i++) {
if ($page_pd['hi_pd_scripting_active'.$pd_i] == '1') { // activated
preg_match('/newsletter\(["|\'].*["|\']\)/',trim($page_pd['hi_pd_scripting_code'.$pd_i]),$mres);
if ($mres[0]!="")
$pd_i=$plugin_cf['hi_pd_scripting']['scripts_max_on_page']; // found, destroy loop
}
}
}
}
*/
}
preg_match('/(["|\'].*["|\'])/',$mres[1],$mres);
$temp=preg_replace('/["|\']/',"",$mres[0]);