[ADB] Welcome Message Mod 0.1.0

[djchrisnet]

MOD Title: Welcome Message Mod
MOD Description: Fügt eine Willkommens-Nachricht dem Header hinzu.
MOD Version: 0.1.0
phpBB Version: 3.0.0-Gold
Format:
Demo: http://djchrisnet.de
Download:

Ihr habt die Möglichkeit, alle Nachrichten der jeweiligen Stunde und jeder verfügbaren Sprache über die Sprachfiles induviduell anpassen. Die englische Sprachdatei wurde nur vorbereitet, also nicht wundern dass im englischen nichts angezeigt wird. Der User wird mit seinem Nicknamen begrüßt. Bei Gästen wird der Nickname zur faq verlinkt, ansonsten gehts ab ins eigene Profil.

Bei Fragen oder Verbesserungen gerne benachrichtigen.

[djchrisnet]

Ich würde gerne mal vorsichtig fragen, ob sich jemand der "profis" hier damit auseinandergesetzt hat, und in bezug auf Code Guiness, ...... mir irgendwas falsch gelaufen ist.

[Dave]

Hi

Ich denke das folgendes besser ist/wäre:

Code: Alles auswählen

//Adding Language Files
$user->add_lang('mods/lang_welcome');

$user_id = $user->data['user_id'];

//Are you a Guest ?
if ($user_id == ANONYMOUS || $user->data['is_bot']) 
{
	$user_link = append_sid("{$phpbb_root_path}faq.$phpEx"); 
	$user_name = $user->lang['WELCOME_GUEST'];
}
else
{
	$user_link = append_sid("{$phpbb_root_path}memberlist.$phpEx", "mode=viewprofile&u=$user_id");
	$user_name = $user->data['username'];
}

//Creating Site Vars
$template->assign_vars(array(
	'W_HELLO'	=> $user->lang['WELCOME_HELLO'],
	'W_NAME'	=> $user_name,
	'W_LINK'	=> $user_link,
	'W_COLOR'	=> $user->data['user_colour'],
	'W_MESSAGE'	=> $user->lang['WELCOME_TEXT'][$user->format_date(time(), date('G'))],
));

Wenn du dazu fragen hast dann schieß los
btw: ungetestet

MfG
Dave

[lefty74]

sieht so aehnlich aus wie dieser hier
http://www.phpbb.com/community/viewtopi ... 0&t=567191

[djchrisnet]

sieht so aehnlich aus wie dieser hier
http://www.phpbb.com/community/viewtopi ... 0&t=567191

ohne über den verlinkten Mod negatives schreiben zu wollen!!!

Der Verlinkte Mod ist allerdings nicht multi-language fähig, ausserdem sind teile davon in javascript programmiert. keine Sommerzeit anpassung, keine Zeitzonen. So kann für jede Sprache und jede Stunde eine eigene Nachricht verfasst werden. Gerade aus diesem Grund lädt er auch die Texte nicht aus der datenbank. Ich halte es so für besser und es werden unnötige Query´s gesparrt!


@ Dave: Das mit dem Bot-anpassung ist gar nicht mal so schlecht! So wird ja z.b. im Google Cache auch Gast angezeigt, anstatt Google [Bot]. Werde das übernehmen. Allerdings so:

Code: Alles auswählen

if ($user->data['user_id'] == ANONYMOUS || $user->data['is_bot']) 

[Dave]

@ Dave: Das mit dem Bot-anpassung ist gar nicht mal so schlecht! So wird ja z.b. im Google Cache auch Gast angezeigt, anstatt Google [Bot]. Werde das übernehmen. Allerdings so:

Code: Alles auswählen

if ($user->data['user_id'] == ANONYMOUS || $user->data['is_bot']) 

hi,
Ich habe das so gemacht da wir die $user_id auch später nochmal brauchen. Also solltest du sie nicht komplett rauslöschen

[djchrisnet]

Ich werde die Variable $user_id ganz rausnehmen und die ID 2x einlesen, weil da dieser Mod ja auch allen Seiten angezeigt wird und die Variable $user_id eventuel auch anderswo genutzt wird/wurde/werden wird.

[Dave]

Dann denk daran das du aus

Code: Alles auswählen

$user_link = append_sid("{$phpbb_root_path}memberlist.$phpEx", "mode=viewprofile&u=$user_id");

nicht

Code: Alles auswählen

$user_link = append_sid("{$phpbb_root_path}memberlist.$phpEx", "mode=viewprofile&u=$user->data['user_id']");

machen kannst. Da mußte dann wohl den string unterbrechen vorher was ich eigentlich nicht vorhatte und ich glaube auch nicht im phpBB gemacht wird


MfG
Dave

[nickvergessen]

Code: Alles auswählen

$user_link = append_sid("{$phpbb_root_path}memberlist.$phpEx", "mode=viewprofile&u=" . $user->data['user_id']);

Ähm, ich glaube nicht, dass dies all zu selten gemacht wird.

[djchrisnet]

Ähm, ich glaube nicht, dass dies all zu selten gemacht wird.

Wie soll ich dies nun verstehen?