spree 0.10.1 → 0.10.2

data/CHANGELOG

@@ -1,3 +1,20 @@
+== 0.10.2
+
+* # - 1215 Update Vietnamese translation
+* # - 1230 Product#recalculate_count_on_hand migration from 0.9.4 to 0.10.0
+* # - 1234 Billing address not accepting UK state
+* # - 1238 Should not create empty order in db for orders/new action
+* # - 1240 Add test-unit to gem dependency
+* # - 1249 Authorize.net CIM does not work with live account
+* # - 1251 Skip confirmation step if payment profiles are not available
+* # - 1253 Table variants doesn't exist when bootstrapping with extensions installed that modify the Variant model
+* # - 1255 Products groups edit error
+* # - 1257 Coupons do not recalculate credit after redeemed 
+* # - 1261 Type Error when Checkout
+* # - 1263 Submitting a coupon during the checkout confirmation fails
+* # - 1269 Orders Overview page breaks when there are no orders in the last 7 days
+* # - 1276 Product Images not updating properly
+
 == 0.10.1
 
 * # - 1209 Ruby Gems Warning (reverted this fix - RubyGems 1.3.6 is now required again)

data/app/controllers/admin/images_controller.rb

@@ -1,58 +1,49 @@
 class Admin::ImagesController < Admin::BaseController
   resource_controller
   before_filter :load_data
-	
-	new_action.response do |wants|
+  
+  new_action.response do |wants|
     wants.html {render :action => :new, :layout => false}
   end
 
-	create.response do |wants|
-		wants.html {redirect_to admin_product_images_url(@product)}
+  create.response do |wants|
+    wants.html {redirect_to admin_product_images_url(@product)}
   end
 
-	update.response do |wants|
-		wants.html {redirect_to admin_product_images_url(@product)}
+  update.response do |wants|
+    wants.html {redirect_to admin_product_images_url(@product)}
   end
-	
-	create.before :create_before
-	update.before :update_before
-	destroy.before :destroy_before
+  
+  create.before :set_viewable
+  update.before :set_viewable
+  destroy.before :destroy_before
   
   destroy.response do |wants| 
     wants.html do
-			render :text => ""
+      render :text => ""
     end
   end
  
   private
 
   def load_data
-		@product = Product.find_by_permalink(params[:product_id])
-		@variants = @product.variants.collect do |variant| 
-			[variant.options_text, variant.id ]
-		end
-		@variants.insert(0, [I18n.t("all"), "All"])
+    @product = Product.find_by_permalink(params[:product_id])
+    @variants = @product.variants.collect do |variant| 
+      [variant.options_text, variant.id ]
+    end
+    @variants.insert(0, [I18n.t("all"), "All"])
   end
 
-  def create_before
-		if params[:image].has_key? :viewable_id
-			if params[:image][:viewable_id] == "All"
-				object.viewable_type = 'Product'
-				object.viewable_id = @product.id
-			else
-				object.viewable_type = 'Variant'
-				object.viewable_id = params[:image][:viewable_id]
-			end
-		else
-			object.viewable_type = 'Product'
-			object.viewable_id = @product.id
-		end
-	end
-	
-	def update_before
-	  if params[:image][:viewable_id] == "All"
-        object.viewable_type = 'Product'
-        object.viewable_id = @product.id
+  def set_viewable
+    if params[:image].has_key? :viewable_id
+      if params[:image][:viewable_id] == "All"
+        object.viewable = @product
+      else
+        object.viewable_type = 'Variant'
+        object.viewable_id = params[:image][:viewable_id]
+      end
+    else
+      object.viewable = @product
     end
   end
   

data/app/controllers/checkouts_controller.rb

@@ -81,7 +81,9 @@ class CheckoutsController < Spree::BaseController
       if params[:payment_source].present? && source_params = params.delete(:payment_source)[params[:checkout][:payments_attributes].first[:payment_method_id].underscore]
         params[:checkout][:payments_attributes].first[:source_attributes] = source_params
       end
-      params[:checkout][:payments_attributes].first[:amount] = @order.total
+      if (params[:checkout][:payments_attributes])
+        params[:checkout][:payments_attributes].first[:amount] = @order.total
+      end
     end
     params[:checkout]
   end

data/app/controllers/spree/base_controller.rb

@@ -21,12 +21,12 @@ class Spree::BaseController < ActionController::Base
   # retrieve the order_id from the session and then load from the database (or return a new order if no
   # such id exists in the session)
   def find_order
-    unless session[:order_id].blank?
+    if !session[:order_id].blank?
       @order = Order.find_or_create_by_id(session[:order_id])
