app/models/landable/template.rb in landable-1.13.1 vs app/models/landable/template.rb in landable-1.13.2

@@ -4,36 +4,35 @@
     include Landable::Librarian
 
     # attributes
     attr_accessor :temp_author
 
-    validates_presence_of   :name, :slug, :description
+    validates_presence_of :name, :slug, :description
     validates_uniqueness_of :name, case_sensitive: false
     validates_uniqueness_of :slug, case_sensitive: false
 
     before_save :slug_has_no_spaces
 
-
     belongs_to :published_revision,   class_name: 'Landable::TemplateRevision'
-    has_many   :audits,               class_name: 'Landable::Audit', as: :auditable
-    has_many   :revisions,            class_name: 'Landable::TemplateRevision'
+    has_many :audits,               class_name: 'Landable::Audit', as: :auditable
+    has_many :revisions,            class_name: 'Landable::TemplateRevision'
 
     has_and_belongs_to_many :pages,   join_table: Page.templates_join_table_name
 
     delegate :count, to: :pages, prefix: true # Returns how many Pages a Template lives in!
 
-    before_save -> template {
+    before_save lambda  { |template|
       template.is_publishable = true unless template.published_revision_id_changed?
     }
 
     def deactivate
-      publish!(author_id: temp_author.id, notes: "This template has been trashed")
+      publish!(author_id: temp_author.id, notes: 'This template has been trashed')
 
       super
     end
 
-    def name= val
+    def name=(val)
       self[:name] = val
       self[:slug] ||= (val && val.underscore.gsub(/[^\w_]/, '_').gsub(/_{2,}/, '_'))
     end
 
     def partial?
@@ -50,11 +49,11 @@
         republish_associated_pages(options)
       end
     end
 
     def republish_associated_pages(options)
-      options[:template] = self.name
+      options[:template] = name
       pages.each do |page|
         page.republish!(options) if page.published?
       end
     end
 
@@ -66,12 +65,11 @@
 
       save!
     end
 
     def slug_has_no_spaces
-      if self.slug =~ /\s/ # check if whitespace
-        self.slug = self.slug.underscore.gsub(/[^\w_]/, '_').gsub(/_{2,}/, '_')
-      end
+      return unless slug =~ /\s/ # check if whitespace
+      self.slug = slug.underscore.gsub(/[^\w_]/, '_').gsub(/_{2,}/, '_')
     end
 
     class << self
       def create_from_partials!
         Partial.all.map(&:to_template)