Überschreiben Sie Schienen Helfer mit Zugang zum original

Will ich mit rails' vertraut Helfer, aber mit leicht veränderten Funktionalität. Ich sehe das so, ich möchte in der Lage sein, etwas zu tun:

module AwesomeHelper
  #... create alias of stylesheet_link_tag to old_stylesheet_link_tag
  def stylesheet_link_tag(*args)
    if @be_awesome
      awesome_stylesheet_link_tag *args
    else
      old_stylesheet_link_tag *args
    end
  end
end

So wie ich es sehe, habe ich drei Optionen:

Monkey-patching: Wiedereröffnung der rails-helper-Modul. Wenn die rails-team jemals ändern Sie die Namen Ihrer Helfer-Modul, mein code wird zu einer Quelle der Sprödigkeit. Nicht unüberwindbar, aber nicht ideal.
Verwenden unterschiedliche Methodennamen: Versuchen zu kleben, um die common-Rail-Schnittstelle kann mein Untergang. Meine änderungen kann eine Quelle der Verwirrung für andere Entwickler
Abnehmen-Methoden (neu): Nicht sicher, ob dies funktionieren würde, oder ob es die gleichen Nachteile wie 1. Wird diese Forschung, aber dies könnte ein guter Ausgangspunkt.

Also die Frage hier ist, bin ich stecken mit einem dieser sub-optimale Lösungen, oder gibt es eine andere Möglichkeit, die ich noch nicht berücksichtigt? Wenn ich mich für option 3, gibt es einen Weg, es zu tun, ohne die direkte Ansprache der rails-helper-Modul?

(Hinweis: ich habe entfernt den Kontext an, denn es fügt nichts zu der Frage.)

InformationsquelleAutor user208769 | 2012-05-06

34

Gibt es einen besseren Weg als alle Ihre aufgeführten Optionen. Verwenden Sie einfach super:
```
module AwesomeHelper
  def stylesheet_link_tag(*sources)
    if @be_awesome
      awesome_stylesheet_link_tag *sources
    else
      super
    end
  end
end
```
Überschreiben stylesheet_link_tag im AwesomeHelper wird sichergestellt, dass, wenn stylesheet_link_tag wird aufgerufen, Ruby wird ihm begegnen in der Methode lookup-Weg, bevor es trifft ActionView::Helpers::AssetTagHelper. Wenn @be_awesome ist true bekommen Sie in die Hand nehmen und aufhören, die Dinge richtig es ist, und wenn nicht, wird der Anruf zu super ohne Klammern, transparent Durchlaufen alle Argumente bis zu den Schienen der Umsetzung. Auf diese Weise müssen Sie nicht zu befürchten, dass das Rails-core-team bewegt die Dinge um auf Sie!
- Wissen Sie was... das ist so verrückt offensichtlich, ich bin versucht, rack-mein Gehirn, um herauszufinden, warum ich dachte, es würde nicht funktionieren! Ich werde es heute Abend ausprobieren und wenn es funktioniert, werde ich mit strengen Worten, mit meinem Gehirn, wahrscheinlich mit einer Mauer. Nach, natürlich, akzeptieren Sie Ihre Antwort... 😀
- Hehehe. Das ist genial. So gut ich verstehen, überschreiben von Methoden, die in dieser Art und Weise ist in der Regel vorzuziehen, in jeder Situation. Klasse#Vorfahren kann sehr hilfreich sein, herauszufinden, einen guten Platz in der Methode lookup-Pfad zu entführen Versand-Methode (oder wo Ihr custom Modul mit dem überschreiben muss enthalten sein, um die beste Wirkung).
- Was? 🙂 Sie haben wohl Witze! Dies ist ein RIESIGES Manko! Ihre Art und Weise haben Sie, auch Sie Helfer in JEDER Klasse mit AssetTagHelper. Die Zeit vergeht und der ein oder andere vergessen zu können, benötigen Sie Ihre patch enthalten sein. Sie gehören einfach AssetTagHelper und beginnen sich zu Fragen: warum wird meine Website sieht jetzt anders??? Es ist gut, wenn Sie und patch-maker sind die gleiche person. Aber was, wenn nicht?
- Ja, Sie sind richtig. Um die Verwendung der code, es muss enthalten sein. 🙂 Ich halte nicht, dass ein "gotcha". Mit AbstractController::Helpers::ClassMethods::helper in ApplicationController kann Hilfe bei der Milderung Ihrer Anliegen.
InformationsquelleAutor Cade
6