+    elsif request.get?
+      @order = Order.new(:user => current_user)
     else
-      @order = Order.new
-      @order.user = current_user
-      @order.save
+      @order = Order.create(:user => current_user)
     end
     session[:order_id]    = @order.id
     session[:order_token] = @order.token

data/app/helpers/checkouts_helper.rb

@@ -2,6 +2,7 @@ module CheckoutsHelper
 
   def checkout_progress
     steps = Checkout.state_names.reject { |n| n == "complete" }.map do |state|
+      next if state == "confirm" and not Gateway.current.payment_profiles_supported?
       text = t("checkout_steps.#{state}")
 
       css_classes = []

data/app/models/address.rb

@@ -27,6 +27,7 @@ class Address < ActiveRecord::Base
   end
 
   def state_name_validate
+    country = country_id ? Country.find(country_id) : nil
     return if country.blank? || country.states.empty?
     if state_name.blank? || country.states.name_or_abbr_equals(state_name).empty?
       errors.add(:state, :invalid)

data/app/models/checkout.rb

@@ -62,8 +62,9 @@ class Checkout < ActiveRecord::Base
     event :next do
       transition :to => 'delivery', :from  => 'address'
       transition :to => 'payment', :from => 'delivery'
-      transition :to => 'confirm', :from => 'payment'
+      transition :to => 'confirm', :from => 'payment', :if => Proc.new { Gateway.current and Gateway.current.payment_profiles_supported? }
       transition :to => 'complete', :from => 'confirm'
+      transition :to => 'complete', :from => 'payment'
     end
   end
   def self.state_names

data/app/models/coupon.rb

@@ -21,7 +21,6 @@ class Coupon < ActiveRecord::Base
       order.save
       coupon_credits.create({
           :order => order, 
-          :amount => amount,
           :description => "#{I18n.t(:coupon)} (#{code})"
         })
     end

data/app/models/product.rb

@@ -34,7 +34,7 @@ class Product < ActiveRecord::Base
     :conditions => ["variants.is_master = ? AND variants.deleted_at IS NULL", true]
 
   delegate_belongs_to :master, :sku, :price, :weight, :height, :width, :depth, :is_master
-  delegate_belongs_to :master, :cost_price if Variant.column_names.include? "cost_price"
+  delegate_belongs_to :master, :cost_price if Variant.table_exists? && Variant.column_names.include?("cost_price")
 
   after_create :set_master_variant_defaults
   after_create :add_properties_and_option_types_from_prototype
@@ -188,6 +188,13 @@ class Product < ActiveRecord::Base
     deleted_at
   end
 
+  # split variants list into hash which shows mapping of opt value onto matching variants
+  # eg categorise_variants_from_option(color) => {"red" -> [...], "blue" -> [...]}
+  def categorise_variants_from_option(opt_type)
+    return {} unless option_types.include?(opt_type)
+    variants.active.group_by {|v| v.option_values.detect {|o| o.option_type == opt_type} }
+  end
+
   private
 
   def recalculate_count_on_hand

data/app/models/variant.rb

@@ -9,7 +9,7 @@ class Variant < ActiveRecord::Base
 
   validate :check_price
   validates_presence_of :price
-  validates_numericality_of :cost_price, :allow_nil => true if Variant.column_names.include? "cost_price"
+  validates_numericality_of :cost_price, :allow_nil => true if Variant.table_exists? && Variant.column_names.include?("cost_price")
 
   before_save :touch_product
 

data/config/database.yml

@@ -1,57 +1,41 @@
-# Default database configuration
-
+#   gem install mysql -- --include=/usr/local/lib
+# On Windows:
+#   There is no gem for Windows.  Install mysql.so from RubyForApache.
+#   http://rubyforge.org/projects/rubyforapache
+#
+# And be sure to use new-style password hashing:
+#   http://dev.mysql.com/doc/refman/5.0/en/old-client.html
 development:
   adapter: mysql
-  database: spree_dev
+  database: spree_development
   username: root
-  password: 
-  host: localhost   
-
-# development:
-#  adapter:  postgresql
-#  database: spree_dev
-#  username: spree
-#  password: spree_dev
-#  host: localhost   
+  password:
+  host: localhost
   
 # development:
-#  adapter: sqlite3
-#  database: db/development.sqlite3   
-                                     
+#   adapter: postgresql
+#   database: spree_dev
+#   username: postgres
+#   password:
+#   host: localhost
+      
+# development:
+#   adapter: sqlite3
+#   database: db/development.sqlite3.db
+
+# Warning: The database defined as 'test' will be erased and
+# re-generated from your development database when you run 'rake'.
+# Do not set this db to the same as development or production.
 test:
   adapter: mysql
   database: spree_test
   username: root
