Offizielle Vorgehensweise um eine bestehendes Modul zu erweitern
Eingetragen von DiJae (154)am 16.09.2008 - 12:59 Uhr in
Hallo,
ich habe mir das Modul 'Taxonomy Timer' heruntergeladen, welches grundsätzlich meine Anforderungen erfüllt. Allerdings nimmt es als Startdatum für die Laufzeit immer den Zeitpunkt, an dem der Inhalt erstellt wurde. Ich benötige aber die Möglichkeit, das ich wählen kann, welches Datumsfeld als Startzeit dient, d.h. für meinen Inhalt setze ich die Startzeit über ein CCK-Datumsfeld. Nun würde ich daher gerne das Modul entsprechend anpassen bzw. erweitern. Meine Frage: Wie geht man hier vor? Denn ich würde die neue Version dann auch gerne der Community zur Verfügung stellen. Schreibe ich ein neues Modul auf Basis von Taxonomy Timer? Erstelle ich nur einen Patch? Ich müsste bei meiner 'Korrektur' vor allem private Methoden im Modul ändern, die wiederum mittels des Hooks 'node_api' aufgerufen werden. Vielleicht ist dies noch eine nützliche Info.
Gruß
Dirk
- Anmelden oder Registrieren um Kommentare zu schreiben
Hallo, ich weiß nun nicht
am 16.09.2008 - 13:22 Uhr
Hallo,
ich weiß nun nicht ab wann dir ein csv slot für dein Modul zur Verfügung gestellt wird.
Aber ich hatte gute Erfahrungen einfach mal den Maintainer des Moduls an zu schreiben und einen Patch zu machen. :)
Wenn da gar nichts geht, kannst du dir das immer noch überlegen oder wenn die nichts mehr machen, das Modul übernehmen.
---
Viele Grüße,
Kars-T
Arbeit: comm-press
Hobbies: Tower Defense HQ, plamo.de, Blog
Modul erweitern
am 16.09.2008 - 13:22 Uhr
Moin!
Da Du dann doch einige prinzipielle Änderungen am Modul selbst vornehmen musst empfiehlt es sich, einen (grösseren) Patch zu schreiben.
Versuche, Deinen Ansatz so generisch wie möglich zu verwirklichen.
Also:
- als Standardoption die Startzeit auf das Erstelldatum setzen
- falls CCK vorhanden ist bei den Einstellungen eine Auswahl der für den Nodetypen möglichen CCK-Felder anbieten und dann dessen Wert als Startzeit nehmen
Dann kann man das gegebenenfalls auch noch in andere Richtungen erweitern.
hth,
Stefan
Danke für die Hinweise.
am 16.09.2008 - 15:19 Uhr
Danke für die Hinweise. Habe beide Vorschläge aufgegriffen, d.h. habe den Autor erst einmal angeschrieben und wenn er die Erweiterung nicht vorsieht, werde ich sie vornehmen und als Patch zur Verfügung stellen.