ereg_replace to preg_replace help? :)

wedran

Hello to everyone

i have problem converting one old php event calendar script to new php,
so i have part of code here:

$today_d = date("j",$sys['now_offset'] + $usr['timezone'] * 3600);
$today_m = date("n",$sys['now_offset'] + $usr['timezone'] * 3600);
$today_y = date("Y",$sys['now_offset'] + $usr['timezone'] * 3600);
$m_c = date("F",mktime(0,0,0,$m,1,$y));
$m_c = ereg_replace($m_c,$L_month[$m_c],$m_c);
$m_c .= date(" Y",mktime(0,0,0,$m,1,$y));
$m_p = date("F",mktime(0,0,0,$m-1,1,$y));
$m_p = ereg_replace($m_p,$L_month[$m_p],$m_p);
$m_p .= date(" Y",mktime(0,0,0,$m-1,1,$y));
$m_n = date("F",mktime(0,0,0,$m+1,1,$y));
$m_n = ereg_replace($m_n,$L_month[$m_n],$m_n);
$m_n .= date(" Y",mktime(0,0,0,$m+1,1,$y));
$l1 = sed_mktime(0,0,0,$m,1,$y);
$l2 = sed_mktime(0,0,0,$m+1,1,$y);

what i need to change to convert it to preg_replace function?

i am pretty newbie to php 🙂

tnx

Weedpacket

I don't see anything there that uses regular expressions anyway. Why not just use [man]str_replace[/man]?

wedran

like $m_n = str_replace($m_n,$L_month[$m_n],$m_n); ? probably not 🙂 as i sad i am newbie,
that is old plugin for sedition cms, and need to be change to preg_replace or something?
when i just change it to preg_replace i get this server error: PHP Warning: preg_replace(): Delimiter must not be alphanumeric or backslash in ...
so, what to do? 🙂

Weedpacket

wedran wrote:
like $m_n = str_replace($m_n,$L_month[$m_n],$m_n); ? probably not

Yes, because there aren't any regular expressions being used in that piece of code.

that is old plugin for sedition cms

A poorly-written one, frankly.

At another glance, it can be simplified further. As well as the absence of anything involving regular expressions, the choice of replacement is based on the entire string, the entire string is matched, and the entire string is replaced. So

$m_c = ereg_replace($m_c,$L_month[$m_c],$m_c);

would be replaced by

$m_c = $L_month[$m_c];

No need for ereg_replace, preg_replace, or str_replace.

wedran

ok, thank you very much 🙂