-  password: 
-  host: localhost 
-
-# test:
-#  adapter: sqlite3
-#  database: db/test.sqlite3
-  
-# staging:
-#   #adapter: sqlite3
-#   #database: db/staging.sqlite3
-#   adapter: mysql
-#   database: spree_dev
-#   username: spree
-#   password: spree
-#   host: localhost
-  
-#test:
-#  adapter: postgresql
-#  database: spree_test
-#  username: spree
-#  password: spree_dev
-#  host: localhost
-
-#test:
-#  adapter: sqlite3
-#  database: db/test.sqlite3
+  password:
+  host: localhost
 
 production:
   adapter: mysql
-  database: spree
+  database: spree_development
   username: root
   password: 
-  host: localhost
+  host: localhost

data/config/environment.rb

@@ -5,7 +5,7 @@
 # ENV['RAILS_ENV'] ||= 'production'
 
 # Specifies gem version of Rails to use when vendor/rails is not present
-SPREE_GEM_VERSION = '0.10.1' unless defined? SPREE_GEM_VERSION
+SPREE_GEM_VERSION = '0.10.2' unless defined? SPREE_GEM_VERSION
 
 # Bootstrap the Rails environment, frameworks, and default configuration
 require File.join(File.dirname(__FILE__), 'boot')

data/config/environments/test.rb

@@ -22,4 +22,5 @@ config.action_mailer.delivery_method = :test
 config.action_controller.allow_forgery_protection = false
 
 config.gem "shoulda", :lib => "shoulda", :version => "2.10.2"
-config.gem "factory_girl", :lib => "factory_girl", :version => "1.2.3"      
+config.gem "factory_girl", :lib => "factory_girl", :version => "1.2.3"
+config.gem 'test-unit', :lib => 'test/unit', :version => '~>2.0.5' if RUBY_VERSION.to_f >= 1.9

data/config/initializers/workarounds_for_ruby19.rb

@@ -99,4 +99,29 @@ if RUBY_VERSION.to_f >= 1.9
     end
   end
 
+  module I18n
+    module Backend
+      class Simple
+        protected
+        
+        def lookup(locale, key, scope = [])
+          return unless key
+          init_translations unless initialized?
+          keys = I18n.send(:normalize_translation_keys, locale, key, scope)
+          begin
+            keys.inject(translations) do |result, k|
+              if (x = result[k.to_sym]).nil?
+                return nil
+              else
+                x
+              end
+            end
+          rescue
+            return nil
+          end
+        end
+      end
+    end
+  end
+  
 end

data/config/locales/vn.yml

@@ -24,12 +24,12 @@ vn:
         address1: Địa chỉ
         address2: "Địa chỉ (tiếp)"
         city: Thành phố
-        country: "Country"
-        first_name: "First Name"
-        last_name: "Last Name"
+        country: "Quốc gia"
+        first_name: "Tên"
+        last_name: "Họ"
         phone: Điện thoại
-        state: "State"
-        zipcode: "Mã Zip"
+        state: "Bang"
+        zipcode: "Mã bưu điện"
       checkout:
         bill_address:
           address1: "Địa chỉ thanh toán"
@@ -38,7 +38,7 @@ vn:
           lastname: "Họ"
           phone: "Điện thoại"
           state: "Bang"
-          zipcode: "Mã Zip"
+          zipcode: "Mã bưu điện"
         ship_address:
           address1: "Địa chỉ"
           city: "Thành phố"
@@ -82,14 +82,14 @@ vn:
         shipping_category: "Loại hình vận chuyển"
         tax_category: "Biểu thuế"
       product_group:
-        name: "Name"
-        product_count: "Product count"
-        product_scopes: "Product scopes"
-        products: "Products"
+        name: "Tên"
+        product_count: "Số lượng sản phẩm"
+        product_scopes: "Phạm vi sản phẩm"
+        products: "Sản phẩm"
         url: "URL"
       product_scope:
-        arguments: "Arguments"
-        description: "Description"
+        arguments: "Tham số"
+        description: "Chú thích"
       property:
         name: Tên
         presentation: Trình bày
@@ -213,7 +213,7 @@ vn:
   add_option_value: "Thêm giá trị của tùy chọn"
   add_product: "Thêm sản phẩm"
   add_product_properties: "Thêm đặc tính sản phẩm"
-  add_scope: "Add a scope"
+  add_scope: "Thêm phạm vi"
   add_state: "Thêm bang"
   add_to_cart: "Mua hàng"
   add_zone: "Thêm vùng"
