Startseite
  • » Home
  • » Handbuch & FAQ
  • » Forum
  • » Übersetzungsserver
  • » Suche
Startseite › Forum › Drupalcenter.de › Module › Modul-Entwicklung ›

Wie einen funktionierenden Patch ins Release bekommen?

Eingetragen von sry (8)
am 17.08.2014 - 16:09 Uhr in
  • Modul-Entwicklung
  • Drupal 7.x

Servus,

im Modul Realname konnte man in 6.x die Sichtbarkeit von der Berechtigung abhängig machen, in 7.x nicht (vgl. auch http://www.drupalcenter.de/node/39134 )
Inzwischen war die Maintainerin von Realname 6.x so nett, das in einem (gut funktionierenden) Patch umzusetzen, aber er wird nicht übernommen: https://www.drupal.org/node/2068195
Was muss/kann ich tun, um das zu erreichen? Kann mir vielleicht ein Contributer aus dem Kreis hier weiterhelfen?

TIA,
Sándor

‹ Settings-Page für ein Modul erzeugen hook_cron und hook_menu in einem Modul verwenden ›
  • Anmelden oder Registrieren um Kommentare zu schreiben

Code muss wohl noch weiter entwickelt werden

Eingetragen von C_Logemann (914)
am 17.08.2014 - 17:46 Uhr

Ich habe mal kurz in den Thread geschaut: Du hast einen Feature Request gestellt und die Haupt-Maintainerin des Moduls (NancyDru) hat die Funktion portiert, aber erst in einem ersten Schritt. Ein anderer der Maintainer hat an der Lösung etwas bemängelt, was NacncyDru sogar bestätigt hat. Ok, Du hast den Patch getestet, das ist schon mal hilfreich für die Maintainer, aber offensichtlich hat noch niemand anderes die Zeit gefunden weiter am Code zu arbeiten. Du hast ja sogar schon Donation angeboten, was auch ein Weg ist weiter zu kommen. Du kannst außer nett nachfragen bei diesen drei Maintainern evtl. einen anderen Programmierer bitten und evtl. für seinen Aufwand bezahlen, den Code weiter zu entwickeln. Aber unterm Strich haben im Moment erstmal nur alle drei Maintainer, die schon in der Issue aktiv sind, die Berechtigung, Code zum Modul zu committen, also einen Patch einzupflegen.

Funktionalität ist übrigens für verantwortungsvolle Maintainer nicht das einzige Kriterium zur Übernahme von Code. Und es spricht für die Qualität dieses Moduls, daß sich die Haupt-Maintainerin nicht einfach über die Bedenken der anderen Maintainer hinweg setzt.

Update: Ich habe mal Rückfragen in der Issue gestellt und den Status zu "needs work" geändert, da die Maintainer dies wohl wollen, wie ich "zwischen den Zeilen" gelesen habe. Auf jeden Fall kommt die Issue damit erstmal auch in deren Listen wieder nach oben.

Update 2: Übrigens landet ein Patch der committed wird nicht automatisch gleich in einem Release. Er wird erstmal Teil der Dev-Version. Daraus wird dann evtl. früher oder später ein Release erzeugt.

  • Anmelden oder Registrieren um Kommentare zu schreiben

Danke schonmal...

Eingetragen von sry (8)
am 17.08.2014 - 17:45 Uhr

...für die Erläuterungen, und dass Du auf drupal.org in den Thread eingestiegen bist.

CU,
Sándor

  • Anmelden oder Registrieren um Kommentare zu schreiben

Verhalten der Maintainer nicht eindeutig in diesem Fall

Eingetragen von C_Logemann (914)
am 17.08.2014 - 17:50 Uhr
sry schrieb

...für die Erläuterungen, und dass Du auf drupal.org in den Thread eingestiegen bist.

Ich finde das Verhalten der Maintainer in diesem Fall auch nicht eindeutig. Nicht, daß es Aufgabe der Maintainer ist das selbst zu machen, aber wenn Sie Verbesserungen wünschen, sollten sie meiner Meinung nach auch den Status der Issue entsprechend ändern. Dies habe ich dann wiederum mit meinem Kommentar angesprochen.

  • Anmelden oder Registrieren um Kommentare zu schreiben

Mist, ein Doppel-Post

Eingetragen von C_Logemann (914)
am 17.08.2014 - 17:53 Uhr

Oh man, jetzt mir das auch mal passiert mit einem Doppel-Post.

Dann nutze ich das mal zu einem kleinen Hinweis: Leute, wenn Ihr Änderungen auf der Berechtigungs-Seite oder der Module-Seite speichert, hütet Euch vor mehrmaligen Drücken des Senden-Buttons. Das kann großes Chaos in Eurer Konfiguration nach sich ziehen.

  • Anmelden oder Registrieren um Kommentare zu schreiben

Leider Stillstand

Eingetragen von sry (8)
am 19.10.2014 - 15:05 Uhr

Servus,

außer dass ein "willzzz" den Status auf "Needs Review" geändert hat, blieb alles wie es war.

Carsten Logemann schrieb

Du kannst außer nett nachfragen bei diesen drei Maintainern evtl. einen anderen Programmierer bitten und evtl. für seinen Aufwand bezahlen, den Code weiter zu entwickeln.

Nett nachgefragt habe ich IMO durchaus, passiert aber nix. Kannst Du mir sagen, wie die Alternative "einen anderen Programmierer bitten..." läuft, v. a. im Hinblick darauf, dass wohl nur die Maintainer etwas committen dürfen?

TIA,
Sándor

  • Anmelden oder Registrieren um Kommentare zu schreiben

Patch überarbeitet

Eingetragen von C_Logemann (914)
am 19.10.2014 - 18:10 Uhr

Jetzt kannst Du auch noch mal testen und vllt. andere motivieren auch Tests durchzuführen.

  • Anmelden oder Registrieren um Kommentare zu schreiben

Danke schonmal

Eingetragen von sry (8)
am 19.10.2014 - 23:19 Uhr

werde gerne Tests lostreten.

CU,
Sándor

  • Anmelden oder Registrieren um Kommentare zu schreiben

Patch needs work (still)

Eingetragen von C_Logemann (914)
am 19.10.2014 - 23:24 Uhr

Nun, einer der Maintainer möchte gerne noch ein paar Verbesserungen. Dazu gehören nun auch Tests (was er auch schon mal früher hätte erwähnen können). Diese Tests sind zwar grundsätzlich sehr sinnvoll aber mindestens so aufwendig wie die bisherige Arbeit, die schon von den anderen in diesen Patch geflossen sind. Ich will hier gerade nicht versprechen, daß ich das so bald liefern kann in meiner Freizeit.

  • Anmelden oder Registrieren um Kommentare zu schreiben

Benutzeranmeldung

  • Registrieren
  • Neues Passwort anfordern

Aktive Forenthemen

  • Drupal CMS installieren
  • [erledigt]MP3 in Drupal 10 einbinden
  • (gelöst)Drupal 11 installieren
  • Titel ausblenden
  • Ich brauche dringen Hilfe zu Updates oder ggf. wwie geht Composer?
  • Dynamische Ansicht von Seiteninhalt (als Tabelle?)
  • Vergabe von Berechtigungen für bestimmte Rollen; mir fehlt der Haken bzw. das „Veröffentlicht“
  • Medien und andere Daten mit Feeds von Drupal 7 auf Drupal 10 migrieren
  • Rolle erstellen nicht zu finden
  • für drupal11 ein Slider Modul
  • [gelöst] W3CSS Paragraphs Views
  • Drupal 11 neu aufsetzen und Bereiche aus 10 importieren
Weiter

Neue Kommentare

  • Verwende doch das Tag dafür,
    vor 1 Woche 1 Stunde
  • Guckst du hier: step by step
    vor 6 Tagen 15 Stunden
  • Guckst du hier: step by step
    vor 6 Tagen 15 Stunden
  • Ich habe ja keine Angst vor
    vor 2 Wochen 2 Tagen
  • Ist grundsätzlichmachbar – aber nichts für „einfach mal schnell“
    vor 2 Wochen 4 Tagen
  • Vielen Dank erst einmal, aber
    vor 2 Wochen 6 Tagen
  • Du hast die "Trusted host
    vor 2 Wochen 6 Tagen
  • Bitte genauer den aktuellen Lösungs-Ansatz beschreiben
    vor 4 Wochen 5 Tagen
  • Git und rsync sind die wichtigsten Werkzeuge
    vor 5 Wochen 1 Tag
  • Arrrrg. Nix Tabelle :-D /*
    vor 9 Wochen 3 Tagen

Statistik

Beiträge im Forum: 250284
Registrierte User: 20500

Neue User:

  • RandallFloop
  • Inga GuAph
  • Robertolix

» Alle User anzeigen

User nach Punkten sortiert:
wla9464
stBorchert6003
quiptime4972
Tobias Bähr4019
bv3924
ronald3857
md3717
Thoor3678
Alexander Langer3416
Exterior2903
» User nach Punkten
Zur Zeit sind 0 User und 15 Gäste online.

Hauptmenü

  • » Home
  • » Handbuch & FAQ
  • » Forum
  • » Übersetzungsserver
  • » Suche

Quicklinks I

  • Infos
  • Drupal Showcase
  • Installation
  • Update
  • Forum
  • Team
  • Verhaltensregeln

Quicklinks II

  • Drupal Jobs
  • FAQ
  • Drupal-Kochbuch
  • Best Practice - Drupal Sites - Guidelines
  • Drupal How To's

Quicklinks III

  • Tipps & Tricks
  • Drupal Theme System
  • Theme Handbuch
  • Leitfaden zur Entwicklung von Modulen

RSS & Twitter

  • Drupal Planet deutsch
  • RSS Feed News
  • RSS Feed Planet
  • Twitter Drupalcenter
Drupalcenter Team | Impressum & Datenschutz | Kontakt
Angetrieben von Drupal | Drupal is a registered trademark of Dries Buytaert.
Drupal Initiative - Drupal Association