From 4b2e927e619b9eba6830bb4566dbd6c15f0d1356 Mon Sep 17 00:00:00 2001
From: Matteo Latini <mtylty@gmail.com>
Date: Fri, 22 Jun 2012 13:11:59 +0200
Subject: [PATCH 01/10] adds coherent RecordNotFound behaviour

---
 app/models/spree/blog.rb                      |  6 ++-
 config/routes.rb                              |  4 +-
 .../posts_controller_helper.rb                |  6 ++-
 .../spree/blogs/blog_integration_test.rb      | 43 +++++++++++++++++++
 test/support/helpers.rb                       |  6 +++
 5 files changed, 61 insertions(+), 4 deletions(-)
 create mode 100644 test/integration/spree/blogs/blog_integration_test.rb

diff --git a/app/models/spree/blog.rb b/app/models/spree/blog.rb
index bca700a..1c4bbef 100644
--- a/app/models/spree/blog.rb
+++ b/app/models/spree/blog.rb
@@ -18,7 +18,11 @@ def self.find_by_permalink!(path)
   end
 
   def self.find_by_permalink(path)
-    find_by_permalink!(path) rescue ActiveRecord::RecordNotFound
+    if path.length < 3
+      raise ActiveRecord::RecordNotFound # pemalink length is at least 3
+    else
+      find_by_permalink!(path) rescue ActiveRecord::RecordNotFound
+    end
   end
   
   def self.to_options
diff --git a/config/routes.rb b/config/routes.rb
index b3028c3..f63479b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -4,7 +4,7 @@
 
     namespace :admin do
       
-      resources :blogs, :constraints => { :id => /[a-z0-9\-\_\/]{3,}/ }
+      resources :blogs, :constraints => { :id => /[a-z0-9\-\_\/]{0,}/ }
       
       resources :posts do 
         resources :images,   :controller => "post_images" do
@@ -20,7 +20,7 @@
       
     end
      
