Contao-Camp 2024
Ergebnis 1 bis 28 von 28

Thema: die Erweiterung und PHP 8

  1. #1
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Fehler die Erweiterung und PHP 8

    Hallo @hschottm und ggf. Mitentwickler,

    nachdem die Kombination ContaoManager <-> PHP 7.4 bei meinem Hoster nicht ordentlich funktioniert (7.3 geht), "musste" ich auf PHP 8 wechseln.
    Dabei ist aufgefallen, dass die Erweiterung - ich verwende sie gefühlt "schon immer" (seit Typolight, jetzt mit Contao 4.9) - nicht zu PHP 8 kompatibel ist. Es ist aber dem Anschein nach leicht, sie anzupassen:
    Code:
    $ diff CalendarImport.php.orig CalendarImport.php
    39c39
    <         $arrCalendars = $this->Database->prepare("SELECT id FROM tl_calendar WHERE id IN (" . join($arrCalendars, ',') . ") AND ical_source = ?")
    ---
    >           $arrCalendars = $this->Database->prepare("SELECT id FROM tl_calendar WHERE id IN (" . join(',', $arrCalendars) . ") AND ical_source = ?")
    839c839
    <                             $arrFields['cep_participants'] = join($attendees, ',');
    ---
    >                             $arrFields['cep_participants'] = join(',', $attendees);
    854c854
    <                             $arrFields['cep_contact'] = join($contacts, ',');
    ---
    >                             $arrFields['cep_contact'] = join(',', $contacts);
    und
    Code:
    $ diff ContentICal.php.orig ContentICal.php      
    94c94
    <         $this->Template->href = $this->addToUrl("ical=" . join($arrCalendars, ',') . "&title=" . urlencode($this->strTitle));
    ---
    >         $this->Template->href = $this->addToUrl("ical=" . join(',', $arrCalendars) . "&title=" . urlencode($this->strTitle));
    Mehr Stellen gibt es wohl nicht.
    (Ich habe, damit das diff "hübscher" ist, noch einen Umbruch in Zeile 39 entfernt.)
    Die Funktion join() ist ein Alias auf implode(). Ich deute das so: die Parameter dürfen bereits seit PHP4 vertauscht sein und müssen es seit PHP8; siehe https://www.php.net/manual/de/function.implode.php

    Viele Grüße
    purzel

  2. #2
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    Zitat Zitat von purzel Beitrag anzeigen
    Es ist aber dem Anschein nach leicht, sie anzupassen:
    Viele Grüße
    purzel

    @purzel: warum macht Du nicht gleich einen PR?!?

  3. #3
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Äh... sorry, was ist ein PR?

  4. #4
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    sorry - ein Pull-Request auf Github... also Deine Änderungswünsche direkt zum Übernehmen in das Projekt

  5. #5
    Contao-Urgestein Avatar von fiedsch
    Registriert seit
    09.07.2009.
    Ort
    München
    Beiträge
    2.935

    Standard

    Die grundlegende Beschreibung zu PRs findest Du in der Contao-Dokumantation. Das Prinzip ist das gleiche, auch, wenn es dort um die Texte der Dokumentation geht:

    https://docs.contao.org/manual/de/be...quest-erzeugen
    Contao-Community-Treff Bayern: http://www.contao-bayern.de

  6. #6
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    oder guck mal hier https://youtu.be/RvJPMeENSLY ab ~18:20 ... ist didaktisch nicht ganz ausgefeilt, aber man sieht das Prinzip

  7. #7
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Ah, danke für die "Erleuchtung"
    Man braucht offenbar einen GitHub-Account. Ich habe keinen und würde den vermutlich auch im Anschluss nie wieder brauchen.
    Würde diese paar Zeilen bitte jemand von Euch da rein schieben? Ich könnte auch @hschottm eine PN schicken...

  8. #8
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    Zitat Zitat von purzel Beitrag anzeigen
    Man braucht offenbar einen GitHub-Account. Ich habe keinen und würde den vermutlich auch im Anschluss nie wieder brauchen.
    den Github-Account braucht man "immer"! ;-)

  9. #9
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Aber... GitHub ist doch Microsoft... Muss ich?
    Ich benutze noch nicht einmal Windoze

  10. #10
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    ich bin der Meinung, wenn man auch nur annähernd was isn Richtung "Quelltext" macht - damit ist auch Templating eingeschlossen - braucht man einen Github-Account...

    Wie will man ein Ticket erstellen oder kommentieren, Handbuch korrigieren/ergänzen, PR einreichen... oder über Github eine Spende an Contao machen ;-)

  11. #11
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Grmpf, so ein sch***!

    Ich musste extra einen anderen Browser verwenden.

    Jetzt habe ich einen Account bei einem Microsoft Dienst *würg*
    Geändert von purzel (05.01.2022 um 15:02 Uhr)

  12. #12
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Ich bin wohl inkompatibel zu GitHub. Einen Pull-Request habe ich nicht hinbekommen, habe nun ein Issue erstellt.

  13. #13

  14. #14
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Ich schätze https://github.com/hschottm/calendar_ical - zumindest habe ich es da versucht

    {später}
    Ob es besser gewesen wäre, https://github.com/Craffft/contao-calendar-ical-bundle zu nehmen? Siehe https://github.com/hschottm/calendar_ical/issues/10
    Da hab' ich aber das selbe Problem: bin zu blöd ein PR zu erstellen
    Geändert von purzel (05.01.2022 um 16:25 Uhr)

  15. #15
    Contao-Urgestein Avatar von fiedsch
    Registriert seit
    09.07.2009.
    Ort
    München
    Beiträge
    2.935

    Standard

    Welches Repository? Hängt davon ab, was bei Dir installiert ist (siehe Deine composer.json).

    Wenn Du uns nicht sagst, was Deine Probleme beim Erstellen eines PR sind, dann können wir Dir nicht helfen.
    Contao-Community-Treff Bayern: http://www.contao-bayern.de

  16. #16

  17. #17
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Zitat Zitat von fiedsch Beitrag anzeigen
    Welches Repository? Hängt davon ab, was bei Dir installiert ist (siehe Deine composer.json).
    "craffft/contao-calendar-ical-bundle": "^4.2"

    Zitat Zitat von fiedsch Beitrag anzeigen
    Wenn Du uns nicht sagst, was Deine Probleme beim Erstellen eines PR sind, dann können wir Dir nicht helfen.
    Mein Problem mit dem PR ist, dass ich (bleiben wir mal bei dem hschottm-Repo - bei dem Anderen, was ja wohl "nur" ein fork ist, ist es das selbe Problem)
    1. oben auf "Pull Requests (2)" klicke
    2. dann auf den grünen Button rechts "New Pull Request"
    3. und dann nicht weiß was ich machen muss

    Ich habe keine Idee, was mir "Choose different branches or forks above to discuss and review changes" sagen will. Im Grunde will ich mich aber auch nicht auch noch in GitHub einarbeiten (müssen) - ich bin kein Entwickler, habe den Fehler nur durch eine Fehlermeldung im Contao gefunden, ein bisschen recherchiert - und hatte Glück, dass anscheinend nur die drei^h^h^h^hvier o.g. Stellen geändert werden müssen damit es wieder funktioniert.

    Zitat Zitat von zonky Beitrag anzeigen
    im Prinzip richtig - genauer wäre das https://github.com/hschottm/calendar_ical/issues/17
    Genau, das hab ich geschrieben (waldschrat = purzel, weil der Name purzel schon vergeben war)

  18. #18
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    * Fork erzeugen und dorthin wechseln (hast Du gemacht)
    * Edit: einen Branch erzeugen - siehe Screenshot Clipboard03.jpg
    * im Tab "< > Code" zu der Datei gehen, die geändert werden soll und auf die Datei klicken
    * oben rechts ist ein Stift für den Bearbeitungsmodus => Klick
    * Datei bearbeiten - ggf. auch den Header aktualisieren (z.B. Jahreszahl anpassen oder sich als Author eintragen)
    * unten Titel und Kommentar eingeben und speichern (Bei Kommentar ist es prima die Nummer des Issues mit anzugeben ein "#<nr>" reicht für die Verlinkung)
    * es erscheint dann automatisch ein Gelb hinterlegtes Feld + grüner Button, ob Du einen PR anlegen möchtest => Klick auf Button
    * im Original gucken, ob bei Tab "Pull requests" Deiner mit eingetragen ist...

    bei größeren Änderungen sollte man das eher lokal machen - von Gitlab gibt es ein einfaches Tool oder Sourcetree als GUI

  19. #19
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Zuerst: Danke für die Anleitung!

    Puh, das war eine Geburt - Hoffentlich richtig gemacht.

    Ein Fork hatte ich noch nicht erzeugt - das waren Andere. Jetzt aber, sonst fehlte mir nämlich das "Create Branch" Ding.
    Das mit Header aktualisieren habe ich nicht gefunden.
    Es hat einen Moment gedauert, bis ich gepeilt habe, dass "commit" für Speichern steht. Seufz.
    Aber ich glaube, nun steht's drin - im Craffft repo (welches ja auch in meiner composer.json gelistet ist)

    guxtu hier: https://github.com/Craffft/contao-ca...bundle/pull/27

  20. #20
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    sieht doch ganz gut aus!

  21. #21
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Danke

    Irgendwie doof, das seh' ich jetzt erst: auf https://github.com/Craffft/contao-calendar-ical-bundle steht unter About:
    NOT MAINTAINED: Contao extension calendar_ical
    Das könnte bedeuten, dass sich da niemand wirklich drum kümmert oder?

    Im Grunde finde ich das (im Moment noch) nicht schlimm, denn ich habe in meiner Installation ja meine Patches eingebaut

  22. #22
    AG Pressearbeit
    Community-Moderator
    Buchautor 'Contao für Webdesigner'
    Avatar von planepix
    Registriert seit
    05.06.2009.
    Ort
    Stuttgart
    Beiträge
    6.464
    Partner-ID
    107
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Ich habe mir nicht alle Unterschiede angesehen; es gibt noch diese Erweiterung von 1up mit ical-Export:
    https://github.com/1up-lab/contao-ical-export
    ---------------------------------
    Beste Grüße planepix
    Contao für Webdesigner (Website), Twitter: @contaowebdesign
    weitzeldesign
    Contao-Sprechstunde
    Contao Schulungen: https://www.weitzeldesign.com/cms-co...chulungen.html
    Contao Jahrbuch: www.contao-jahrbuch.de
    Contao Agenturtag: www.contao-agenturtag.de
    Contao Stammtisch Stuttgart: www.contao-stammtisch-stuttgart.de
    Contao 4 Erfahrungen als Gitbook: https://app.gitbook.com/@planepix/s/...-mit-contao-4/
    Contao 4 & Manager Hosterhinweise: https://github.com/contao/contao-manager/wiki

    Schon wieder ein Update?
    Glücklich sind die, die den Wert erkennen – und wertschätzen.
    „Muss man machen wie beim Zahnarzt. Der bestraft einen auch mit hohen Rechnungen wenn man die Pflege vernachlässigt.”

  23. #23
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    Zitat Zitat von purzel Beitrag anzeigen
    Irgendwie doof, das seh' ich jetzt erst: auf https://github.com/Craffft/contao-calendar-ical-bundle steht unter About:
    NOT MAINTAINED: Contao extension calendar_ical
    Das könnte bedeuten, dass sich da niemand wirklich drum kümmert oder?
    hab ich auch gesehen - ggf. hat er den Fork nur aufgemacht, da sein PR zu Contao 4 seit Sommer 2017 einstaubt... https://github.com/hschottm/calendar_ical/pull/11

  24. #24
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    Zitat Zitat von planepix Beitrag anzeigen
    Ich habe mir nicht alle Unterschiede angesehen; es gibt noch diese Erweiterung von 1up mit ical-Export:
    https://github.com/1up-lab/contao-ical-export
    da haben wir wierdermal das Phänomen, dass diejenigen, die gut programmieren können, in der Regel keine Readme/Doku schreiben können ;-)

  25. #25
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard

    Zitat Zitat von planepix Beitrag anzeigen
    Ich habe mir nicht alle Unterschiede angesehen; es gibt noch diese Erweiterung von 1up mit ical-Export:
    https://github.com/1up-lab/contao-ical-export
    Kann die Import? Export ist mir "völlig latte" wie man so schön sagt (mein' ich nicht böse!). Ich brauche nur die Import-Funktion (und die üblicherweise nur einmal im Jahr für ~100+ Events)
    Irgendwann kam bei der von mir genutzten Erweiterung (*) die Unterstützung für CSV dazu, was das Generieren der Eventliste für mich (und vielleicht Andere) drastisch erleichtert.

    (*) ob ich ganz früher mal die "Urversion" von hschottm genutzt habe weiß ich nicht mehr, benutze das Ding wie gesagt schon seit Typolight

    Zitat Zitat von zonky Beitrag anzeigen
    hab ich auch gesehen - ggf. hat er den Fork nur aufgemacht, da sein PR zu Contao 4 seit Sommer 2017 einstaubt... https://github.com/hschottm/calendar_ical/pull/11
    Also ist iCodr8 = Craffft ? Oder verstehe ich schon wieder was bei Github nicht bzw. falsch?

  26. #26
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    Zitat Zitat von purzel Beitrag anzeigen
    Also ist iCodr8 = Craffft ?
    davon gehe ich mal aus, der-waldschrat

  27. #27
    Contao-Fan
    Registriert seit
    26.03.2010.
    Beiträge
    327

    Standard


    Zitat Zitat von purzel Beitrag anzeigen
    (waldschrat = purzel, weil der Name purzel schon vergeben war)
    Jetzt: gitpurzel

  28. #28
    Contao-Urgestein Avatar von zonky
    Registriert seit
    19.03.2010.
    Ort
    Berlin, Rdf
    Beiträge
    9.682
    User beschenken
    Wunschliste

    Standard

    ich bin dort deshalb auch zonky2

Aktive Benutzer

Aktive Benutzer

Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)

Lesezeichen

Lesezeichen

Berechtigungen

  • Neue Themen erstellen: Nein
  • Themen beantworten: Nein
  • Anhänge hochladen: Nein
  • Beiträge bearbeiten: Nein
  •