Seite 4 von 16

Verfasst: 15.03.2007 20:12
von BB-BF-BM
@buegelfalte: danke, dass du mich so tatkräftig unterstützt!

Klar kannst du den dazugehörigen Code posten (sofern er nicht zu lang ist), dann kommt das auch schon in das nächste Release!


Werde aber erst Samstag dazu kommen, alles übersichtlich aufzulisten und abzuarbeiten!

EDIT: @Surfer: gut, werde ich ebenfalls mit niedriger Priorität als Feature-Wunsch aufnehmen.

Verfasst: 15.03.2007 20:18
von Surfer90
Ups, da eben noch kein weiterer Beitrag geschrieben war, hatte ich meinen Beitrag editiert, doch nun haste schon längst geantwortet.


Wollte eigendlich noch hinzufügen, dass ich doch noch leider einen Fehler bei mir habe:

Code: Alles auswählen

Fatal error: Unsupported operand types in /var/www/web5/html/klassenforum/phpBB2/calendar.php on line 283 

..kommmt bei der Emailbenachrichtigung bei mir

Verfasst: 15.03.2007 20:39
von buegelfalte
BB-BF-BM hat geschrieben:Klar kannst du den dazugehörigen Code posten (sofern er nicht zu lang ist), dann kommt das auch schon in das nächste Release!
Ist nur ganz kurz, hab nur den eMail-Versand eingebaut, die dazugehörige Einstellungsmöglichkeit zum Abschalten der Funktion im ACP fehlt:

Code: Alles auswählen

#
#-----[ OPEN ]------------------------------------------
#
calendar.php

#
#-----[ FIND ]------------------------------------------
#
			$template->assign_block_vars('message', array(
				'title'	=>	$lang['Information'],
				'msg'	=>	$lang['CAL_Adding_Event_Successful']
			));

#
#-----[ AFTER, ADD ]------------------------------------------
#
			// Mail an admin
			include_once($phpbb_root_path . 'includes/emailer.'.$phpEx);
			$emailer = new emailer($board_config['smtp_delivery']);
			
			$emailer->from($board_config['board_email']);
			$emailer->replyto($board_config['board_email']);
		
			$emailer->use_template('cal_admin_notify', $board_config['default_lang']);
			$emailer->email_address($board_config['board_email']);
			$emailer->set_subject($lang['CAL_Notify']);
		
			$emailer->assign_vars(array(
				'USERNAME'	=> $userdata['username'],
				'NAME'		=> $HTTP_POST_VARS['name'],
				'EVENT'		=> $HTTP_POST_VARS['desc'],
				'EMAIL_SIG'	=> (!empty($board_config['board_email_sig'])) ? str_replace('<br />', "\n", "-- \n" . $board_config['board_email_sig']) : ''
			));
		
			$emailer->send();
			$emailer->reset();
dazu gehört natürlich noch die Datei cal_admin_notify.tpl im Ordner "language\lang_german\email":

Code: Alles auswählen

Subject: Termin-Benachrichtigung
Charset: iso-8859-1

{USERNAME} hat einen neuen Termin eingetragen:


Name:
{NAME}

Beschreibung:
{EVENT}


{EMAIL_SIG}
Das wars schon ... :)

EDIT: da fehlt natürlich noch die Kategorie und das Datum des Termins, aber so tief bin ich noch nicht in die Struktur der Daten eingestiegen.

Verfasst: 15.03.2007 23:12
von buegelfalte
Und noch eine Anregung aus dem Praxistext:
Sehr gut wäre auch eine Erweiterung der Berechtigungen, so daß man einzelnen Usern den Zugriff verbieten (!) kann.
Grund: ich habe z.B. einen (frei zugänglichen) Testaccount für UploadPic im Forum und prompt tragen die Tester mir irgendwelche Nonsens-Termine in den Kalender ein ...

Verfasst: 16.03.2007 14:34
von kazwo
Ich find die Modifikation sehr gut, vielen Dank :)

Allerdings hätte ich auch eine kleine Anregung:

Wenn man im ACP die Berechtigungen unter Einstellungen ändert, dann werden die geändert, aber man bekommt keine Benachrichtung, dass sie geändert wurden.
Vielleicht könntest du da noch sowas einbauen, denn das kann ein wenig verwirren, da man schnell denkt es hätte sich nichts getan.

Verfasst: 17.03.2007 10:05
von BB-BF-BM
@Surfer90: ich kann den Fehler nicht nachvollziehen. Aber ansonsten such mal in calendar.php nach

Code: Alles auswählen

$userdata = $db->sql_fetchrow() + $userdata;
und ersetze das mit

Code: Alles auswählen

$userdata = array_merge($userdata, $db->sql_fetchrow());
Am meisten wundert mich das
..kommmt bei der Emailbenachrichtigung bei mir
müsste eigentlich immer auftreten, wenn du deine Einstellungen änderst.

Release: Version 0.2.1

Verfasst: 17.03.2007 12:53
von BB-BF-BM
Version 0.2.1 ist fertig.

Download / Demo wie vorher (siehe 1.Post)

Ich habe dem ersten Beitrag eine ToDo-List hinzugefügt, in der ihr sehen könnt, wie weit der Entwicklungsstand ist.

Wie immer ist Feedback willkommen!

Verfasst: 17.03.2007 14:37
von kazwo
Mir ist eine Sache aufgefallen:

Entweder hast du den Ordner "cache" im Ordner "calendar" vergessen zu erstellen oder vergessen in die install.txt zu schreiben, dass ein Ordner "cache" im Ordner "calendar" erstellt werden muss.

Ansonsten fällt mir nichts auf, super Arbeit und dankeschön.

Verfasst: 17.03.2007 17:22
von BB-BF-BM
Seltsam...
Muss daran liegen, dass der Ordner leer ist...
Ich setz mal standardmäßig eine leere Textdatei in den Ordner, damit das funktioniert...

Verfasst: 17.03.2007 18:34
von Surfer90
BB-BF-BM hat geschrieben:@Surfer90: ich kann den Fehler nicht nachvollziehen. Aber ansonsten such mal in calendar.php nach

Code: Alles auswählen

$userdata = $db->sql_fetchrow() + $userdata;
und ersetze das mit

Code: Alles auswählen

$userdata = array_merge($userdata, $db->sql_fetchrow());
Am meisten wundert mich das
..kommmt bei der Emailbenachrichtigung bei mir
müsste eigentlich immer auftreten, wenn du deine Einstellungen änderst.

Ich hab jetzt erstmal dein Update durchgeführt und nochmal alles getestet.
Jetzt kommt bei mir was neues Lustiges, wenn ich die Einstellung der Benachrichtigung verändere.

Code: Alles auswählen

Der Termin wurde erfolgreich gelöscht!
Also das kommt als Bestätigung, wenn ich auf "Einstellung ändern" klicke..

Wenigstens kein Fehler, aber trotzdem komisch......