Merge pull request #92 from balkansalat/rails3-ajaxArticleDelete

Delete StockArticles without reloading; small fixes, ignore secret_token initializer.

.gitignore

@@ -1,6 +1,7 @@
 log/*.log
 tmp/**/*
 config/*.yml
+config/initializers/secret_token.rb
 db/*.sqlite3
 nbproject/
 config/environments/development.rb

app/assets/stylesheets/bootstrap_and_overrides.css.less

@@ -37,6 +37,11 @@ body {
 
 // Custom styles
 
+// Fix empty dd tags in horizontal dl, see https://github.com/twitter/bootstrap/issues/4062
+.dl-horizontal { 
+    dd { .clearfix(); }
+}
+
 @mainRedColor: #ED0606;
 
 .logo {
@@ -194,4 +199,4 @@ tr.unavailable {
   input, select {
     border-color: red;
   }
-}
+}

app/controllers/articles_controller.rb

@@ -20,7 +20,7 @@ class ArticlesController < ApplicationController
       sort = "article_categories.name, articles.name"
     end
 
-    @articles = Article.where(supplier_id: @supplier).includes(:article_category).order(sort)
+    @articles = Article.where(supplier_id: @supplier, :type => nil).includes(:article_category).order(sort)
     @articles = @articles.where('articles.name LIKE ?', "%#{params[:query]}%") unless params[:query].nil?
 
     @articles = @articles.page(params[:page]).per(@per_page)
@@ -231,4 +231,4 @@ class ArticlesController < ApplicationController
       redirect_to supplier_articles_path(@supplier), :notice => "Der Katalog ist aktuell."
     end
   end
-end
+end

app/controllers/orders_controller.rb

@@ -126,4 +126,4 @@ class OrdersController < ApplicationController
                 :type => 'text/plain; charset=utf-8; header=present',
                 :disposition => "attachment; filename=#{order.name}"
   end
-end
+end

app/controllers/stockit_controller.rb

@@ -32,11 +32,12 @@ class StockitController < ApplicationController
   end
 
   def destroy
-    StockArticle.find(params[:id]).destroy
-    redirect_to stock_articles_path
+    @article = StockArticle.find(params[:id])
+    @article.destroy
+    render :layout => false
   rescue => error
-    flash[:error] = "Ein Fehler ist aufgetreten: " + error.message
-    redirect_to stock_articles_path
+    render :partial => "destroy_fail", :layout => false,
+      :locals => { :fail_msg => "Ein Fehler ist aufgetreten: " + error.message }
   end
 
   #TODO: Fix this!!

app/documents/order_fax.rb

@@ -19,6 +19,10 @@ class OrderFax < OrderPdf
       text contact[:street], align: :right
       move_down 5
       text "#{contact[:zip_code]} #{contact[:city]}", align: :right
+      move_down 5
+      if @order.supplier.customer_number != ''
+        text "Kundennummer: #{@order.supplier.customer_number}", align: :right
+      end
       move_down 10
       text contact[:phone], size: 9, align: :right
       move_down 5
@@ -34,6 +38,7 @@ class OrderFax < OrderPdf
       text "Fax: " + @order.supplier.fax
     end
 
+    move_down 5
     text Date.today.strftime('%d.%m.%Y'), align: :right
 
     move_down 10
@@ -66,4 +71,4 @@ class OrderFax < OrderPdf
               #align: {0 => :left}
   end
 
-end
+end

app/views/stockit/_destroy_fail.js.haml

@@ -0,0 +1,5 @@
+-# please polish the following line if you know how, same in view destroy
+var errorDiv = $('<div class="alert fade in alert-error"><a class="close" data-dismiss="alert" href="#">x</a></div>');
+
+errorDiv.append(document.createTextNode('#{j(fail_msg)}'));
+$('div.container-fluid').prepend(errorDiv);

app/views/stockit/destroy.js.haml

@@ -0,0 +1,14 @@
+-# please polish the following line if you know how, same in partial _destroy_fail
+var successDiv = $('<div class="alert fade in alert-success"><a class="close" data-dismiss="alert" href="#">x</a></div>');
+
+successDiv.append(document.createTextNode('Artikel #{j(@article.name)} gelöscht.'));
+$('div.container-fluid').prepend(successDiv);
+
+-# WARNING: If you try to use the escape j(...) here, an error occurs:
+-# Ein Fehler ist aufgetreten: undefined method `gsub' for 50:Fixnum
+-# However, it should work without without escaping.
+-# Note that article names which are purely numeric, e.g. 12345, are escaped correctly (see above).
+
+$('#stockArticle-#{@article.id}').remove();
+
+-# WARNING: Do not use a simple .fadeOut() above, because it conflicts with the show/hide function of unavailable articles.

app/views/stockit/index.html.haml

@@ -29,7 +29,6 @@
         - Supplier.all.each do |supplier|
           %li= link_to supplier.name, new_supplier_delivery_path(supplier), tabindex: -1
 
-
 %table.table.table-hover#articles
   %thead
     %tr
@@ -45,7 +44,7 @@
       %th
   %tbody
     - for article in @stock_articles
-      %tr{:class => stock_article_classes(article)}
+      %tr{:class => stock_article_classes(article), :id => "stockArticle-#{article.id}"}
         %td=h article.name
         %td= article.quantity
         %td= article.quantity - article.quantity_available
@@ -58,7 +57,8 @@
         %td
           = link_to "Bearbeiten", edit_stock_article_path(article), class: 'btn btn-mini'
           = link_to "Löschen", article, :method => :delete, :confirm => "Bist Du sicher?",
-            class: 'btn btn-mini btn-danger'
+            class: 'btn btn-mini btn-danger', :remote => true
+
 %p
   Aktueller Lagerwert:
   = number_to_currency StockArticle.stock_value

config/initializers/secret_token.rb

@@ -1,7 +0,0 @@
-# Be sure to restart your server when you modify this file.
-
-# Your secret key for verifying the integrity of signed cookies.
-# If you change this key, all old signed cookies will become invalid!
-# Make sure the secret is at least 30 characters and all random,
-# no regular words or you'll be exposed to dictionary attacks.
-Foodsoft::Application.config.secret_token = '2be5574568ff4d270b108399078a8e485b363af84d441d02d2a6fd3fc51a8c015065790b7e414134e6d97ffc40da898a5a12f66f9de6b992b7ea96e7a34839b8'