[2.11 RC2] Controller.php remove article_raster_designer

[mae]

Seit mehreren Versionen befindet sich in der Controller.php ein nutzloser HOOK:


Controller.php, Zeile 273

            // HOOK: trigger the article_raster_designer extension
            elseif (in_array('article_raster_designer', $this->Config->getActiveModules()))
            {
                return RasterDesigner::load($objPage->id, $strColumn);
            } 

Dieser sollte dokumentiert oder entfernt werden.

[alex]

Dann sei doch so gut und melde das bei github (https://github.com/contao/core/issues).

[mae]

Dann sei doch so gut und melde das bei github (https://github.com/contao/core/issues).

Ich bin seit Jahren Mitglied auf Gitorious

[alex]

Ich bin seit Jahren Mitglied auf Gitorious - ich denke nicht daran, zu gitnub zu wechseln...

Nun, dort ist aber nun mal die Stelle, wo Fehler und Features für den Core gemeldet werden. Und von wechseln habe ich auch nicht gesprochen....!

[mae]

Der Hook ist nicht umsonst. Das ist ein privater Hook für einen Extension Entwickler. Wer das ist weiß ich nicht, aber es macht einen Sinn dne zu entfernen wenn der noch verwendet wird.

Moment, dass ist jetzt wohl nicht im Sinne eines Community-Developments.

Sprich, jeder kann sich hier nach belieben Hooks dazu wünschen ?

Es gibt keine Extension innerhalb der Repo, der diesen Hook verwendet.

Er hätte sich den Hook auch dazu schreiben können, das wäre zwar Pfusch, aber Core-Hacking via Extension sollte auch gelernt sein (ja, es ist der gleiche User...).

[BugBuster]

Es gibt keine Extension innerhalb der Repo, der diesen Hook verwendet.

Hmm, der kam mit der 2.6.2 dazu. Keine Ahnung ob der noch gebraucht wird.
https://github.com/contao/core/issues/166

[andreasisaak]

Lasst uns bitte mal aus dieser persönlichen Ebene rauskommen. Das nervt tierisch und lässt neue User denken das wir alle hirnlose Affen sind.

Zum HOOK - ich bin auch schon öfter auf diesen gestossen und finde es bedenklich das Hooks im Core enthalten sind, die Extensions unterstützen die nicht mehr existieren oder nie im als "frei" im ER veröffentlicht wurden. Imho sollte der User der damaligen Anfrage angeschrieben werden und dann sollte der Hook entfernt werden.

Oder man entfernt ihn im Zuge von Contao 3 einfach gleich mit. Wenn er gebraucht wird, kann sich der Dev ja melden!!

[mae]

Hmm, der kam mit der 2.6.2 dazu. Keine Ahnung ob der noch gebraucht wird.
https://github.com/contao/core/issues/166

Danke -das ist das einzige Snippet, welches nicht der regulären Contao Syntax entspricht.
Es ist nur einseitig nutzbar und zwar wenn man den ungefähren Inhalt der Klasse `RasterDesigner` kennt - was ja nicht der Fall ist.

Der Hook wird scheinbar nur für private Zwecke verwendet und kann daher auch manuell nachgetragen werden.

[mae]

Lasst uns bitte mal aus dieser persönlichen Ebene rauskommen. Das nervt tierisch und lässt neue User denken das wir alle hirnlose Affen sind.

Zum HOOK - ich bin auch schon öfter auf diesen gestossen und finde es bedenklich das Hooks im Core enthalten sind, die Extensions unterstützen die nicht mehr existieren oder nie im als "frei" im ER veröffentlicht wurden. Imho sollte der User der damaligen Anfrage angeschrieben werden und dann sollte der Hook entfernt werden.

Oder man entfernt ihn im Zuge von Contao 3 einfach gleich mit. Wenn er gebraucht wird, kann sich der Dev ja melden!!

Genau - ich finde es sogar im Allgemeinen sehr bedenklich, Hooks für irgendwelche Erweiterungen ein zu richten.

Sollte nicht ein Nutzen im globalen Scope im Vordergrund stehen?

[Toflar]

Meine Güte, was ist das den jetzt wieder?

Da ist ein Hook drin, der vor einer geschätzten Ewigkeit eingebaut wurde, weil Leo damals keine Möglichkeit gesehen hat, eine allgemeine Lösung für das Problem anzubieten. Na und? Ist doch super? Das heisst wir kriegen Hooks, wenn wir sie benötigen. Sogar wenn wir einen individuellen Hook brauchen.
Dieser Hook wird niemanden töten, versprochen!
Macht doch aus einer Mücke keinen Elefanten. Mein Gott.

Bitte Thread schliessen oder noch lieber, gleich entfernen. Sowas möchte ich hier nicht lesen.
Glen hat Leo ja darauf hingewiesen (Danke Glen)

[MiTsch]

DoNotFeedTroll.png

Ja, bitte den Thread schließen !

[andreasisaak]

DoNotFeedTroll.png

Ja, bitte den Thread schließen !

Und nur weil ich anderer Meinung bin als Toflar bin ich automatisch ein Troll? Ich bitte dich diese Kinderkacke sein zu lassen und dich entweder nicht einzumischen oder was sinnvolles beizutragen. Das der Hook niemanden umbringt ist mir wohl bewusst aber für einen sauberen Core kann man über eine Reinigung in Contao 3 nachdenken!

[Nina]

Entspannt euch bitte etwas. Es bringt doch niemandem etwas, wenn man gleich persönlich wird und sich an die Gurgel geht.

Wenn es sich wirklich um einen Hook handelt, der nicht mehr benötigt wird oder sonst irgendwie problematisch ist, könnt ihr ja in Github darauf hinweisen (bzw. habt das glaub ich schon getan). Sich hier gegenseitig anzugiften oder pauschal gegen alles zu schießen bringt auf jeden Fall nichts. Wir haben doch alle das gleiche Ziel - ein gutes CMS - also diskutiert bitte respektvoll und konstruktiv miteinander.

Danke

[MiTsch]

@andreas
Hallo andreas,

solltest Du meinen Beitrag auf Dich bezogen haben, so war das nicht meine Absicht und das was ich damit zum Ausdruck bringen wollte ist dann leider falsch angekommen. Dies tut mir leid! Auf Grund deiner Beiträge hier und das was Du in die Community einbringst schätze ich dich sehr.

Der Kontext in dem mein Beitrag steht ist eigentlich ein anderer. Ich nenn es dann mal ganz offen beim Namen. Nachdem folgender Beitrag von mae an dem Punkt angekommen war wo er sagt: Es sollte Dank genug sein, dass ich mich überhaupt mit Contao beschäftige. empfand ich den ganzen Beitrag von Anfang an als angelegte Provokation.

Und nachdem diese nicht mehr funktionierte da Alex in weiser deeskalierender Vorraussicht den Thread schloß, eröffnete mae diesen Thread hier unmittelbar danach. Und ich denke das auch dieser Beitrag hier wieder die gleiche Motivation mit sich führt.

Ich denke da wie Nina: "Wir haben doch alle das gleiche Ziel - ein gutes CMS - also diskutiert bitte respektvoll und konstruktiv miteinander."

Ich maße mir nicht an zu entscheiden ob ein Hook sinnvoll ist oder nicht, dazu fehlt mir der Einblick in die Thematik und äußere mich auch nicht wenn ich nichts zum Thema Beitragen kann sondern lese dann nur interessiert still mit.

Meine Zustimmung zu Toflar begründet sich in seiner Aussage: "Bitte Thread schliessen."
Mir gefällt der Gedanke der hinter einer Community steckt und ich versuche das zurück zu geben was ich durch sie erhalte, insofern ärgert es mich schon wenn ein konstruktives und respektvolles miteinander gestört wird.

Viele Grüße
MiTsch

[andreas.schempp]

Ich hab den Thread eben entdeckt. Ich weis woher der Hook kommt, und "wem er gehört". Er ist aktuell auch in Verwendung (eine Contao 2.9 Installation, scheint also aktualisiert zu werden), daher darf er nicht ohne Ersatz gestrichen.

[BugBuster]

Dann schreib das mal hier dazu, damit Leo auch bescheid weiß.
https://github.com/contao/core/issues/166