Ergebnis 1 bis 24 von 24

Thema: 4.13.29: Controller\ContentElement

  1. #1
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard 4.13.29: Controller\ContentElement

    Hi,
    ich versuche eine vorhandene Extension auf 4.x umzustellen und habe ein Problem mit dem ContentElement:

    PHP-Code:
    src/ContaoManager
    src
    /Controller
    src
    /DependencyInjection
    src
    /Module
    src
    /Resources
    src
    /Resources/config
    src
    /Resources/contao
    src
    /Resources/contao/config
    src
    /Resources/contao/dca
    src
    /Resources/contaolanguages
    src
    /Resources/contao/languages/en
    src
    /Resources/contao/models
    src
    /Resources/contao/templates
    src
    /Resources/public
    src/Resources/public/images 
    src/Controller/VoteController.php
    PHP-Code:
    <?php
    declare(strict_types=1);

    /*
     *     Vote Bundle
     *
     *    @copyright    (c) 2023 - 2023 Florian Daeumling, Germany. All right reserved
     *     @license     https://github.com/toteph42/vote/blob/master/LICENSE
     */

    namespace VoteBundle\Controller\ContentElement;

    use 
    Contao\CoreBundle\Controller\ContentElement\AbstractContentElementController;
    use 
    Contao\ContentModel;
    use 
    Contao\Template;
    use 
    Symfony\Component\HttpFoundation\Request;
    use 
    Symfony\Component\HttpFoundation\Response;

    /**
     *  @ContentElement("vote", category="texts", template="ce_votes")
     */
    class VoteController extends AbstractContentElementController {

        public const 
    TYPE 'vote';

         protected function 
    getResponse(Template $templateContentModel $modelRequest $request): Response {

             
    $template->text $model->text;

            return 
    $template->getResponse();
        }

    }

    ?>
    src/Resources/config/services.yaml
    PHP-Code:
    services:
        
    _defaults:
            
    autoconfiguretrue
            
        VoteBundle
    \Controller\ContentElement\VoteController:
            
    tags:
                -
                    
    namecontao.content_element
                    category
    texts 
    src/Resources/contao/dca/
    PHP-Code:
    <?php
    declare(strict_types=1);

    /*
     *     Vote Bundle
     *
     *    @copyright    (c) 2023 - 2023 Florian Daeumling, Germany. All right reserved
     *     @license     https://github.com/toteph42/vote/blob/master/LICENSE
     */

    /**
     * Add a palette to tl_content
     */
    use Contao\Backend;
    use 
    VoteBundle\Controller\ContentElement\VoteController;

    $GLOBALS['TL_DCA']['tl_content']['palettes'][VoteController::TYPE] = '{type_legend},type,headline;{include_legend},'.
            
    'vote,vote_current;{protected_legend:hide},protected;{expert_legend:hide},guests,cssID,space';

    /**
     * Add a field to tl_content
     */
    $GLOBALS['TL_DCA']['tl_content']['fields']['vote'] = [
        
    'label'                   => &$GLOBALS['TL_LANG']['tl_content']['vote'],
        
    'exclude'                 => true,
        
    'inputType'               => 'select',
        
    'options_callback'        => [ 'tl_content_class''getVotes' ],
        
    'eval'                    => [ 'includeBlankOption' => true'chosen' => true'tl_class' => 'w50' ],
        
    'sql'                     => "int(10) unsigned NOT NULL default '0'"
    ];

    $GLOBALS['TL_DCA']['tl_content']['fields']['vote_current'] = [
        
    'label'                   => &$GLOBALS['TL_LANG']['tl_content']['vote_current'],
        
    'exclude'                 => true,
        
    'inputType'               => 'checkbox',
        
    'eval'                    => [ 'tl_class' => 'w50 m12' ],
        
    'sql'                     => "char(1) NOT NULL default ''"
    ];

    /**
     * Provide miscellaneous methods that are used by the data configuration array.
     */
    class tl_content_class extends Backend {

        
    /**
         * Get all votes and return them as array
         * @return array
         */
        
    public function getVotes(): array {

            
    $arr = [];

            
    $obj$this->Database->execute("SELECT id, title FROM tl_vote ORDER BY title");

            while (
    $obj->next())
                
    $arr[$obj->id] = $obj->title;

            return 
    $arr;
        }

    }

    ?>
    src/Resources/contao/languages/en/default.php
    PHP-Code:
    <?php

    /*
     *     Vote Bundle
     *
     *    @copyright    (c) 2023 - 2023 Florian Daeumling, Germany. All right reserved
     *     @license     https://github.com/toteph42/vote/blob/master/LICENSE
     */

    use VoteBundle\Controller\ContentElement\VoteController;

    // Content elements
    $GLOBALS['TL_LANG']['CTE'][VoteController::TYPE] = [ 'Vote''Displays a voting.' ];
    src/Resources/contao/templates/ce_votes.html5
    PHP-Code:
    <div class="<?php echo $this->class?><?php if ($this->featured): ?> featured<?php endif; ?> block"<?php echo $this->cssID?><?php if ($this->style): ?> style="<?php echo $this->style?>"<?php endif; ?>>
    <?php if ($this->headline): ?>

    <<?php echo $this->hl?>><?php echo $this->headline?></<?php echo $this->hl?>>
    <?php endif; ?>

    <?php echo $this->vote?>

    </div>
    Code:
    composer install
    php.exe vendor/bin/contao-console debug:fragment
    content_element wird nicht angezeigt. Im Debug Modus kommt dann die Meldung:

    Content element class "" (content element "vote") does not exist

    Kann mir bitte jemand helfen?
    Geändert von toteph42 (04.08.2023 um 16:57 Uhr)
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  2. #2
    Contao-Fan
    Registriert seit
    24.02.2021.
    Beiträge
    984
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Hi,
    ich versuche eine vorhandene Extension auf 4.x umzustellen und habe ein Problem mit dem ContentElement:
    Gibt es einen Link zum Repo, damit man es selbst mal versuchen könnte?
    Jetzt einen Code-Review ohne alle Dateien
    - in einem unbekannten System...
    - könnte dauern
    - kostet viel Zeit
    - müsste man erstmal alles selbst einrichten

    Einfacher wäre es natürlich eine dev-develop bereitzustellen

  3. #3
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Hallo,
    ja gerne: https://github.com/toteph42/vote

    Mir würde ja auch ein Hinweis auf ein - nach neuesten Regeln gebautes - und BE Elemente nutzende Erweiterung schon weiter helfen.... Da könnte ich ja selbst vergleichen, was dort anders ist. Habe aber bisher kein Bundle gefunden.
    Geändert von toteph42 (05.08.2023 um 11:39 Uhr)
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  4. #4
    Contao-Fan
    Registriert seit
    24.02.2021.
    Beiträge
    984
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Hallo,
    ja gerne: https://github.com/toteph42/vote

    Mir würde ja auch ein Hinweis auf ein - nach neuesten Regeln gebautes - und BE Elemente nutzende Erweiterung schon weiter helfen.... Da könnte ich ja selbst vergleichen, was dort anders ist. Habe aber bisher kein Bundle gefunden.
    https://docs.contao.org/dev/framework/content-elements/

    Mit Attributen (Ab PHP 8.0 und Contao 4.13) habe ich schon einige programmiert, schau mal hier:
    https://github.com/oveleon/contao-ba...Controller.php
    https://github.com/contao-thememanag...Controller.php


    Habe deinen Fehler gefunden

    - Dein Bundle heißt "VoteBundle" - Deine DependencyInjection-Datei heißt VotingExtension.php, muss aber VoteExtension.php heißen

    - In deiner DependencyInjection lädst du eine services.yml, deine Datei heißt aber services.yaml.

    - Deine services.yaml sollte so aussehen (/app gibt es nicht):
    Code:
    services:
        _defaults:
            autoconfigure: true
    
        VoteBundle\Controller\ContentElement\VoteController:
            tags:
                -
                    name: contao.content_element
                    category: texts
    In deinem VoteController kannst du auch das use Statement für die ServiceAnnotation hinzufügen:
    PHP-Code:
    use Contao\CoreBundle\ServiceAnnotation\ContentElement
    Geändert von zoglo (05.08.2023 um 12:43 Uhr)

  5. #5
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Vielen Dank - ich werde es ausprobieren!
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  6. #6
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Hallo Zogglo,

    erst einmal habe ich die Erweiterung noch mehr an die neuen Standards angepasst (language und weitere Kleinigkeiten) und dann umbenannt: https://github.com/toteph42/votes

    Im Backend erscheint jetzt nicht wie gewünscht ##VOTES##, sondern nur die Nummer des Datensatzes in der 'tl_votes'. Im Frontend erscheint auch nur die Datensatznummer. Da helfen auch leider die Links auf deine Erweiterungen nicht weiter....

    Würdest du freundlicher Weise nochmal schauen, was ich falsch mache?
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  7. #7
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Im Backend erscheint jetzt nicht wie gewünscht ##VOTES##, sondern nur die Nummer des Datensatzes in der 'tl_votes'. Im Frontend erscheint auch nur die Datensatznummer.
    Was genau meinst du damit? Zeige Screenshots.
    » sponsor me via GitHub or PayPal or Revolut

  8. #8
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Unabhängig von deinem aktuellen Problem: dein contao Ordner ist am falschen Ort. Entweder du nutzt src/Resources/contao/ - oder contao/ im Root, wenn du dem aktuellen Symfony Standard folgst: https://github.com/inspiredminds/con...le.php#L19-L22
    » sponsor me via GitHub or PayPal or Revolut

  9. #9
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Darüberhinaus stimmt die Registrierung deiner Frontendmodule nicht. Du hast den falschen FQCN angegeben: https://github.com/toteph42/votes/bl...ig.php#L34-L35
    » sponsor me via GitHub or PayPal or Revolut

  10. #10
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Die schließenden ?> solltest du auch aus PHP Dateien entfernen. Siehe https://www.php-fig.org/psr/psr-2/#22-files
    » sponsor me via GitHub or PayPal or Revolut

  11. #11
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Hallo,
    vielen Dank!

    Ich habe alle Änderungen vorgenommen. Allerdings funktioniert es immer noch nicht. Im Anhang die Bildschirmfotos der Umfrage, des BE und des FE. Updates sind in Github verfügbar.
    Angehängte Grafiken Angehängte Grafiken
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  12. #12
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Für die "ContentElement" Annotation hast du das use Statement vergessen: https://github.com/toteph42/votes/bl...roller.php#L20
    Das kannst du danach löschen: https://github.com/toteph42/votes/bl...ig.php#L38-L41
    » sponsor me via GitHub or PayPal or Revolut

  13. #13
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Hallo,
    habe alles eingebaut und geändert. Funktioniert immer noch nicht. Also alles auf Null setzen und Contao neu installieren. Geht immer noch nicht, aber jetzt bekomme ich die Meldung
    Code:
    Class Contao\ManagerPlugin\PluginLoader located in xxx/vendor/contao/manager-plugin/src\Resources\PluginLoader.php does not comply with psr-4 autoloading standard. Skipping.
    Composer 2.5.8
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  14. #14
    Contao-Fan
    Registriert seit
    24.02.2021.
    Beiträge
    984
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Hallo Zogglo,

    erst einmal habe ich die Erweiterung noch mehr an die neuen Standards angepasst (language und weitere Kleinigkeiten) und dann umbenannt: https://github.com/toteph42/votes

    Im Backend erscheint jetzt nicht wie gewünscht ##VOTES##, sondern nur die Nummer des Datensatzes in der 'tl_votes'. Im Frontend erscheint auch nur die Datensatznummer. Da helfen auch leider die Links auf deine Erweiterungen nicht weiter....

    Würdest du freundlicher Weise nochmal schauen, was ich falsch mache?

    Wenn du etwas änderst, dann sind meine Erklärungen nicht mehr zu gebrauchen, da wie Spooky schon erwähnt hat, der FQCN nicht passt + du evtl. einem anderen PSR-Standard folgst.
    Ein Code-Review kann ich dir hier nicht geben, die Symfony-Dokumentation und das Abschauen von anderen Plugins (Hier wurden schon 4 genannt) wird dir hier weiter helfen.

  15. #15
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Schade. Es ist wohl offensichtlich, dass ich kein Contao-/Symphony-Spezialist bin. Ich versuche trotzdem dem - an sich - tollen Projekt etwas beizutragen.

    1. Es gibt keine Dokumentation, der ich folgen könnte, um die Erweiterung zum laufen zu bringen (zumindest nicht mit meinen Anforderungen).
    2. Die erwähnten Beispiele implementieren leider nicht die von mir gewünschte Funktion.
    3. Beim Contao Manager habe ich EXAKT mich an deinen Beispielen gehalten. Funktioniert aber trotzdem nicht.
    4. Es ist toll, wenn ihr auf Regeln zu FQCN (https://www.php-fig.org/psr/psr-4/) verweist. Die Beschreibung des Standards habe ich ebenfalls versucht genau zu folgen. 'Toteph42\\VoteBundle\\ContaoManager\\Plugin' entspricht exakt dem 'namespace Toteph42\VoteBundle\ContaoManager;'. Was daran falsch ist, kann ich nicht mehr nachvollziehen...
    5. Ich dachte das Forum wäre für "helping hand", aber wenn dem nicht so ist, dann tut es mir leid, Euch belästigt zu haben.


    Ich erwarte keinen Code-Review. Ich dachte in meinem kleinen Hirn nur,, es gäbe jemanden, der mich auf Beispiele verweisen kann, anhand dessen ich das Bundle programmieren könnte. Dem scheint ja leider nicht so zu sein. Also alle Zeit, die ich investiert habe für die Tonne. Sehr schade. Ich dachte eigentlich Contao wäre ein tolles CMS. Wenn aber Erweiterungen so kompliziert zu programmieren sind, dann muss ich wohl nach den vielen Jahren,den ich Contao treu war, mich einem anderen CMS zu wenden. Das ist wirklich schade.

    Einmal davon abgesehen, dass ihr durch die Verkomplizierung immer mehr Leute abschrecken werdet, die einen kleinen Beitrag liefern wollen...

    Sorry Euch belästigt zu haben...
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  16. #16
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Hallo,
    habe alles eingebaut und geändert. Funktioniert immer noch nicht. Also alles auf Null setzen und Contao neu installieren. Geht immer noch nicht
    Eine Neuinstallation von Contao wird nichts bringen. Als Programmierer*in musst du einfach die Probleme Schritt für Schritt analysieren und beheben - und nicht einfach mit "Funktioniert immer noch nicht" abtun.


    Zitat Zitat von toteph42 Beitrag anzeigen
    aber jetzt bekomme ich die Meldung
    Code:
    Class Contao\ManagerPlugin\PluginLoader located in xxx/vendor/contao/manager-plugin/src\Resources\PluginLoader.php does not comply with psr-4 autoloading standard. Skipping.
    Composer 2.5.8
    Das ist nur eine Warnung, die kannst du ignorieren.
    » sponsor me via GitHub or PayPal or Revolut

  17. #17
    Contao-Fan
    Registriert seit
    24.02.2021.
    Beiträge
    984
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen

    1. Ich dachte das Forum wäre für "helping hand", aber wenn dem nicht so ist, dann tut es mir leid, Euch belästigt zu haben.


    Also alle Zeit, die ich investiert habe für die Tonne.
    Die selbe Zeit habe ich am Wochenende (60 Minuten für die Recherche) auch investiert, meine Lösung hatte geholfen, hier hast du aber wieder alles abgeändert und fragst also nochmal nach.
    Meine Zeit war also auch für die Tonne.

    Was ich damit sagen möchte ist:
    Wenn du Schritt für Schritt eines der genannten Plugins kopierst... und einfach nur die Namen abänderst... dann würde das klappen.

    Wir helfen gerne, die Basis muss aber stimmen.. und bei Plugins reicht nicht immer nur ein Screenshot oder Code aus (Wie in deinem Beispiel erwähnt, ich konnte den Fehler erst entdecken, nachdem ich den gesamten Code gesehen habe... welcher teils sogar nicht mehr mit dem im ersten Post genannten Code übereinstimmte)

    Wenn du nach deiner Frage schon wieder 30 Sachen am Code verändert hast, wir davon nichts wissen... aber schon Zeit investiert haben... dann war die Antwort für die Tonne.
    Was ich dir vorschlagen könnte ist einfach nur ein Pull-Request wo ich deinen Code halb auf Vordermann bringe (Mehr als 10 Minuten möchte ich damit aber nicht mehr verbringen, da man auch mal entspannen möchte),
    eine Erklärung "warum es falsch ist" kann ich dir dann aber an den ganzen Stellen nicht geben, da es deutlich länger dauern würde -> Damit meine ich, dass ich dir kein Code-Review geben kann.

    Bedenke, dass es auch meine freie Zeit ist, welche ich hier für die Community opfere, es dann aber nicht geschätzt wird und man sofort so eine lange "Ja keiner hilft mir"-Antwort ins Gesicht gedrückt bekomme.
    Geändert von zoglo (08.08.2023 um 15:28 Uhr)

  18. #18
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Es geht um das Thema Migration. Leider finde ich dazu relativ wenig Dokumentation.

    Ich habe mir das https://packagist.org/packages/codefog/contao-polls Bundle vorgenommen, da es im Prinzip dass macht, was ich will. Aber eben eine Funktion fehlt. Ich habe dabei gleich versucht die 4.13-Regeln zu Extensions zu implementieren (und natürlich einige Fehler und Depreciated behoben).

    Dummer Weise hatte ich Probleme mit dem Service. Goggle hat nicht weiter geholfen. Deshalb dieser Post. Ich hatte gehofft, ein Spezialist könnte mir gleich die Lösung sagen. War aber eben nix. Nach reichlich raten habe ich dann die Kategorie dann doch erraten
    Code:
    category: includes
    - nur so erscheint die Extension als Content Element.

    Nun ist es dann doch vollbracht: https://github.com/toteph42/voting
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  19. #19
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Warum kochst du lieber dein eigenes Süppchen, als bspw. einen Fork + PR zu machen?
    » sponsor me via GitHub or PayPal or Revolut

  20. #20
    Contao-Fan
    Registriert seit
    24.02.2021.
    Beiträge
    984
    Contao-Projekt unterstützen

    Support Contao

    Standard

    Zitat Zitat von Spooky Beitrag anzeigen
    Warum kochst du lieber dein eigenes Süppchen, als bspw. einen Fork + PR zu machen?
    Passt doch. Ist Open Source und jeder ist hierbei frei, was er machen möchte. Die Lizenz im vorherigen Bundle erlaubt es doch.
    Ein PR hilft immer, damit man zusammenarbeiten kann, so hat jeder was davon.
    Manchmal sind aber die eigenen Interessen in der Hinsicht anders, sodass du doch lieber deinen Fork nutzt, anstatt einen PR zu machen.

    Alles legitim, die Lizenz darf halt nur nicht mehr geändert werden... sonst verstößt man hier gegen andere Sachen :P

  21. #21
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Ja, legitim ist es auf jeden Fall. Habe mich nur gewundert.
    » sponsor me via GitHub or PayPal or Revolut

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

    Standard

    wie Spooky schrieb, legitim ist es allemal aber ob sinnig ist (für mich) die Frage

    Aus meiner Sicht sind die Repos primär die Aufforderung zur Mitarbeit an dem jeweiligen Projekt und nicht unbedingt die "Diversifizierung" in zig Abspaltungen.

    Warum? Zwei Gründe:

    1.) Weil es für andere User schwer zu durchblicken ist, wenn es zu einem Thema x-verschiedene, mehr oder wenig ähnliche, Erweiterungen gibt - mir sind 100 gut gepflegte Erweiterungen lieber als marketing-schreierische 10.000 von denen 99% man in die Tonne kloppen kann...

    2.) Wenn von Erweiterung des Users A der B einen Fork macht und Feature X einbaut, User C auch einen Fork zieht und Bug xy fixt, User D von B einen Fork macht um das mit Contao N kompatibel zu machen... wer setzt sich dann mal hin und zieht das alles zusammen? In der Erweiterungsliste hat man damit natürlich mehr "Auswahl"... - siehe 1.

  23. #23
    Contao-Nutzer Avatar von toteph42
    Registriert seit
    14.03.2023.
    Beiträge
    27

    Standard

    Hallo, ich bin ja prinzipiell bei Euch. Da ich aber an allen Ecken und Enden den Code umgeschrieben habe und zudem auch die beiden FE-Module weg gelassen habe, war ich mir nicht sicher, ob dass überhaupt Sinn macht, einen PR los zu schicken. Ich kenne mich ehrlich kaum aus mit PR. Wenn ich den jetzt los schicke, dann habe ich verstanden, dass der Entwickler entscheidet, ob er meine Änderungen akzeptiert. Und dann? Übernimmt er alle Änderungen, dann werde ich benachrichtigt und lösche meinen Fork? Und wenn er nicht reagiert / die nicht annimmt, was dann? Bleibt es dann bei zwei Bundles?
    __________________________________________
    Live long and prosper
    Florian

    Spenden GitHub

  24. #24
    Community-Moderator
    Wandelndes Contao-Lexikon
    Avatar von Spooky
    Registriert seit
    12.04.2012.
    Ort
    Scotland
    Beiträge
    35.517
    Partner-ID
    10107

    Standard

    Zitat Zitat von toteph42 Beitrag anzeigen
    Da ich aber an allen Ecken und Enden den Code umgeschrieben habe und zudem auch die beiden FE-Module weg gelassen habe, war ich mir nicht sicher, ob dass überhaupt Sinn macht, einen PR los zu schicken.
    Wenn das so viele grundsätzliche Änderungen waren, dann ist ein eigenes Package vermutlich sinnvoller, ja.
    » sponsor me via GitHub or PayPal or Revolut

Aktive Benutzer

Aktive Benutzer

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

Berechtigungen

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