[Frage] Wann kommt ein Bugfix in die offizielle TYPO3 Source, wie ist der Ablauf?

  • zappzarapp zappzarap...
    Jedi-Meister
    0 x
    347 Beiträge
    3 Hilfreiche Beiträge
    07. 01. 2015, 15:39

    Hallo,

    ich habe eine Extension gebaut die "intensiv" die Fuktionalität von displayCond im TCA einsetzt. Jetzt bin ich draufgekommen, dass das ganze nicht funktioniert wenn man den l10n_mode auf exclude oder mergeIfNotBlank setzt. Habe dann alternativ probiert es mit subtype_value_field zu machen mit dem selben Problem.

    Ich habe dazu jetzt diesen Eintrag im Bugtracker gefunden:
    https://forge.typo3.org/issues/59130

    Da ist ein Link auf einen Fix vohanden der auf "Needs Code Review" steht
    https://review.typo3.org/#/c/30436/

    Ich habe diesen fix jetzt testweise bei mir lokal in der source eingebaut und damit funktionieren die displayCond Funktionen auch korrekt in übersetzten Datensätzen.

    Ich frage mich jetzt wie es hier jetzt konkret weitergeht, dass das auch in die offizielle TYPO3 Source eingebaut wird immerhin dürfte es diesen fix schon seit 8 Monaten geben ("Updated by Gerrit Code Review 8 months ago" )

    Liebe Grüße,
    Matthias


  • 1
  • kitsunet kitsunet
    Flash Gordon
    0 x
    2559 Beiträge
    27 Hilfreiche Beiträge
    08. 01. 2015, 19:42

    Das kann ich Dir erklären :)

    Und zwar ist dieser Bugfix als WIP (Work in Progress) gekennzeichnet, solange das der Fall ist, wird das normalerweise gar nicht gemerged. Warum der WIP ist müsstest Du den Autor des Changes fragen und ihn vielleicht bitten das fertig zu machen oder das ganze aufzutrennen, so dass der Teil, der funktioniert schon mal gemerged werden kann.

    Danach muss der Change dann Reviews bekommen. D.h. Leute (wie Du) müssen ausprobieren und den Code angucken und mit den Coding Guidelines vergleichen und jeweils (man kann auch nur eins von beidem machen) die Wertung für "verified" (Code funktioniert, das beinhaltet aber auch Tests laufen weiterhin) und "code review" (Code macht Sinn und entspricht Coding Guidelines) vergeben. Haben dann genug bewertet (da weiß ich gerade die Zahlen für den CMS Core nicht, ich glaube zwei Active Contributor mindestens und gerne weitere Reviews von jedem) kann ein Active Contributor entscheiden es zu mergen.

    config.baseURL = http://www.kitsunet.com/
    TYPO3 Flow und Neos Community Contact
    Release Manager TYPO3 Neos 1.1
    Ich habe Probleme mit den PMs hier, also schreibt mir bitte eine Mail oder über Twitter!

  • 1