@@ -231,7 +231,7 @@ vn:
   allowed_ssl_in_production_mode: "SSL sẽ {{not}} được dùng trong sản xuất"
   already_registered: Đã đăng kí?
   alternative_phone: Điện thoại khác
-  amount: Số lượng
+  amount: Giá trị
   analytics_trackers: Analytics Trackers
   are_you_sure: "Bạn có chắn chắn không?"
   are_you_sure_category: "Bạn có chắc bạn muốn xóa loại mặt hàng này không?"
@@ -243,7 +243,7 @@ vn:
   assign_taxons: "Ấn định đơn vị phân loại"
   authorization_failure: "Không được ủy quyền truy cập"
   authorized: Được ủy quyền
-  available_on: "Có hàng vào"
+  available_on: "Có hàng vào ngày"
   available_taxons: "Đơn vị phân loại hiện có"
   awaiting_return: Đang đợi trả về
   back: Quay lại
@@ -251,7 +251,7 @@ vn:
   backordered: Đã đặt hàng trước
   backordering_is_allowed: "Đã đặt hàng trước {{not}} được cho phép"
   balance_due: "Tiền cần thanh toán"
-  best_selling_products: "Sản phầm bán chạy nhất"
+  best_selling_products: "Sản phẩm bán chạy nhất"
   best_selling_taxons: "Đơn vị phân loại hàng bán chạy nhất"
   bill_address: "Địa chỉ thanh toán"
   billing: Thanh Toán
@@ -262,7 +262,7 @@ vn:
   cancel: Hủy
   canceled: Đã hủy
   cannot_create_returns: Không thể trả hàng vì đơn hàng chưa được gửi.
-  capture: bắt
+  capture: Lấy tiền
   card_code: "Mã thẻ"
   card_details: "Thông tin thẻ"
   card_number: "Số thẻ"
@@ -281,12 +281,12 @@ vn:
     # keys correspond to Checkout state names:
     address: Địa chỉ
     complete: Hoàn tất
-    confirm: Confirm
+    confirm: Xác nhận
     delivery: Vận chuyển
     payment: Thanh toán
   cheque: Séc
   city: Thành phố
-  clone: Sao y
+  clone: Tạo bản sao
   code: Mã
   combine: Nhập vào
   comp_order: "Đơn hàng kép"
@@ -296,7 +296,7 @@ vn:
   configuration: Cấu hình
   configuration_options: "Tùy chọn cấu hình"
   configurations: Cấu hình
-  configured: Configured
+  configured: Đã được cấu hình
   confirm: Xác nhận
   confirm_delete: "Xác nhận xóa"
   confirm_password: "Xác nhận mật khẩu"
@@ -330,7 +330,7 @@ vn:
   customer_details: "Thông tin khách hàng"
   customer_search: "Tìm kiếm khách hàng"
   date_created: Ngày tạo
-  date_range: "Chuỗi ngày"
+  date_range: "Giới hạn ngày"
   debit: Nợ
   delete: Xóa
   depth: Sâu
@@ -345,7 +345,7 @@ vn:
   editing_option_types: "Sửa đổi Kiểu tùy chọn"
   editing_payment_method: Sửa đổi Phương thức Thanh toán
   editing_product: "Sửa đổi sản phẩm"
-  editing_product_group: "Editing Product Group"
+  editing_product_group: "Sửa đổi Nhóm sản phẩm"
   editing_property: "Sửa đổi đặc tính"
   editing_prototype: "Sửa đổi nguyên mẫu"
   editing_shipping_category: "Sửa đổi loại chuyển phát"
@@ -377,7 +377,7 @@ vn:
   extensions: Gói mở rộng
   filename: Tên tệp tin
   final_confirmation: "Chứng thực cuối cùng"
-  finalize: Finalize
+  finalize: Hoành thành
   finalized_payments: Thanh toán đã hoàn tất
   first_item: Món hàng đầu tiên giá
   first_name: "Tên"
@@ -428,9 +428,9 @@ vn:
   item: Món
   item_description: "Miêu tả món hàng"
   item_total: "Tổng số món"
-  items: "Món"
+  items: "Số lượng"
   last_14_days: "14 ngày trước"
-  last_5_orders: "5 đơn hàng cuối"
+  last_5_orders: "5 đơn hàng gần đây nhất"
   last_7_days: "7 ngày trước"
   last_month: "Tháng trước"
   last_name: "Họ"
@@ -532,7 +532,7 @@ vn:
   options: Tùy chọn
   or: hoặc
   ord_qty: "Số lượng"
