diff options
| author | Teddy Wing | 2018-01-23 14:45:47 +0100 | 
|---|---|---|
| committer | Zog | 2018-01-25 17:18:01 +0100 | 
| commit | 4e034f42107a2016db3a032d3c0cd1a5f8e43635 (patch) | |
| tree | 77876552ea31d590e6c8f483cff0726399d2f1af /lib | |
| parent | 5ef7e7c47e9639af6e76cac4a520909ef36a148d (diff) | |
| download | chouette-core-4e034f42107a2016db3a032d3c0cd1a5f8e43635.tar.bz2 | |
AF83::Decorator::Link: Change #class? to #default_class
Instead of checking for an existing class and then setting the class,
provide a method to set a default class when one hasn't been defined.
This does effectively the same thing but moves the condition inside the
method.
Thanks Johan for the suggestion!
Refs #5586
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/af83/decorator/link.rb | 7 | 
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/af83/decorator/link.rb b/lib/af83/decorator/link.rb index 10e6091aa..fd50069fe 100644 --- a/lib/af83/decorator/link.rb +++ b/lib/af83/decorator/link.rb @@ -25,8 +25,11 @@ class AF83::Decorator::Link      link_class args    end -  def class? -    @options[:link_class] && !@options[:link_class].empty? +  def default_class *args +    has_class = @options[:link_class] && !@options[:link_class].empty? +    return if has_class + +    self.class args    end    def method_missing name, *args, &block  | 
