Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Load after CalendarBundle instead of CoreBundle #8

Merged
merged 1 commit into from
Mar 3, 2018

Conversation

bauer00
Copy link

@bauer00 bauer00 commented Mar 2, 2018

This extension extends the CalendarBundle functionality, so it has to be loaded after the CalendarBundle.
I've had a problem where the DCA changes hadn't any effect, because the DCA of the CalendarBundle wasn't loaded. (Contao 4.4.14)

This extension extends the CalendarBundle functionality, so it has to be loaded after the CalendarBundle.
I've had a problem where the DCA changes hadn't any effect, because the DCA of the CalendarBundle wasn't loaded. (Contao 4.4.14)
@bauer00
Copy link
Author

bauer00 commented Mar 2, 2018

Das wird wahrscheinlich auch das erwähnte Problem aus dem Pull request #7 beheben.

@Jaaaan
Copy link

Jaaaan commented Mar 2, 2018

Das löst auch das Problem aus #7 .
Unter PHP 7.1 läuft das Bundle einwandfrei, mit 7.2 bekommt man beim Import von CSV/.ics-Dateien folgende Warnung:
Warning: Use of undefined constant ENCODE_AMPERSANDS - assumed 'ENCODE_AMPERSANDS' (this will throw an Error in a future version of PHP) in /vendor/craffft/contao-calendar-ical-bundle/src/Resources/contao/classes/CalendarImport.php on line 242 & 1000

@iCodr8 iCodr8 merged commit a6e5e91 into Craffft:master Mar 3, 2018
@bauer00
Copy link
Author

bauer00 commented Mar 3, 2018

Die Warnung kommt davon, da eine undefinierte Konstante "ENCODE_AMPERSANDS" verwendet wird. Dies ist wohl bis PHP 7.2 kein Problem, aber wird wohl in späteren Versionen einen Error produzieren.
Die Funktion ampersand erwartet als 2. Parameter einen Boolean welcher bestimmt ob die Funktion die ampersands decoded oder encoded.
Ich kann die Konstante nirgens finden, evtl. gabs die mal in einer älteren Contao-Version. Die Konstante sollte denke ich durch den korrekten Boolean ersetzt werden, oder entfernt werden, falls true gewünscht ist.

@bauer00 bauer00 deleted the patch-1 branch March 3, 2018 00:15
@Jaaaan Jaaaan mentioned this pull request Mar 3, 2018
@Jaaaan
Copy link

Jaaaan commented Mar 3, 2018

Ggf. #9 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants