Compare commits
5 commits
78da4feafe
...
66c56ea24d
Author | SHA1 | Date | |
---|---|---|---|
66c56ea24d | |||
e5cd68a4ad | |||
32547e495f | |||
b808f83931 | |||
6835616170 |
6 changed files with 74 additions and 8 deletions
4
Gemfile
4
Gemfile
|
@ -90,8 +90,6 @@ group :development do
|
||||||
|
|
||||||
# Get infos when not using proper eager loading
|
# Get infos when not using proper eager loading
|
||||||
gem 'bullet'
|
gem 'bullet'
|
||||||
# Display Active Record queries as tables in the console
|
|
||||||
gem 'table_print'
|
|
||||||
end
|
end
|
||||||
|
|
||||||
group :development, :test do
|
group :development, :test do
|
||||||
|
@ -103,6 +101,8 @@ group :development, :test do
|
||||||
# allow to use `debugger` https://github.com/conradirwin/pry-rescue
|
# allow to use `debugger` https://github.com/conradirwin/pry-rescue
|
||||||
gem 'pry-rescue'
|
gem 'pry-rescue'
|
||||||
gem 'pry-stack_explorer'
|
gem 'pry-stack_explorer'
|
||||||
|
# Display Active Record queries as tables in the console
|
||||||
|
gem 'table_print'
|
||||||
end
|
end
|
||||||
|
|
||||||
group :test do
|
group :test do
|
||||||
|
|
|
@ -226,10 +226,14 @@ $(function() {
|
||||||
$('a[data-decrease_tolerance]').on('touchclick', function() {
|
$('a[data-decrease_tolerance]').on('touchclick', function() {
|
||||||
decreaseTolerance($(this).data('decrease_tolerance'));
|
decreaseTolerance($(this).data('decrease_tolerance'));
|
||||||
});
|
});
|
||||||
|
$('a[data-reorder_previous]').on('touchclick', function() {
|
||||||
|
console.log('reorder_previous');
|
||||||
|
// update($(this).data('reorder_previous'), $(this).data('quantity'), $(this).data('tolerance'));
|
||||||
|
});
|
||||||
|
|
||||||
$('a[data-confirm_switch_order]').on('touchclick', function() {
|
$('a[data-confirm_switch_order]').on('touchclick', function() {
|
||||||
return (!modified || confirm(I18n.t('js.ordering.confirm_change')));
|
return (!modified || confirm(I18n.t('js.ordering.confirm_change')));
|
||||||
});
|
});
|
||||||
|
|
||||||
updateButtons($(document));
|
updateButtons($(document));
|
||||||
});
|
});
|
|
@ -35,6 +35,7 @@ class GroupOrder < ApplicationRecord
|
||||||
data[:account_balance] = ordergroup.nil? ? BigDecimal.new('+Infinity') : ordergroup.account_balance
|
data[:account_balance] = ordergroup.nil? ? BigDecimal.new('+Infinity') : ordergroup.account_balance
|
||||||
data[:available_funds] = ordergroup.nil? ? BigDecimal.new('+Infinity') : ordergroup.get_available_funds(self)
|
data[:available_funds] = ordergroup.nil? ? BigDecimal.new('+Infinity') : ordergroup.get_available_funds(self)
|
||||||
|
|
||||||
|
|
||||||
# load prices and other stuff....
|
# load prices and other stuff....
|
||||||
data[:order_articles] = {}
|
data[:order_articles] = {}
|
||||||
order.articles_grouped_by_category.each do |article_category, order_articles|
|
order.articles_grouped_by_category.each do |article_category, order_articles|
|
||||||
|
@ -59,9 +60,31 @@ class GroupOrder < ApplicationRecord
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
# add counts from the previous group order
|
||||||
|
if previous_group_order
|
||||||
|
previous_group_order.group_order_articles.each do |goa|
|
||||||
|
order_article_id = OrderArticle.find_by(order: order, article: goa.order_article.article)&.id
|
||||||
|
puts " order ID: #{order.id}"
|
||||||
|
puts " article ID: #{goa.order_article.article.id}"
|
||||||
|
puts "ID: #{order_article_id}"
|
||||||
|
data[:order_articles][order_article_id] ||= {}
|
||||||
|
data[:order_articles][order_article_id][:previous_quantity] = goa.quantity
|
||||||
|
data[:order_articles][order_article_id][:previous_tolerance] = goa.tolerance
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
puts data
|
||||||
|
|
||||||
data
|
data
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def previous_group_order
|
||||||
|
previous_order = ordergroup.orders.where(supplier: order.supplier).where.not(id: order.id).recent.first
|
||||||
|
return nil unless previous_order
|
||||||
|
|
||||||
|
ordergroup.group_orders.find_by(order_id: previous_order.id)
|
||||||
|
end
|
||||||
|
|
||||||
def save_group_order_articles
|
def save_group_order_articles
|
||||||
for order_article in order.order_articles
|
for order_article in order.order_articles
|
||||||
# Find the group_order_article, create a new one if necessary...
|
# Find the group_order_article, create a new one if necessary...
|
||||||
|
|
|
@ -95,7 +95,17 @@
|
||||||
%td{colspan: "9"}
|
%td{colspan: "9"}
|
||||||
- order_articles.each do |order_article|
|
- order_articles.each do |order_article|
|
||||||
%tr{class: "#{cycle('even', 'odd', name: 'articles')} order-article #{get_missing_units_css_class(@ordering_data[:order_articles][order_article.id][:missing_units])}", valign: "top"}
|
%tr{class: "#{cycle('even', 'odd', name: 'articles')} order-article #{get_missing_units_css_class(@ordering_data[:order_articles][order_article.id][:missing_units])}", valign: "top"}
|
||||||
%td.name= order_article.article.name
|
%td.name
|
||||||
|
= order_article.article.name
|
||||||
|
- if @ordering_data[:order_articles][order_article.id][:previous_quantity]
|
||||||
|
%span.label
|
||||||
|
last time:
|
||||||
|
= @ordering_data[:order_articles][order_article.id][:previous_quantity]
|
||||||
|
+
|
||||||
|
= @ordering_data[:order_articles][order_article.id][:previous_tolerance]
|
||||||
|
%a.btn.btn-ordering{"data-reorder_previous": order_article.id, "data-quantity": @ordering_data[:order_articles][order_article.id][:previous_quantity], "data-tolerance": @ordering_data[:order_articles][order_article.id][:previous_tolerance]}
|
||||||
|
%i.icon-repeat
|
||||||
|
order previous
|
||||||
- if @order.stockit?
|
- if @order.stockit?
|
||||||
%td= truncate order_article.article.supplier.name, length: 15
|
%td= truncate order_article.article.supplier.name, length: 15
|
||||||
%td= h order_article.article.origin
|
%td= h order_article.article.origin
|
||||||
|
|
|
@ -6,6 +6,7 @@ services:
|
||||||
command: ./proc-start web
|
command: ./proc-start web
|
||||||
ports:
|
ports:
|
||||||
- "3000:3000"
|
- "3000:3000"
|
||||||
|
platform: linux/x86_64
|
||||||
|
|
||||||
foodsoft_worker:
|
foodsoft_worker:
|
||||||
build:
|
build:
|
||||||
|
|
|
@ -1,8 +1,10 @@
|
||||||
require_relative '../spec_helper'
|
require_relative '../spec_helper'
|
||||||
|
|
||||||
describe GroupOrder do
|
describe GroupOrder do
|
||||||
let(:user) { create :user, groups: [create(:ordergroup)] }
|
let(:ordergroup) { create(:ordergroup) }
|
||||||
let(:order) { create :order }
|
let(:user) { create :user, groups: [ordergroup] }
|
||||||
|
let(:supplier) { create(:supplier, article_count: 3) }
|
||||||
|
let(:order) { create :order, supplier: supplier }
|
||||||
|
|
||||||
# the following two tests are currently disabled - https://github.com/foodcoops/foodsoft/issues/158
|
# the following two tests are currently disabled - https://github.com/foodcoops/foodsoft/issues/158
|
||||||
|
|
||||||
|
@ -15,10 +17,36 @@ describe GroupOrder do
|
||||||
# end
|
# end
|
||||||
|
|
||||||
describe do
|
describe do
|
||||||
let(:go) { create :group_order, order: order, ordergroup: user.ordergroup }
|
let(:go) { create :group_order, order: order, ordergroup: ordergroup }
|
||||||
|
|
||||||
it 'has zero price initially' do
|
it 'has zero price initially' do
|
||||||
expect(go.price).to eq(0)
|
expect(go.price).to eq(0)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
describe "load data for javascript" do
|
||||||
|
let(:group_order) { create :group_order, order: order, ordergroup: ordergroup }
|
||||||
|
let!(:article) { order.articles.first }
|
||||||
|
let(:order_article) { OrderArticle.find_or_create_by!(order: order, article: article) }
|
||||||
|
let(:previous_order) { create :order, supplier: supplier, starts: 20.days.ago, ends: 18.days.ago }
|
||||||
|
let(:previous_group_order) { create :group_order, order: previous_order, ordergroup: ordergroup }
|
||||||
|
let(:previous_order_article) { OrderArticle.find_or_create_by!(order: previous_order, article: article) }
|
||||||
|
|
||||||
|
it "includes data from the last order" do
|
||||||
|
create :group_order_article,
|
||||||
|
order_article: previous_order_article,
|
||||||
|
group_order: previous_group_order,
|
||||||
|
quantity: 23,
|
||||||
|
tolerance: 11
|
||||||
|
|
||||||
|
order.order_articles.map(&:update_results!)
|
||||||
|
order.group_orders.map(&:update_price!)
|
||||||
|
|
||||||
|
order_article_data = group_order.load_data[:order_articles][order_article.id]
|
||||||
|
|
||||||
|
expect(order_article_data[:previous_quantity]).to eq(23)
|
||||||
|
expect(order_article_data[:previous_tolerance]).to eq(11)
|
||||||
|
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue