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 - 15: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 (912)
am 17.08.2014 - 16: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 - 16: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 (912)
am 17.08.2014 - 16: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 (912)
am 17.08.2014 - 16: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 - 14: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 (912)
am 19.10.2014 - 17: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 - 22:19 Uhr

werde gerne Tests lostreten.

CU,
Sándor

  • Anmelden oder Registrieren um Kommentare zu schreiben

Patch needs work (still)

Eingetragen von C_Logemann (912)
am 19.10.2014 - 22: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

  • für drupal11 ein Slider Modul
  • [gelöst] W3CSS Paragraphs Views
  • Drupal 11 neu aufsetzen und Bereiche aus 10 importieren
  • Wie erlaubt man neuen Benutzern auf die Resetseite zugreifen zu dürfen.
  • [gelöst] Anzeigeformat Text mit Bild in einem Artikel, Drupal 11
  • Social Media Buttons um Insteragram erweitern
  • Nach Installation der neuesten D10-Version kein Zugriff auf Website
  • Composer nach Umzug
  • [gelöst] Taxonomie Begriffe zeigt nicht alle Nodes an
  • Drupal 11 + Experience Builder (Canvas) + Layout Builder
  • Welche KI verwendet ihr?
  • Update Manger läst sich nicht Installieren
Weiter

Neue Kommentare

  • melde mich mal wieder, da ich
    vor 2 Wochen 2 Stunden
  • Hey danke
    vor 2 Wochen 20 Stunden
  • Update: jetzt gibt's ein
    vor 2 Wochen 1 Tag
  • Hallo, im Prinzip habe ich
    vor 2 Wochen 6 Tagen
  • Da scheint die Terminologie
    vor 2 Wochen 6 Tagen
  • Kannst doch auch alles direkt
    vor 3 Wochen 3 Tagen
  • In der entsprechenden View
    vor 3 Wochen 3 Tagen
  • Dazu müsstest Du vermutlich
    vor 3 Wochen 3 Tagen
  • gelöst
    vor 5 Wochen 6 Tagen
  • Ja natürlich. Dass ist etwas,
    vor 6 Wochen 21 Stunden

Statistik

Beiträge im Forum: 250233
Registrierte User: 20451

Neue User:

  • ByteScrapers
  • Mroppoofpaync
  • 4aficiona2

» Alle User anzeigen

User nach Punkten sortiert:
wla9461
stBorchert6003
quiptime4972
Tobias Bähr4019
bv3924
ronald3857
md3717
Thoor3678
Alexander Langer3416
Exterior2903
» User nach Punkten
Zur Zeit sind 0 User und 28 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