-    constraints :blog_id => /([a-z0-9\-\_\/]{3,})/ do
+    constraints :blog_id => /([a-z0-9\-\_\/]{0,})/ do
       
       constraints(
         :year  => /\d{4}/,
diff --git a/lib/spree_essential_blog/posts_controller_helper.rb b/lib/spree_essential_blog/posts_controller_helper.rb
index 78a108b..c76612b 100644
--- a/lib/spree_essential_blog/posts_controller_helper.rb
+++ b/lib/spree_essential_blog/posts_controller_helper.rb
@@ -24,7 +24,11 @@ def get_tags
   end
 
   def get_blog
-    @blog = Spree::Blog.find_by_permalink!(params[:blog_id])
+    begin
+      @blog = Spree::Blog.find_by_permalink!(params[:blog_id])
+    rescue ActiveRecord::RecordNotFound => e
+      render_404(e)
+    end
   end
   
 end
diff --git a/test/integration/spree/blogs/blog_integration_test.rb b/test/integration/spree/blogs/blog_integration_test.rb
new file mode 100644
index 0000000..8c6e067
--- /dev/null
+++ b/test/integration/spree/blogs/blog_integration_test.rb
@@ -0,0 +1,43 @@
+#! /usr/bin/env ruby
+# encoding: UTF-8
+
+require 'test_helper'
+  
+
+class Spree::Blogs::BlogIntegrationTest < SpreeEssentials::IntegrationCase
+  
+  def setup
+    Spree::Blog.destroy_all
+  end
+   
+  context "with some blogs" do
+    
+    setup do
+      3.times{ |i| Factory.create(:spree_blog, :name => "My blog #{i}") }
+    end
+    
+    should "render 404 if blog is not found" do
+      with_driver(:rack_test) do
+        visit spree.blog_posts_path("does_not_exist")
+        assert page.status_code == 404, "Expected response to be 404"
+      end
+    end
+
+    should "render 404 if blog name is shorter than 3 characters" do
+      Factory.create(:spree_blog, :name => "abc")
+
+      with_driver(:rack_test) do
+        visit spree.blog_posts_path("a")
+        assert page.status_code == 404, "Expected response to be 404"
+
+        visit spree.blog_posts_path("ab")
+        assert page.status_code == 404, "Expected response to be 404"
+
+        visit spree.blog_posts_path("abc")
+        assert page.status_code == 200, "Expected response to be 200"
+      end
+    end
+    
+  end
+
+end
diff --git a/test/support/helpers.rb b/test/support/helpers.rb
index 905cfbe..9fe56b2 100644
--- a/test/support/helpers.rb
+++ b/test/support/helpers.rb
@@ -11,3 +11,9 @@ def setup_action_controller_behaviour(controller_class)
   @routes = Spree::Core::Engine.routes
   @controller = controller_class.new
 end
+
+def with_driver(driver)
+  Capybara.current_driver = driver
+  yield
+  Capybara.use_default_driver
+end

From e4dee3aa0ef1cb02d8087556aada5f8fe5066366 Mon Sep 17 00:00:00 2001
From: Matteo Latini <mtylty@gmail.com>
Date: Fri, 22 Jun 2012 15:01:05 +0200
Subject: [PATCH 02/10] fixes typo

---
 app/views/spree/blogs/admin/posts/show.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/spree/blogs/admin/posts/show.html.erb b/app/views/spree/blogs/admin/posts/show.html.erb
index dd9f8f2..1631b1e 100644
--- a/app/views/spree/blogs/admin/posts/show.html.erb
+++ b/app/views/spree/blogs/admin/posts/show.html.erb
@@ -12,6 +12,6 @@
 <br class="clear" />
 
 <p>
-  <%= link_to_edit @user %> &nbsp;
+  <%= link_to_edit @post %> &nbsp;
   <%= link_to t('back'), collection_url %>
 </p>

From 1c9e0c7a6eb254eb92ca67b5830f4db9b7d0dfb6 Mon Sep 17 00:00:00 2001
From: Matteo Latini <mtylty@gmail.com>
Date: Fri, 22 Jun 2012 15:42:41 +0200
Subject: [PATCH 03/10] adds autodate i18n

---
 app/views/spree/blogs/admin/posts/_form.html.erb | 9 +++++++--
 config/locales/en.yml                            | 6 ++++++
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/app/views/spree/blogs/admin/posts/_form.html.erb b/app/views/spree/blogs/admin/posts/_form.html.erb
index 6580111..3b593fc 100644
--- a/app/views/spree/blogs/admin/posts/_form.html.erb
+++ b/app/views/spree/blogs/admin/posts/_form.html.erb
@@ -12,7 +12,7 @@
 
 <%= form.field_container :posted_at do %>
   <%= form.label :posted_at, t("activerecord.attributes.post.posted_at") %><br />
-  <%= form.text_field :posted_at, :class => 'text', :value => ((l @post.posted_at, :format => :long) unless @post.posted_at.nil?) %><br />
+  <%= form.text_field :posted_at, :class => 'text', :value => (@post.posted_at unless @post.posted_at.nil?) %><br />
   <%= error_message_on :post, :posted_at %>
 <% end %>
 
@@ -36,7 +36,12 @@
   <script type="text/javascript">
   //<![CDATA[
     $(document).ready(function() {
-      $('#post_posted_at').autodate();
+      $('#post_posted_at').autodate({
+        date_format: "<%=t 'autodate.date_format' %>",
+        empty_string: "<%=t 'autodate.empty_string' %>",
+        input_empty: "<%=t 'autodate.input_empty' %>",
+        input_invalid: "<%=t 'autodate.input_invalid' %>"
+      });
       $('#post_body').markItUp(markdownSettings);
     });
   //]]>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 6aeb070..553bedc 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -104,3 +104,9 @@ en:
   date:
     formats:
       month_year: "%B %Y"
+
+  autodate:
+    date_format: "dddd, MMMM d, yyyy h:mm:ss tt"
+    empty_string: "Type a date above"
+    input_empty: "Enter any date here.."
+    input_invalid: "Please enter a valid date."

From f293620873632d154f42035878685a59bf00071b Mon Sep 17 00:00:00 2001
From: Alberto Vena <kennyadsl@gmail.com>
Date: Tue, 26 Jun 2012 11:36:06 +0200
Subject: [PATCH 04/10] adds blogs to menu items

---
 app/models/spree/blog.rb                              | 6 +++++-
 app/overrides/add_blogs_to_menu.rb                    | 5 +++++
 app/views/spree/shared/_blog_main_menu_items.html.erb | 5 +++++
 3 files changed, 15 insertions(+), 1 deletion(-)
 create mode 100644 app/overrides/add_blogs_to_menu.rb
 create mode 100644 app/views/spree/shared/_blog_main_menu_items.html.erb

diff --git a/app/models/spree/blog.rb b/app/models/spree/blog.rb
index 1c4bbef..c5c0eb4 100644
--- a/app/models/spree/blog.rb
+++ b/app/models/spree/blog.rb
@@ -32,7 +32,11 @@ def self.to_options
   def to_param
     self.permalink.gsub(/(^\/+)|(\/+$)/, "")
   end
-  
+
+  def matches?(_path)
+    ( _path.delete("/").match(permalink))
+  end
+
 private
 
   def permalink_availablity
diff --git a/app/overrides/add_blogs_to_menu.rb b/app/overrides/add_blogs_to_menu.rb
new file mode 100644
index 0000000..2d82e3e
--- /dev/null
+++ b/app/overrides/add_blogs_to_menu.rb
@@ -0,0 +1,5 @@
+Deface::Override.new(:virtual_path  => "spree/shared/_store_menu",
+                     :name          => "spree_essential_blog_main_menu_items",
+                     :insert_after  => "#home-link[data-hook]",
+                     :partial       => "spree/shared/blog_main_menu_items",
+                     :disabled      => false)
\ No newline at end of file
diff --git a/app/views/spree/shared/_blog_main_menu_items.html.erb b/app/views/spree/shared/_blog_main_menu_items.html.erb
new file mode 100644
index 0000000..6b1b855
--- /dev/null
+++ b/app/views/spree/shared/_blog_main_menu_items.html.erb
@@ -0,0 +1,5 @@
+<% unless @blogs.blank? %>
+  <% @blogs.each do |blog| %>
+    <%= content_tag(:li, link_to(blog.name, blog_posts_path(blog)), :class => blog.matches?(request.path)  ? "active" : nil) %>
+  <% end %>
+<% end %>

From cbc983d11e83184ad4a113143896d7eecdcc4a0c Mon Sep 17 00:00:00 2001
From: Matteo Latini <mtylty@gmail.com>
Date: Tue, 26 Jun 2012 14:58:38 +0200
Subject: [PATCH 05/10] adds some missing translations

---
 app/views/spree/blogs/admin/blogs/index.html.erb | 2 +-
 app/views/spree/blogs/admin/posts/index.html.erb | 6 +++---
 app/views/spree/blogs/admin/posts/show.html.erb  | 2 +-
 config/locales/en.yml                            | 7 +++++++
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/app/views/spree/blogs/admin/blogs/index.html.erb b/app/views/spree/blogs/admin/blogs/index.html.erb
index 5a49a6e..5c8a337 100644
--- a/app/views/spree/blogs/admin/blogs/index.html.erb
+++ b/app/views/spree/blogs/admin/blogs/index.html.erb
@@ -1,6 +1,6 @@
 <%= render :partial => 'spree/admin/shared/configuration_menu' %>
 
-<h1>Listing Blogs</h1>
+<h1><%=t "spree.blogs.admin.blogs.index.listing" %></h1>
 
 <table class="index">
   <thead>
diff --git a/app/views/spree/blogs/admin/posts/index.html.erb b/app/views/spree/blogs/admin/posts/index.html.erb
index 17ede87..32d82d3 100644
--- a/app/views/spree/blogs/admin/posts/index.html.erb
+++ b/app/views/spree/blogs/admin/posts/index.html.erb
@@ -1,13 +1,13 @@
 <div class='toolbar'>
   <ul class='actions'>
     <li>
-      <p><%= button_link_to "New Post", new_object_url, :icon => 'add' %></p>
+      <p><%= button_link_to t("spree.blogs.admin.posts.new.new_post"), new_object_url, :icon => 'add' %></p>
     </li>
   </ul>
   <br class='clear' />
 </div>
 
-<h1>Listing Posts</h1>
+<h1><%=t "spree.blogs.admin.posts.index.listing" %></h1>
 
 <table class="index">
   <thead>
@@ -25,7 +25,7 @@
       <td><%= link_to post.title, object_url(post) %></td>
       <td><%= post.blog.nil? ? "n/a" : link_to(post.blog.name, admin_posts_path(:search => { :blog_id_equals => post.blog_id })) %></td>
       <td><%= l post.posted_at, :format => :date_time %></td>
-      <td><%= post.live ? 'Yes' : 'No' %></td>
+      <td><%= post.live ? t(:yes) : t(:no) %></td>
       <td>
       	<%= link_to_edit post %> &nbsp;
       	<%= link_to_delete post %>
diff --git a/app/views/spree/blogs/admin/posts/show.html.erb b/app/views/spree/blogs/admin/posts/show.html.erb
index 1631b1e..a49f5d9 100644
--- a/app/views/spree/blogs/admin/posts/show.html.erb
+++ b/app/views/spree/blogs/admin/posts/show.html.erb
@@ -1,5 +1,5 @@
 <%= render :partial => 'spree/blogs/admin/shared/post_tabs', :locals => {:current => "Post Details"} %>
-<p>posted in <%= link_to @post.blog.name, edit_admin_blog_path(@post.blog) %></p>
+<p><%=t 'spree.blogs.posted_in' %> <%= link_to @post.blog.name, edit_admin_blog_path(@post.blog) %></p>
 <h4><%= h(l @post.posted_at, :format => :detailed).gsub(/\s0/, ' ') %></h4>
 <br/>
 
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 553bedc..60fb0b2 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -3,6 +3,9 @@ en:
   # Spree Hax! Missing translation...
   image_settings_description: "Image Settings"
 
+  'yes': "Yes"
+  'no': "No"
+
   activerecord:
     attributes:
       blog:
@@ -33,10 +36,13 @@ en:
           new:
             new_blog: New Blog
           index:
+            listing: Listing Blogs
             new_blog: New Blog
 
 
         posts:
+          index:
+            listing: Listing Posts
           new:
             new_post: New Post
         post_products:
@@ -61,6 +67,7 @@ en:
         title: Recent Posts
       archive: Blog Archive
       no_posts: No posts found!
+      posted_in: posted in
       posts:
         index:
           blog: Blog

From 909c7b1666bf1785d40b6cf46c260f74fc93f95e Mon Sep 17 00:00:00 2001
From: Matteo Latini <mtylty@gmail.com>
Date: Tue, 31 Jul 2012 13:51:14 +0300
Subject: [PATCH 06/10] prevents RoutingError

---
 config/routes.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/routes.rb b/config/routes.rb
index f63479b..c03bc22 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -20,7 +20,7 @@
       
     end
      
-    constraints :blog_id => /([a-z0-9\-\_\/]{0,})/ do
+    constraints :blog_id => /(.*)/ do
       
       constraints(
         :year  => /\d{4}/,

From 94e991d8ea3b8759ac13baae544bd4c9b181ce99 Mon Sep 17 00:00:00 2001
From: Alberto Vena <kennyadsl@gmail.com>
Date: Tue, 31 Jul 2012 15:08:24 +0300
Subject: [PATCH 07/10] updates blogpost page title

---
 app/controllers/spree/blogs/posts_controller.rb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/controllers/spree/blogs/posts_controller.rb b/app/controllers/spree/blogs/posts_controller.rb
index 03cf1ae..4861781 100644
--- a/app/controllers/spree/blogs/posts_controller.rb
+++ b/app/controllers/spree/blogs/posts_controller.rb
@@ -41,6 +41,7 @@ def search
 	
   def show
     @post = default_scope.includes(:tags, :images, :products).find_by_path(params[:id]) rescue nil
+    @title = @post.title if @post && @post.title.present?
     return redirect_to archive_posts_path unless @post
   end
   

From 7bab9074f900abfc395cba6760d14105b490184f Mon Sep 17 00:00:00 2001
From: Alberto Vena <kennyadsl@gmail.com>
Date: Fri, 26 Jul 2013 23:28:07 +0200
Subject: [PATCH 08/10] uses standard date formats

---
 app/views/spree/blogs/admin/posts/index.html.erb       | 2 +-
 app/views/spree/blogs/admin/posts/show.html.erb        | 2 +-
 app/views/spree/blogs/admin/shared/_post_tabs.html.erb | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/views/spree/blogs/admin/posts/index.html.erb b/app/views/spree/blogs/admin/posts/index.html.erb
index 32d82d3..54915f5 100644
--- a/app/views/spree/blogs/admin/posts/index.html.erb
+++ b/app/views/spree/blogs/admin/posts/index.html.erb
@@ -24,7 +24,7 @@
     <tr id="<%= dom_id post %>">
       <td><%= link_to post.title, object_url(post) %></td>
       <td><%= post.blog.nil? ? "n/a" : link_to(post.blog.name, admin_posts_path(:search => { :blog_id_equals => post.blog_id })) %></td>
-      <td><%= l post.posted_at, :format => :date_time %></td>
+      <td><%= l post.posted_at, :format => :short %></td>
       <td><%= post.live ? t(:yes) : t(:no) %></td>
       <td>
       	<%= link_to_edit post %> &nbsp;
diff --git a/app/views/spree/blogs/admin/posts/show.html.erb b/app/views/spree/blogs/admin/posts/show.html.erb
index a49f5d9..aa0210d 100644
--- a/app/views/spree/blogs/admin/posts/show.html.erb
+++ b/app/views/spree/blogs/admin/posts/show.html.erb
@@ -1,6 +1,6 @@
 <%= render :partial => 'spree/blogs/admin/shared/post_tabs', :locals => {:current => "Post Details"} %>
 <p><%=t 'spree.blogs.posted_in' %> <%= link_to @post.blog.name, edit_admin_blog_path(@post.blog) %></p>
-<h4><%= h(l @post.posted_at, :format => :detailed).gsub(/\s0/, ' ') %></h4>
+<h4><%= h(l @post.posted_at, :format => :short).gsub(/\s0/, ' ') %></h4>
 <br/>
 
 <% for image in @post.images do %>
diff --git a/app/views/spree/blogs/admin/shared/_post_tabs.html.erb b/app/views/spree/blogs/admin/shared/_post_tabs.html.erb
index 733e9e7..0c6c833 100644
--- a/app/views/spree/blogs/admin/shared/_post_tabs.html.erb
+++ b/app/views/spree/blogs/admin/shared/_post_tabs.html.erb
@@ -2,7 +2,7 @@
 
 <% content_for :sidebar do %>
 
-  <h3 class="name"><%= @post.title %><span class='sku'><%= l (@post.posted_at || Time.now), :format => :date_only %></span></h3>
+  <h3 class="name"><%= @post.title %><span class='sku'><%= l (@post.posted_at || Time.now), :format => :short %></span></h3>
   <br clear="clear" />
 
   <ul class="sidebar post-menu">

From c7ce4745f3b549f10dc2e79cf3d3997546d7230c Mon Sep 17 00:00:00 2001
From: Daniele Palombo <ksemuldie@gmail.com>
Date: Fri, 9 Sep 2016 12:02:03 +0200
Subject: [PATCH 09/10] Solved bug in scope where

use lambdas for date based rails scopes
http://blog.timothyandrew.net/blog/2013/04/08/use-lambdas-for-date-based-rails-scopes/
---
 app/models/spree/post.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/models/spree/post.rb b/app/models/spree/post.rb
index e74225a..1c776fd 100644
--- a/app/models/spree/post.rb
+++ b/app/models/spree/post.rb
@@ -24,8 +24,8 @@ class Spree::Post < ActiveRecord::Base
   @@per_page = 10
 
   scope :ordered, order("posted_at DESC")
-  scope :future,  where("posted_at > ?", Time.now).order("posted_at ASC")
-  scope :past,    where("posted_at <= ?", Time.now).ordered
+  scope :future,  , lambda { where("posted_at > ?", Time.now).order("posted_at ASC") }
+  scope :past,      lambda { where("posted_at <= ?", Time.now).ordered }
   scope :live,    where(:live => true )
 
  	before_validation :create_path, :if => proc{ |record| record.title_changed? }

From 7b53c92049f4362da439715de82f4baf9c71d1f6 Mon Sep 17 00:00:00 2001
From: Daniele Palombo <ksemuldie@gmail.com>
Date: Fri, 9 Sep 2016 12:44:28 +0200
Subject: [PATCH 10/10] Update post.rb

---
 app/models/spree/post.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/models/spree/post.rb b/app/models/spree/post.rb
index 1c776fd..3335e52 100644
--- a/app/models/spree/post.rb
+++ b/app/models/spree/post.rb
@@ -24,8 +24,8 @@ class Spree::Post < ActiveRecord::Base
   @@per_page = 10
 
   scope :ordered, order("posted_at DESC")
-  scope :future,  , lambda { where("posted_at > ?", Time.now).order("posted_at ASC") }
-  scope :past,      lambda { where("posted_at <= ?", Time.now).ordered }
+  scope :future,  lambda { where("posted_at > ?", Time.now).order("posted_at ASC") }
+  scope :past,    lambda { where("posted_at <= ?", Time.now).ordered }
   scope :live,    where(:live => true )
 
  	before_validation :create_path, :if => proc{ |record| record.title_changed? }