Skip to content

Commit

Permalink
Add possibility to copy an order
Browse files Browse the repository at this point in the history
If a supplier has a long list of articles, which are all available, but
the order should contain only a subset of them, selecting them for every
order can take a long time. Starting with a copy of an existing order
can safe a lot of time.
  • Loading branch information
paroga committed Feb 18, 2015
1 parent 65c7673 commit a2ff9db
Show file tree
Hide file tree
Showing 8 changed files with 19 additions and 9 deletions.
5 changes: 5 additions & 0 deletions app/controllers/orders_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,11 @@ def show
# Page to create a new order.
def new
@order = Order.new(supplier_id: params[:supplier_id]).init_dates
@checkall_articels = true
if params[:order_id]
@base_order = Order.find(params[:order_id])
@checkall_articels = false
end
end

# Save a new order.
Expand Down
3 changes: 2 additions & 1 deletion app/views/orders/_form.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@
%i.icon-tag
- for article in articles
/ check if the article is selected or has an error
- included = @order.article_ids.include?(article.id.to_s)
- article_ids_order = @base_order || @order
- included = article_ids_order.article_ids.include?(article.id.to_s)
- row_class = ''
- if included
- row_class = 'selected'
Expand Down
2 changes: 1 addition & 1 deletion app/views/orders/index.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
= link_to t('.action_receive'), receive_order_path(order), class: 'btn btn-small btn-success'

%td
= link_to t('ui.edit'), '#', class: 'btn btn-small disabled', tabindex: -1
= link_to t('ui.copy'), new_order_path(order_id: order, supplier_id: order.supplier), class: 'btn btn-small'
= link_to t('ui.show'), order, class: 'btn btn-small'
= link_to t('ui.delete'), order, data: {confirm: t('.confirm_delete')}, method: :delete,
class: 'btn btn-small btn-danger'
Expand Down
14 changes: 7 additions & 7 deletions app/views/orders/new.html.haml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
- title t('.title')

= render 'form'

- content_for :javascript do
:javascript
// select all articles by default
$(function() {
$('#checkall').click();
});
- if @checkall_articels
- content_for :javascript do
:javascript
// select all articles by default
$(function() {
$('#checkall').click();
});
1 change: 1 addition & 0 deletions config/locales/de.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1649,6 +1649,7 @@ de:
actions: Aktionen
cancel: Abbrechen
close: Schließen
copy: Kopieren
delete: Löschen
edit: Bearbeiten
marks:
Expand Down
1 change: 1 addition & 0 deletions config/locales/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1658,6 +1658,7 @@ en:
actions: Actions
cancel: Cancel
close: Close
copy: Copy
delete: Delete
edit: Edit
marks:
Expand Down
1 change: 1 addition & 0 deletions config/locales/fr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1668,6 +1668,7 @@ fr:
actions: Actions
cancel: Annuler
close: Fermer
copy: Copier
delete: Supprimer
edit: Modifier
marks:
Expand Down
1 change: 1 addition & 0 deletions config/locales/nl.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1653,6 +1653,7 @@ nl:
actions: Acties
cancel: Annuleren
close: Sluiten
copy: Kopiëren
delete: Verwijder
edit: Bewerk
marks:
Expand Down

0 comments on commit a2ff9db

Please sign in to comment.