Merge pull request #385 from linuxfrorg/add_validations

Add extra validations for links, trackers; and news

Author: Oumph

app/controllers/bookmarks_controller.rb

@@ -35,7 +35,9 @@ def create
       redirect_to [@bookmark.owner, @bookmark], notice: "Votre lien a bien été partagé"
     else
       @bookmark.node = Node.new(user_id: current_user.id, cc_licensed: false)
-      @bookmark.node.preview_tags = params[:tags]
+      if params.include?(:tags)
+        @bookmark.node.preview_tags = params[:tags]
+      end
       @bookmark.valid?
       flash.now[:alert] = "Votre lien semble invalide. Le confimez‑vous ?" unless @bookmark.link =~ /\A#{URI::regexp(['http', 'https'])}\z/
       render :new

app/controllers/diaries_controller.rb

@@ -33,7 +33,9 @@ def create
       redirect_to [@diary.owner, @diary], notice: "Votre journal a bien été créé"
     else
       @diary.node = Node.new(user_id: current_user.id, cc_licensed: @diary.cc_licensed)
-      @diary.node.preview_tags = params[:tags]
+      if params.include?(:tags)
+        @diary.node.preview_tags = params[:tags]
+      end
       @diary.valid?
       flash.now[:alert] = "Votre journal ne contient pas de liens. Confirmez‑vous que cela est normal ?" unless @diary.body =~ /<a /
       render :new

app/controllers/news_controller.rb

@@ -46,7 +46,9 @@ def create
       redirect_to news_index_url, notice: "Votre proposition de dépêche a bien été soumise, et sera modérée dans les heures ou les jours à venir"
     else
       @news.node = Node.new(cc_licensed: @news.cc_licensed)
-      @news.node.preview_tags = params[:tags]
+      if params.include?(:tags)
+        @news.node.preview_tags = params[:tags]
+      end
       @news.valid?
       render :new
     end

app/controllers/posts_controller.rb

@@ -24,7 +24,9 @@ def create
       redirect_to forum_posts_url(forum_id: @post.forum), notice: "Votre message a bien été créé"
     else
       @post.node = Node.new
-      @post.node.preview_tags = params[:tags]
+      if params.include?(:tags)
+        @post.node.preview_tags = params[:tags]
+      end
       @post.valid?
       render :new
     end

app/controllers/wiki_pages_controller.rb

@@ -33,7 +33,9 @@ def show
 
   def new
     @wiki_page = WikiPage.new
-    @wiki_page.title = params[:title]
+    if params.include?(:title)
+      @wiki_page.title = params[:title]
+    end
     return not_enough_karma('pages de wiki') unless @wiki_page.creatable_by?(current_account)
     enforce_create_permission(@wiki_page)
   end

app/models/bookmark.rb

@@ -24,14 +24,17 @@ class Bookmark < Content
   belongs_to :owner, class_name: 'User'
 
   validates :title,     presence: { message: "Le titre est obligatoire" },
-                        length: { maximum: 100, message: "Le titre est trop long" }
+                        length: { maximum: 160, message: "Le titre est trop long" },
+                        uniqueness: { message: "Un lien avec le même titre a déjà été proposé" }
   validates :link, presence: { message: "Vous ne pouvez pas poster un lien vide" },
                    http_url: { message: "Le lien n'est pas valide" },
-                   length: { maximum: 255, message: "Le lien est trop long" }
+                   length: { maximum: 255, message: "Le lien est trop long" },
+                   uniqueness: { message: "Cette adresse de lien a déjà été proposée" }
+  validates :lang, inclusion: { in: Lang.valid_codes, allow_nil: false, message: "La langue du lien doit être définie" }
 
   def link=(raw)
     raw.strip!
-    return write_attribute :url, nil if raw.blank?
+    return write_attribute :link, nil if raw.blank?
     uri = URI.parse(raw)
     # Default to HTTP link if neither scheme nor host is found
     if uri.scheme.blank? && uri.host.blank?
@@ -44,6 +47,11 @@ def link=(raw)
     write_attribute :link, raw
   end
 
+  def title=(raw)
+    raw.strip!
+    write_attribute :title, raw
+  end
+
   def create_node(attrs={})
     attrs[:cc_licensed] = false
     super

app/models/comment.rb

@@ -50,6 +50,11 @@ def wikify_body
     self.body = wikify(wiki_body, nofollow: nofollow)
   end
 
+  def title=(raw)
+    raw.strip!
+    write_attribute :title, raw
+  end
+
 ### Reading status ###
 
   # Returns true if this comment has been read by the given user,

app/models/diary.rb

@@ -27,7 +27,7 @@ class Diary < Content
   belongs_to :converted_news, class_name: 'News'
 
   validates :title,     presence: { message: "Le titre est obligatoire" },
-                        length: { maximum: 100, message: "Le titre est trop long" }
+                        length: { maximum: 160, message: "Le titre est trop long" }
   validates :wiki_body, presence: { message: "Vous ne pouvez pas poster un journal vide" }
 
   validate :convert_only_cc_licensed_diary, on: :convert

app/models/forum.rb

@@ -31,6 +31,11 @@ class Forum < ActiveRecord::Base
   extend FriendlyId
   friendly_id
 
+  def title=(raw)
+    raw.strip!
+    write_attribute :title, raw
+  end
+
 ### Workflow ###
 
   state_machine :state, initial: :active do

app/models/lang.rb

@@ -8,6 +8,10 @@ def self.all
     vals.zip(keys)
   end
 
+  def self.valid_codes
+    $redis.lrange("lang", 0, -1) - ["xx"]
+  end
+
   def self.[]=(key, value)
     $redis.set("lang/#{key}", value)
     $redis.rpush("lang", key)