Mam kod w pliku SSI.php z forum Simple Machine Forum.
Poniżej mam funkcję w której nie trzeba zwracać uwagę na to co wyświetli, a raczej chodzi o to dlaczego nie działa ?

ORGINALNA. (echo) - oczywiście działa świetnie

[PHP] pobierz, plaintext 
function ssi_welcome($output_method = 'echo')
{
	global $context, $txt, $scripturl;
 
	if ($output_method == 'echo')
	{
		if ($context['user']['is_guest'])
			echo sprintf($txt['welcome_guest'], $txt['guest_title']);
		else
			echo $txt['hello_member'], ' <strong>', $context['user']['name'], '</strong>', allowedTo('pm_read') ? ', ' . $txt['msg_alert_you_have'] . ' <a href="' . $scripturl . '?action=pm">' . $context['user']['messages'] . ' ' . ($context['user']['messages'] == '1' ? $txt['message_lowercase'] : $txt['msg_alert_messages']) . '</a>' . $txt['newmessages4'] . ' ' . $context['user']['unread_messages'] . ' ' . ($context['user']['unread_messages'] == '1' ? $txt['newmessages0'] : $txt['newmessages1']) : '', '.';
	}
	// Don't echo... then do what?!
	else
		return $context['user'];
}
[PHP] pobierz, plaintext 

WŁASNA ($SHOW .=)

[PHP] pobierz, plaintext 
function ssi_welcome($output_method = 'echo')
{
	global $context, $txt, $scripturl;
	if ($output_method == 'echo')
	{
		if ($context['user']['is_guest']) $SHOW .= sprintf($txt['welcome_guest'], $txt['guest_title']);
		else $SHOW .= ' ',$txt['hello_member'], ' <strong>', $context['user']['name'], '</strong>', allowedTo('pm_read') ? ', ' . $txt['msg_alert_you_have'] . ' <a href="' . $scripturl . '?action=pm">' . $context['user']['messages'] . ' ' . ($context['user']['messages'] == '1' ? $txt['message_lowercase'] : $txt['msg_alert_messages']) . '</a>' . $txt['newmessages4'] . ' ' . $context['user']['unread_messages'] . ' ' . ($context['user']['unread_messages'] == '1' ? $txt['newmessages0'] : $txt['newmessages1']) : '', '.';
	}
	else return $context['user'];
}
[PHP] pobierz, plaintext 

Pojawia się błąd.


Dlaczego po zmianie echo na zmienną przestało działać wyświetlanie ?

Ja znam teraz tylko jeden sposób na naprawę, aby funkcjonowało zgodnie z moimi oczekiwaniami czyli przepisać każdą funkcję stosując normalne kodowanie.

Całe przypisanie do $SHOW bruździ bo używa , a nie . jako symbolu konkatenacji stringów. Echo jest konstruktem, który pozwala zastosować , z racji podawania kolejnych stringów jako parametrów dla niego. Innymi słowy:

[PHP] pobierz, plaintext 
echo $cos, 'tekst', $cos2, $cos3;
[PHP] pobierz, plaintext 

jest poprawne. Ale już

[PHP] pobierz, plaintext 
$SHOW .= $cos, 'tekst', $cos2, $cos3;
[PHP] pobierz, plaintext 

jest błędem składni. Ot całe magiczne know-how. Zamień , na . i powinno hulać.

Masz racje. Dzięki.

Ale raczej jest na odwrót.

Bo wszędzie zmienna $SHOW używa kropki, a tylko w funkcjach SSI.php jest przecinek i kropka w jednej linii jednocześnie.

To ja chyba widzę za dużo przecinków Pozwól że przekleję Twój kod i zaznaczę na czerwono gdzie dla $SHOW w Twojej metodzie widać przecinki.

$SHOW .= ' ',$txt['hello_member'], ' <strong>', $context['user']['name'], '</strong>', allowedTo('pm_read') ? ', ' . $txt['msg_alert_you_have'] . ' <a href="' . $scripturl . '?action=pm">' . $context['user']['messages'] . ' ' . ($context['user']['messages'] == '1' ? $txt['message_lowercase'] : $txt['msg_alert_messages']) . '</a>' . $txt['newmessages4'] . ' ' . $context['user']['unread_messages'] . ' ' . ($context['user']['unread_messages'] == '1' ? $txt['newmessages0'] : $txt['newmessages1']) : '', '.';

I naprawdę zwracaj uwagę oraz sprawdź dwa razy, gdy ktoś Ci mówi, że jednak jest inaczej niż myślisz. Popraw te przecinki na kropki, a powinno zadziałać.

Co do uzycia przecinków i kropek w SSI.php, to może to wynikać z dorzucenia tego skryptu przez kogoś, kto nie zwrócił na to uwagi bo całość działałajak powinna i uzycie w echo jest poprawne, więc nie sypało nawet notice'a Ty przerabiałeś i już na to nie zwróciłeś uwagi i stąd nagle błędy. Trudno na początek zauważyć to, bo można mieć wrażenie, że przecinki oddzielają parametry którejś funkcji wewnątrz.