Convert more 'setup do' to 'def setup' for consistency

[rails.git] / app / models / acl.rb

diff --git a/app/models/acl.rb b/app/models/acl.rb
index 2db7fb76522a0342324d235367ae51d0c5932c97..529ccbe3bc26640dfb057674aca5b433ec1582d3 100644 (file)
--- a/app/models/acl.rb
+++ b/app/models/acl.rb
@@ -1,17 +1,23 @@
 class Acl < ActiveRecord::Base
 class Acl < ActiveRecord::Base

+  validates :k, :presence => true
+

   def self.match(address, domain = nil)
     if domain
   def self.match(address, domain = nil)
     if domain

-      condition = Acl.where("address >>= ? OR domain = ?", address, domain)
+      Acl.where("address >>= ? OR domain = ?", address, domain)

     else
     else

-      condition = Acl.where("address >>= ?", address)
+      Acl.where("address >>= ?", address)

     end
   end
 
   def self.no_account_creation(address, domain = nil)
     end
   end
 
   def self.no_account_creation(address, domain = nil)

-    self.match(address, domain).where(:k => "no_account_creation").exists?
+    match(address, domain).where(:k => "no_account_creation").exists?
+  end
+
+  def self.no_note_comment(address, domain = nil)
+    match(address, domain).where(:k => "no_note_comment").exists?

   end
 
   def self.no_trace_download(address, domain = nil)
   end
 
   def self.no_trace_download(address, domain = nil)

-    self.match(address, domain).where(:k => "no_trace_download").exists?
+    match(address, domain).where(:k => "no_trace_download").exists?

   end
 end
   end
 end