Ich nicht dieses Juwel, so werde ich die Antwort, die Sie in einer allgemeineren Art und Weise.

Lassen Sie uns sagen, Sie möchten, melden Sie Anrufe zu link_to Helfer (ja, erfundenes Beispiel, aber es zeigt die Idee). Suche in der API gibt Ihnen zu verstehen, dass link_to im inneren des ActionView::Helpers::UrlHelper Modul. So erstellen Sie eine Datei in Ihrem, sagen wir, config/initializers - Verzeichnis mit folgendem Inhalt:
```
# like in config/initializers/link_to_log.rb
module ActionView::Helpers::UrlHelper

    def link_to_with_log(*args, &block)
        logger.info '**** LINK_TO CALL ***'
        link_to_without_log(*args, &block) # calling the original helper
    end

    alias_method_chain :link_to, :log
end
```
Den Kern dieser Funktionalität -- alias_method_chain (anklickbar). Verwenden Sie es NACH der Definition der Methode xxx_with_feature.
- Ja, dieser Ansatz war es, was ich mit "monkey-patching bestimmten Schienen-Module" -, die gut funktioniert, aber wenn die rails-core-ändern Sie den Namen der module, meinen code bricht. Dies kann nicht eine große Sache sein, aber ich bin neugierig zu sehen, ob es gibt andere Lösungen. Das heißt, vergessen hatte alias_method_chain, danke, dass du mich daran erinnert, dass der!
- P. S: aktualisiert Haben, die Frage zu entfernen, das Juwel Beispiel. Hoffentlich dieses layout ist weniger verwirrend! Danke.
- Risiko ist immer da! Wenn Sie sich sorgen über alias_method_chain dann sollten Sie nicht: es existiert seit der version 1.4.0 (Jahr 2007). Wenn Sie sich sorgen über andere Teile des Programms werden dann sicher anständige tests Berichterstattung.
- Veraltet. stackoverflow.com/questions/3689736/...
InformationsquelleAutor jdoe

Versuchen Sie es mit alias_method:

module AwesomeHelper
  alias_method :original_stylesheet_link_tag, :stylesheet_link_tag

  def stylesheet_link_tag(*sources)
    if @be_awesome
      awesome_stylesheet_link_tag *sources
    else
      original_stylesheet_link_tag *sources
    end
  end
end

InformationsquelleAutor gmile

2

Ich würde wirklich ermutigen, Sie zu erwägen, Ihre option #2, überschreiben das Verhalten der Schienen-Methoden in einer Weise, die offensichtlich für den Anrufer.

Ihre neue Methode genannt werden sollte awesome_stylesheet_link_tag so, die anderen Schienen devs Lesen kann, den code und die Frage stellen "Was ist so toll über das link-tag?".

Als kleine änderung könnte man überschreiben, aber pass in :awesome => true als ein argument, damit Sie zumindest eine Ahnung davon, dass etwas im Gange ist.

Änderung des Verhaltens eine weit verbreitete Methode wie stylesheet_link_tag schafft einen potenziellen künftigen Missverständnis, wo keiner benötigt wird.
- Vielen Dank für den input. Während normalerweise ich Stimme zu, in diesem speziellen Fall denke ich, die Konsistenz ist gerechtfertigt - ich bin dabei verwenden wicked_pdf und habe genau den gleichen code generieren einer PDF-Datei oder eine web-Seite. Während wicked_pdf standardmäßig tut, wie du sagst (wicked_pdf_stylesheet_link_tag), es erfordert zu viel Wiederholung für mich, und ich denke, es ist akzeptabel zu erwarten, dass sich die Funktionalität ändern kann, wenn Sie eine PDF-Datei generieren. Aber Sie machen einen guten Punkt, und ein paar praktische Tipps für mich, vielen Dank.
InformationsquelleAutor austinfromboston

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.