Re: [Beta] Kalender
Verfasst: 16.08.2015 14:53
Wunderbar.
Groß Zeit investieren musst du denke ich nicht.
Ich würde dir den Listener schreiben, aber du müsstest den natürlich mit in deine Zip Archive aufnehmen, denn sonst funktioniert die Koppelung nicht. Einen Eintrag in die service.yml braucht das Ding natürlich auch. Um die Wartung einfach zu halten würde ich den Listener in eine eigene Datei stecken, ebenso wie dazugehörige Sprachvariablen. Wenn du inkompatible Änderungen an der DB vornimmst müsste der Listener entweder von dir mit angepasst werden oder aber du sagst mir dann in dem Fall Bescheid. Wenn du jetzt schon weißt, dass du die DB Schnittstelle noch ändern willst/musst, dann würde ich mit der Erstellung des Listeners auch noch warten (werde ich denke ich sowieso, ich habe auch noch andere Dinge zu erledigen).
Wenn die Koppelung auch andersrum funktionieren soll (also sprich: Für einen Kalendereintrag soll ein neuer Termin gefunden werden), dann brauchen wir zusätzlich einen Event Trigger in deiner Extension, aber dafür habe ich mir deine Extension noch nicht genau genug angesehen und für dich wäre das dann auch nur ein Zweizeiler, der in irgend eine Datei aufgenommen werden müsste.
Wo wir gerade dabei sind: Es gibt da ein kleines Sicherheitsproblem in deiner Extension das du evtl. beheben solltest:
Im includes Ordner gibt es Dateien die sofort Code ausführen, aber nicht mit beginnen. Das ist normalerweise keine gute Idee.
Ich habe mir außerdem mal erlaubt den Extension Prevalidator über die Extension laufen zu lassen. Hier ist die Ausgabe:
Groß Zeit investieren musst du denke ich nicht.
Ich würde dir den Listener schreiben, aber du müsstest den natürlich mit in deine Zip Archive aufnehmen, denn sonst funktioniert die Koppelung nicht. Einen Eintrag in die service.yml braucht das Ding natürlich auch. Um die Wartung einfach zu halten würde ich den Listener in eine eigene Datei stecken, ebenso wie dazugehörige Sprachvariablen. Wenn du inkompatible Änderungen an der DB vornimmst müsste der Listener entweder von dir mit angepasst werden oder aber du sagst mir dann in dem Fall Bescheid. Wenn du jetzt schon weißt, dass du die DB Schnittstelle noch ändern willst/musst, dann würde ich mit der Erstellung des Listeners auch noch warten (werde ich denke ich sowieso, ich habe auch noch andere Dinge zu erledigen).
Wenn die Koppelung auch andersrum funktionieren soll (also sprich: Für einen Kalendereintrag soll ein neuer Termin gefunden werden), dann brauchen wir zusätzlich einen Event Trigger in deiner Extension, aber dafür habe ich mir deine Extension noch nicht genau genug angesehen und für dich wäre das dann auch nur ein Zweizeiler, der in irgend eine Datei aufgenommen werden müsste.
Wo wir gerade dabei sind: Es gibt da ein kleines Sicherheitsproblem in deiner Extension das du evtl. beheben solltest:
Im includes Ordner gibt es Dateien die sofort Code ausführen, aber nicht mit
Code: Alles auswählen
if(!defined('IN_PHPBB')
{
exit;
}Ich habe mir außerdem mal erlaubt den Extension Prevalidator über die Extension laufen zu lassen. Hier ist die Ausgabe:
Ich finde die Warnungen eigentlich wichtiger als die "Fatal error" Einträge.Running Extension Pre Validatoron directory /Daten/Downloads/Software/web/calendar_0.4.3.
Running tests.
Validation: FAILED
Fatal: 7, Error: 0, Warning: 6, Notice: 0
Test results for extension:
Warning: Can't detect the file type for calendar_event.zip, handling it as a binary file.
Warning: Can't detect the file type for feiertag.sql, handling it as a binary file.
Warning: Found a deprecated or removed function call to set_config on line 55, you can use $config->set($key, $value, $cache = true) instead in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\acp\main_info.php
Warning: The route name should start with vendor_namespace (which is hjw_calendar) but started with hjw_main_controller in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\config\routing.yml
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\includes\calendar_event.php
Warning: IN_PHPBB is not defined in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\includes\calendar_event.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\includes\special_days.php
Warning: IN_PHPBB is not defined in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\includes\special_days.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\language\de\calendar.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\language\de_x_sie\calendar.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\language\en\calendar.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\language\fr\calendar.php
Fatal error: Detected windows style newlines instead of UNIX newlines in D:\Daten\Downloads\Software\web\calendar_0.4.3\hjw\calendar\language\fr\info_acp_calendar.php