-  ord_total: "Tổng số"
+  ord_total: "Giá trị"
   order: Đơn hàng
   order_confirmation_note: ""
   order_date: "Ngày đặt hàng"
@@ -545,7 +545,7 @@ vn:
   order_processed_successfully: "Đơn đặt hàng của bạn đã được xử lý thành công"
   order_summary: Tóm tắt đơn đặt hàng
   order_sure_want_to: "Bạn có chắc bạn muốn {{event}} đơn hàng này?"
-  order_total: "Tổng đơn hàng"
+  order_total: "Tổng giá sau thuế"
   order_total_message: "Tổng số tiền sẽ được rút từ thẻ của bạn là"
   order_updated: "Đơn hàng được cập nhật"
   orders: Đơn hàng
@@ -553,7 +553,7 @@ vn:
   out_of_stock: "Hết hàng"
   out_of_stock_products: "Sản phẩm đã hết hàng"
   over_paid: "Trả lố"
-  overview: Tồng kết
+  overview: Tổng kết
   overview_welcome: "Chào mừng bạn đến với phần tổng quan, hiện không đủ thông tin để hiển thị Bảng điều khiển tổng quan.<br/><br/> Bảng điều khiển sẽ tự động hiện ra khi hệ thống đã thu thập đủ số liệu thông kê."
   page_only_viewable_when_logged_in: Trang này chỉ xem được sau khi đã đăng nhập
   page_only_viewable_when_logged_out: Trang này chỉ xem được sau khi đã đăng xuất
@@ -646,7 +646,7 @@ vn:
         sentence: tên hay từ khóa có chứa <em>%s</em>
       in_taxons:
         args:
-          "taxon_names": "Tên Taxon"
+          "taxon_names": "Tên phân loại"
         description: "Tên đơn vị phân loại phải được tách ra với dấu phẩy hoặc chỗ trống (vd: adidas,shoes)"
         name: "Trong các đơn vị phân loại và tất cả đơn vị phân loại con"
         sentence: trong <em>%s</em> và tất cả hậu duệ của chúng
@@ -835,11 +835,11 @@ vn:
   state_setting_description: "Quản lý danh sách các bang và quận huyện của từng quốc gia."
   states: Bang
   status: Tình trạng
-  stop: Tạm ngừng
+  stop: Kết thúc
   store: Cửa hàng
   street_address: "Địa chỉ"
   street_address_2: "Địa chỉ (tiếp)"
-  subtotal: Tổng cộng
+  subtotal: Tổng giá trước thuế
   subtract: Trừ đi
   system: Hệ thống
   tax: Thuế
@@ -860,8 +860,8 @@ vn:
   taxonomy_tree_error: "Thay đồi theo yêu cầu không được chấp nhận và hệ cây đã quay trở về trạng thái như trước, xin hay thử lại lần nữa."
   taxonomy_tree_instruction: "* Nhấp chuột phải vào 1 phần tử con trong hệ cây để truy cập thực đơn để thêm, xóa và sắp xếp một phần tử con."
   taxons: Đơn vị phân loại
-  test: "Test"
-  test_mode: Test Mode
+  test: "Kiểm tra"
+  test_mode: Chế độ kiểm tra
   thank_you_for_your_order: "Cảm ơn đã mua hàng. Xin hãy in ra một bản của trang này để tiện cho việc chứng thực nếu cần."
   this_file_language: "tiếng Việt (VN)"
   this_month: "Tháng này"
@@ -869,7 +869,7 @@ vn:
   thumbnail: "Hình nhỏ"
   to_add_variants_you_must_first_define: "Để thêm biến thể, bạn phải định nghĩa trước"
   top_grossing_products: "Sản phẩm lãi nhiều nhất"
-  total: Tổng
+  total: Giá trị
   tracking: Theo dõi
   transaction: Giao dịch
   transactions: Giao dịch
@@ -906,7 +906,7 @@ vn:
   vat: "VAT"
   version: Phiên bản
   view_shipping_options: "Xem các lựa chọn dịch vụ chuyển phát"
-  void: Hết bảo hành
+  void: Vô hiệu hóa
   website: Trang web
   weight: Khối lượng
   welcome_to_sample_store: "Chào mừng đến cửa hàng mẫu"
@@ -917,7 +917,7 @@ vn:
   year: "Năm"
   you_have_been_logged_out: "Bạn vừa đăng xuất."
   your_cart_is_empty: "Sọt hàng rỗng"
-  zip: Zip
+  zip: Mã bưu điện
   zone: Vùng
   zone_based: "Dựa trên vùng"
   zone_setting_description: "Danh sách các quốc gia, bang hoặc vùng khác được dùng trong nhiều tính toán khác nhau."