From 92ec85dbbdf65e16da7595d6ae30eae8698a49d9 Mon Sep 17 00:00:00 2001 From: Hugo Manet Date: Wed, 7 Dec 2016 21:35:27 +0100 Subject: [PATCH 1/5] =?UTF-8?q?Ajout=20d'un=20.distinct()=20dans=20la=20se?= =?UTF-8?q?lection=20des=20articles=20li=C3=A9s=20=C3=A0=20un=20supplier?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- kfet/views.py | 1 + 1 file changed, 1 insertion(+) diff --git a/kfet/views.py b/kfet/views.py index 3f1e1f4d..039aa65a 100644 --- a/kfet/views.py +++ b/kfet/views.py @@ -1679,6 +1679,7 @@ def order_create(request, pk): articles = (Article.objects .filter(suppliers=supplier.pk) + .distinct() .select_related('category') .order_by('category__name', 'name')) From bc14205d29bb5ed8c80ff285589eba4ea8304dd4 Mon Sep 17 00:00:00 2001 From: Qwann Date: Wed, 1 Feb 2017 19:32:18 +0100 Subject: [PATCH 2/5] fixing decimal issue2 --- kfet/templatetags/kfet_tags.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kfet/templatetags/kfet_tags.py b/kfet/templatetags/kfet_tags.py index 59840b27..2dd704b6 100644 --- a/kfet/templatetags/kfet_tags.py +++ b/kfet/templatetags/kfet_tags.py @@ -40,6 +40,6 @@ def highlight_clipper(clipper, q): @register.filter() def ukf(balance, is_cof): - grant = is_cof and (1 + Settings.SUBVENTION_COF() / 100) or 1 + grant = 1 + float(Settings.SUBVENTION_COF()) / 100 if is_cof else 1. # float nécessaire car sinon problème avec le round de future.builtins return floor(float(balance) * 10 * grant) From 44202811ae09e118482db18092b435504ef8dac4 Mon Sep 17 00:00:00 2001 From: Qwann Date: Sun, 5 Feb 2017 22:39:20 +0100 Subject: [PATCH 3/5] not supporting future.builtins anymore in templatetags --- kfet/templatetags/kfet_tags.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/kfet/templatetags/kfet_tags.py b/kfet/templatetags/kfet_tags.py index 2dd704b6..3b1bb639 100644 --- a/kfet/templatetags/kfet_tags.py +++ b/kfet/templatetags/kfet_tags.py @@ -40,6 +40,5 @@ def highlight_clipper(clipper, q): @register.filter() def ukf(balance, is_cof): - grant = 1 + float(Settings.SUBVENTION_COF()) / 100 if is_cof else 1. - # float nécessaire car sinon problème avec le round de future.builtins - return floor(float(balance) * 10 * grant) + grant = is_cof and (1 + Settings.SUBVENTION_COF() / 100) or 1 + return floor(balance * 10 * grant) From 8d0f8036393ea8563edd57757abceb853db83a45 Mon Sep 17 00:00:00 2001 From: Ludovic Stephan Date: Mon, 6 Feb 2017 16:42:19 -0200 Subject: [PATCH 4/5] Corrige l'affichage du prix fixes #128 --- kfet/static/kfet/js/kfet.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kfet/static/kfet/js/kfet.js b/kfet/static/kfet/js/kfet.js index 31758c36..e3e5a6d9 100644 --- a/kfet/static/kfet/js/kfet.js +++ b/kfet/static/kfet/js/kfet.js @@ -39,7 +39,7 @@ function amountDisplay(amount, is_cof=false, tri='') { function amountToUKF(amount, is_cof=false) { var coef_cof = is_cof ? 1 + settings['subvention_cof'] / 100 : 1; - return Math.floor(amount * coef_cof * 10); + return Math.round(amount * coef_cof * 10); } function isValidTrigramme(trigramme) { From 3fa0e4de6c083db2793f2250cedbfec987f0dc32 Mon Sep 17 00:00:00 2001 From: Ludovic Stephan Date: Sun, 12 Feb 2017 19:05:41 -0200 Subject: [PATCH 5/5] different rounding for accounts --- kfet/static/kfet/js/kfet.js | 5 +++-- kfet/templates/kfet/kpsul.html | 16 ++++++++-------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/kfet/static/kfet/js/kfet.js b/kfet/static/kfet/js/kfet.js index e3e5a6d9..bab21c12 100644 --- a/kfet/static/kfet/js/kfet.js +++ b/kfet/static/kfet/js/kfet.js @@ -37,9 +37,10 @@ function amountDisplay(amount, is_cof=false, tri='') { return amountToUKF(amount, is_cof); } -function amountToUKF(amount, is_cof=false) { +function amountToUKF(amount, is_cof=false, account=false) { + var rounding = account ? Math.floor : Math.round ; var coef_cof = is_cof ? 1 + settings['subvention_cof'] / 100 : 1; - return Math.round(amount * coef_cof * 10); + return rounding(amount * coef_cof * 10); } function isValidTrigramme(trigramme) { diff --git a/kfet/templates/kfet/kpsul.html b/kfet/templates/kfet/kpsul.html index fb59060b..e064a10c 100644 --- a/kfet/templates/kfet/kpsul.html +++ b/kfet/templates/kfet/kpsul.html @@ -221,7 +221,7 @@ $(document).ready(function() { function displayAccountData() { var balance = account_data['trigramme'] != 'LIQ' ? account_data['balance'] : ''; if (balance != '') - balance = amountToUKF(account_data['balance'], account_data['is_cof']); + balance = amountToUKF(account_data['balance'], account_data['is_cof'], true); var is_cof = account_data['trigramme'] ? account_data['is_cof'] : ''; if (is_cof !== '') is_cof = is_cof ? 'COF' : 'Non-COF'; @@ -616,7 +616,7 @@ $(document).ready(function() { for (var elem in article) { article_html.find('.'+elem).text(article[elem]) } - article_html.find('.price').text(amountToUKF(article['price'], false)); + article_html.find('.price').text(amountToUKF(article['price'], false, false)); var category_html = articles_container .find('#data-category-'+article['category_id']); if (category_html.length == 0) { @@ -834,7 +834,7 @@ $(document).ready(function() { .attr('data-opeindex', index) .find('.number').text(nb).end() .find('.name').text(article_data[0]).end() - .find('.amount').text(amountToUKF(amount_euro, account_data['is_cof'])); + .find('.amount').text(amountToUKF(amount_euro, account_data['is_cof'], false)); basket_container.prepend(article_basket_html); updateBasketRel(); } @@ -848,7 +848,7 @@ $(document).ready(function() { .attr('data-opeindex', index) .find('.number').text(amount+"€").end() .find('.name').text(text).end() - .find('.amount').text(amountToUKF(amount, account_data['is_cof'])); + .find('.amount').text(amountToUKF(amount, account_data['is_cof'], false)); basket_container.prepend(deposit_basket_html); updateBasketRel(); } @@ -861,7 +861,7 @@ $(document).ready(function() { .attr('data-opeindex', index) .find('.number').text(amount+"€").end() .find('.name').text('Retrait').end() - .find('.amount').text(amountToUKF(amount, account_data['is_cof'])); + .find('.amount').text(amountToUKF(amount, account_data['is_cof'], false)); basket_container.prepend(withdraw_basket_html); updateBasketRel(); } @@ -903,7 +903,7 @@ $(document).ready(function() { var amount = $(this).find('#id_form-'+opeindex+'-amount'); if (!deleted && type == "purchase") amount.val(amountEuroPurchase(article_id, article_nb)); - basket_container.find('[data-opeindex='+opeindex+'] .amount').text(amountToUKF(amount.val(), account_data['is_cof'])); + basket_container.find('[data-opeindex='+opeindex+'] .amount').text(amountToUKF(amount.val(), account_data['is_cof'], false)); }); } @@ -922,9 +922,9 @@ $(document).ready(function() { basketrel_html += '
Sur 20€: '+ (20-amount).toFixed(2) +' €
'; } else if (account_data['trigramme'] != '' && !isBasketEmpty()) { var amount = getAmountBasket(); - var amountUKF = amountToUKF(amount, account_data['is_cof']); + var amountUKF = amountToUKF(amount, account_data['is_cof'], false); var newBalance = account_data['balance'] + amount; - var newBalanceUKF = amountToUKF(newBalance, account_data['is_cof']); + var newBalanceUKF = amountToUKF(newBalance, account_data['is_cof'], true); basketrel_html += '
Total: '+amountUKF+'
'; basketrel_html += '
Nouveau solde: '+newBalanceUKF+'
'; if (newBalance < 0)