Fehler in Snippet - Nutzung von append_sid()

[BB-BF-BM]

Guten Tag!

Ich glaube, das Snippet Gäste auf "Wer ist online" per Default anzeigen enthält einen Fehler:

In phpBB 3.0 kann bzw. soll man den Querystring (in diesem Beispiel "sg=1") als zweiten Parameter übergeben.
Also sollte der Ersatzcode wie folgt sein:

Code: Alles auswählen

      'U_VIEWONLINE'         => ($auth->acl_gets('u_viewprofile', 'a_user', 'a_useradd', 'a_userdel')) ? append_sid("{$phpbb_root_path}viewonline.$phpEx", 'sg=1') : '',

EDIT: siehe auch Coding Guidelines 2. V "Url formatting"

[Pyramide]

Man kann, muss aber nicht - die alte Variante funktioniert auch weiterhin. Siehe die Beschreibung der Funktion in functions.php:

Code: Alles auswählen

@param string $url The url the session id needs to be appended to (can have params)

[...]

Examples:
append_sid("{$phpbb_root_path}viewtopic.$phpEx?t=1&f=2");
append_sid("{$phpbb_root_path}viewtopic.$phpEx", 't=1&f=2');

[Dr.Death]

Da es nach den aktuellen Coding Guidelines "schöner" aussieht werde ich dieses Snippet anpassen.

Von der Funktion her ändert sich nichts.