Merge branch 'dev_cooperatic' into 1858-readonly-calendar

members/static/js/members.js

@@ -62,6 +62,7 @@ var html_elts = {
     real_capture : $('#real_capture'),
     multi_results : $('#multi_results_preview'),
     cooperative_state : $('#cooperative_state'),
+    status_explanation: $('#status_explanation'),
     next_shifts : $('#next_shifts')
 };
 
@@ -91,8 +92,12 @@ function fill_member_slide(member) {
     }
     html_elts.image_medium.html('<img src="'+img_src+'" width="128" />');
     html_elts.cooperative_state.html(member.cooperative_state);
-    if (member.cooperative_state == 'Désinscrit(e)' || member.cooperative_state == 'Rattrapage') coop_info.addClass('b_red');
-    else if (member.cooperative_state == 'En alerte' || member.cooperative_state == 'Délai accordé') coop_info.addClass('b_orange');
+    if (member.cooperative_state == 'Rattrapage') {
+      var explanation = "Tu as dû manquer un service! Pour pouvoir faire tes courses aujourd'hui, tu dois d'abord sélectionner un rattrapage sur ton espace membre."
+      html_elts.status_explanation.html(explanation)
+    }
+    if (member.cooperative_state == 'Désinscrit(e)') coop_info.addClass('b_red');
+    else if (member.cooperative_state == 'En alerte' || member.cooperative_state == 'Délai accordé' || member.cooperative_state == 'Rattrapage') coop_info.addClass('b_orange');
 
     if (member.shifts.length > 0) {
         html_elts.next_shifts.append('Prochains services : ');

members/static/js/subscription_validation.js

@@ -96,14 +96,7 @@ function process_form_submission(event) {
                     closeModal();
 
                     if (!err) {
-                        var msg = "Vous êtes maintenant enregistré ! ";
-
-                        msg += "<a href='" + em_url + "'>Cliquez ici</a> ";
-                        msg += "pour découvrir l'espace membre";
-                        $('p.intro').remove();
-                        vform.remove();
-                        display_msg_box(msg);
-
+                        window.location.href = em_url;
                     }
 
                 }

members_space/models.py

@@ -36,7 +36,7 @@ class CagetteMembersSpace(models.Model):
         try:
             cond = [
                 ['partner_id', '=', partner_id], 
-                ['create_date', '>', date_from],
+                ['date_begin', '>', date_from],
                 ['date_begin', '<', today],
                 ['state', '!=', 'draft'],
                 ['state', '!=', 'open'],
@@ -88,10 +88,10 @@ class CagetteMembersSpace(models.Model):
                     res = res + []
 
             # Add amnesty line
-            is_amnesty = getattr(settings, 'AMNISTIE_DATE', 'false')
+            is_amnesty = getattr(settings, 'AMNISTIE_DATE', False)
             company_code = getattr(settings, 'COMPANY_CODE', '')
             if is_amnesty and company_code == "lacagette":
-                amnesty={}
+                amnesty = {}
                 amnesty['is_amnesty'] = True
                 amnesty['create_date'] = is_amnesty
                 amnesty['date_begin'] = is_amnesty
@@ -107,7 +107,7 @@ class CagetteMembersSpace(models.Model):
             paginated_res = res[offset:end_index]
 
         except Exception as e:
-            print(str(e))
+            coop_logger.error("get_shifts_history : %s", str(e))
 
         return paginated_res
  
\ No newline at end of file

members_space/static/js/members-space-shifts-exchange.js

@@ -91,8 +91,10 @@ function add_or_change_shift(new_shift_id) {
                 selected_shift = null;
 
                 if (error.status === 400) {
-                    alert(`Désolé ! Le service que vous souhaitez échanger démarre dans moins de 24h. ` +
-                            `Il n'est plus possible de l'échanger.`);
+                    alert(`Désolé ! Le service que tu souhaites échanger démarre dans moins de 24h. ` +
+                          `Afin de faciliter la logistique des services, il n'est plus possible de l'échanger. ` +
+                          `Si tu ne peux vraiment pas venir, tu seras noté.e absent.e à ton service. ` +
+                          `Tu devras alors sélectionner un service de rattrapage sur ton espace membre.`);
                 } else {
                     alert(`Une erreur est survenue. ` +
                         `Il est néanmoins possible que la requête ait abouti, ` +

members_space/static/js/members-space.js

@@ -16,7 +16,8 @@ const possible_cooperative_state = {
     alert: "En alerte",
     up_to_date: "À jour",
     unsubscribed: "Désinscrit.e des créneaux",
-    delay: "En délai"
+    delay: "En délai",
+    gone: "Parti.e"
 };
 
 /* - Data */
@@ -177,7 +178,7 @@ function init_my_info_data() {
 
         $(".delay_date_stop").text(f_date_delay_stop);
         $(".delay_date_stop_container").show();
-    } else if (partner_data.cooperative_state === 'unsubscribed') {
+    } else if (partner_data.cooperative_state === 'unsubscribed' || partner_data.cooperative_state === 'gone') {
         $(".member_shift_name").text('X');
 
         $(".unsuscribed_form_link")

orders/static/css/oders_helper_style.css

@@ -197,6 +197,23 @@
     border-radius: 3px;
 }
 
+#coverage_form > div {
+    display:inline-block;
+    float:left;
+}
+
+#coverage_form .input-wrapper {
+    margin-right: 3px;
+}
+
+#coverage_days_input, #percent_adjust_input {
+    display:  block;
+}
+
+#coverage_days_input {
+    margin-bottom: 3px;
+}
+
 /* -- Table */
 #products_table_filter{
     text-align: right !important;
@@ -224,6 +241,10 @@
     margin-left: 5px;
 }
 
+.main.fa-info-circle {
+    color: #0275d8;
+    cursor:  pointer;
+}
 .custom_cell_content {
     display: flex;
     flex-direction: column;
@@ -346,6 +367,15 @@
     width: 90%;
 }
 
+/* product actions modal*/
+.npa-options {
+    width: fit-content;
+    text-align: left;
+    margin: auto;
+}
+.npa-options label {
+    display:  block;
+}
 /* - Orders created screen */
 
 .order_created_header {
@@ -392,6 +422,8 @@
     padding: 15px;
 }
 
+
+
 /* - Miscellaneous */
 
 footer {

orders/views.py

 from outils.common_imports import *
 from outils.for_view_imports import *
+from outils.common import OdooAPI
 
 from orders.models import Order, Orders, CagetteSuppliers
 from products.models import CagetteProduct, CagetteProducts
@@ -20,7 +21,9 @@ def helper(request):
         'couchdb_server': settings.COUCHDB['url'],
         'db': settings.COUCHDB['dbs']['orders'],
         'odoo_server': settings.ODOO['url'],
-        'metabase_url':  getattr(settings, 'ORDERS_HELPER_METABASE_URL', '')
+        'metabase_url': getattr(settings, 'ORDERS_HELPER_METABASE_URL', ''),
+        'nb_past_days_to_compute_sales_average': OdooAPI().get_system_param('lacagette_products.nb_past_days_to_compute_sales_average'),
+        'nb_of_consecutive_non_sale_days_considered_as_break': OdooAPI().get_system_param('lacagette_products.nb_of_consecutive_non_sale_days_considered_as_break')
     }
 
     template = loader.get_template('orders/helper.html')

outils/common.py

@@ -74,6 +74,18 @@ class OdooAPI:
     def authenticate(self, login, password):
         return self.common.authenticate(self.db, login, password, {})
 
+    def get_system_param(self, key):
+        value = ''
+        try:
+            res = self.search_read('ir.config_parameter',
+                                   [['key', '=', key]],
+                                   ['value'])
+            if res:
+                value = res[0]['value']
+        except Exception as e:
+            coop_logger.error('get_system_param: (%s) %s', key, str(e))
+        return value
+
 class CouchDB:
     """Class to handle interactions with CouchDB"""
 

products/models.py

@@ -7,6 +7,7 @@ import csv
 import tempfile
 import pymysql.cursors
 import datetime
+import re
 
 vcats = []
 
@@ -252,10 +253,43 @@ class CagetteProduct(models.Model):
 
         return res
 
-
+    @staticmethod
+    def update_npa_and_minimal_stock(data):
+        """Update NPA (ne pas acheter) and minimal stock data"""
+        res = {}
+        try:
+            api = OdooAPI()
+            f = {'minimal_stock': data['minimal_stock']}
+
+            if 'simple-npa' in data['npa']:
+                f['purchase_ok'] = 0
+            if 'npa-in-name' in data['npa']:
+                #  Add [NPA] in product name if needed
+                f['name'] = data['name'] if ('[NPA]' in data['name']) else data['name'] + " [NPA]"
+                f['purchase_ok'] = 0
+            elif '[NPA]' in data['name']:
+                #  Remove [NPA] from name
+                f['name'] = re.sub(r'( \[NPA\])', '', data['name'])
+
+            current_name = data['name'] if ('name' not in f) else f['name']
+
+            if 'fds-in-name' in data['npa']:
+                f['name'] = current_name if '[FDS]' in data['name'] else current_name + " [FDS]"
+                f['purchase_ok'] = 0
+            elif '[FDS]' in current_name:
+                f['name'] = re.sub(r'( \[FDS\])', '', current_name)
+
+            if len(data['npa']) == 0:
+                f['purchase_ok'] = 1
+            res["update"] = api.update('product.template', data['id'], f)
+        except Exception as e:
+            res["error"] = str(e)
+            coop_logger.error("update_npa_and_minimal_stock : %s %s", str(e), str(data))
+        return res
 class CagetteProducts(models.Model):
     """Initially used to make massive barcode update."""
 
+
     @staticmethod
     def get_simple_list():
         res = []
@@ -577,7 +611,8 @@ class CagetteProducts(models.Model):
                 "uom_id",
                 "purchase_ok",
                 "supplier_taxes_id",
-                "product_variant_ids"
+                "product_variant_ids",
+                "minimal_stock"
             ]
             c = [['id', 'in', ptids], ['purchase_ok', '=', True]]
             products_t = api.search_read('product.template', c, f)

products/urls.py

@@ -11,6 +11,7 @@ urlpatterns = [
     url(r'^update_product_stock$', views.update_product_stock),
     url(r'^update_product_purchase_ok$', views.update_product_purchase_ok),
     url(r'^update_product_internal_ref$', views.update_product_internal_ref),
+    url(r'^update_npa_and_minimal_stock$', views.update_npa_and_minimal_stock),
     url(r'^labels_appli_csv(\/?[a-z]*)$', views.labels_appli_csv, name='labels_appli_csv'),
     url(r'^label_print/([0-9]+)/?([0-9\.]*)/?([a-z]*)/?([0-9]*)$', views.label_print),
     url(r'^shelf_labels$', views.shelf_labels), # massive print

products/views.py

@@ -3,6 +3,7 @@
 from outils.common_imports import *
 from outils.for_view_imports import *
 
+from members.models import CagetteUser
 from products.models import CagetteProduct
 from products.models import CagetteProducts
 from inventory.models import CagetteInventory
@@ -105,6 +106,8 @@ def update_product_stock(request):
 
 def update_product_purchase_ok(request):
     res = {}
+    is_connected_user = CagetteUser.are_credentials_ok(request)
+    if is_connected_user is True:
         data = json.loads(request.body.decode())
 
         res = CagetteProduct.update_product_purchase_ok(data["product_tmpl_id"], data["purchase_ok"])
@@ -113,9 +116,13 @@ def update_product_purchase_ok(request):
             return JsonResponse(res, status=500)
         else:
             return JsonResponse({"res": res})
+    else:
+        return JsonResponse(res, status=403)
 
 def update_product_internal_ref(request):
     res = {}
+    is_connected_user = CagetteUser.are_credentials_ok(request)
+    if is_connected_user is True:
         data = json.loads(request.body.decode())
 
         res = CagetteProduct.update_product_internal_ref(data["product_tmpl_id"], data["default_code"])
@@ -124,6 +131,26 @@ def update_product_internal_ref(request):
             return JsonResponse(res, status=500)
         else:
             return JsonResponse({"res": res})
+    else:
+        return JsonResponse(res, status=403)
+
+def update_npa_and_minimal_stock(request):
+    res = {}
+    is_connected_user = CagetteUser.are_credentials_ok(request)
+    if is_connected_user is True:
+        try:
+            data = json.loads(request.body.decode())
+            res = CagetteProduct.update_npa_and_minimal_stock(data)
+        except Exception as e:
+            res['error'] = str(e)
+            coop_logger.error("Update npa and minimal stock : %s", res['error'])
+
+        if ('error' in res):
+            return JsonResponse(res, status=500)
+        else:
+            return JsonResponse({"res": res})
+    else:
+        return JsonResponse(res, status=403)
 
 def labels_appli_csv(request, params):
     """Generate files to put in DAV directory to be retrieved by scales app."""

shifts/models.py

@@ -155,6 +155,8 @@ class CagetteShift(models.Model):
                             "origin": 'memberspace',
                             "is_makeup": data['is_makeup'],
                             "state": 'open'}
+            if shift_type == "standard":
+                fieldsDatas['template_created'] = 1  # It's not true but otherwise, presence add 1 standard point, which is not wanted
 
             st_r_id = self.o_api.create('shift.registration', fieldsDatas)
         except Exception as e:
@@ -211,7 +213,7 @@ class CagetteShift(models.Model):
         action = 'create'
 
         # Get partner extension ids
-        cond = [['id','=',data['idPartner']]]
+        cond = [['id', '=', data['idPartner']]]
         fields = ['extension_ids']
         partner_extensions = self.o_api.search_read('res.partner', cond, fields)
         response = False

templates/members/index.html

@@ -107,6 +107,8 @@
                 </div>
                 <div id="cooperative_state">
                 </div>
+                <div id="status_explanation" style="font-weight:bold;">
+                </div>
                  <div id="next_shifts">
                 </div>
               </div>

templates/members_space/faq.html

@@ -107,7 +107,7 @@
             <div class="input-container panel">
               
               <div class="grp_text">
-                <h3><b>Créneau/service c'est quoi la différence?</b></h3>
+                <h3><b>Créneau/service c'est quoi la différence ?</b></h3>
                 <p>
                   Un créneau, c'est une plage récurrente, par exemple, tous les jeudi de semaine A de 13h30 à 16h30.<br/>
                   Un service, c'est une plage horaire en particulier, par exemple le jeudi 28 novembre de 13h30 à 16h30.
@@ -116,7 +116,7 @@
               </div>
 
               <div class="grp_text">
-                <h3><b>Échanger son service:</b></h3>
+                <h3><b>Échanger son service</b></h3>
                 <p>
                   Si tu ne peux pas venir effectuer ton service à la date prévue il te faut l'échanger sur ton espace membre.<br/>
                   <b>ATTENTION:</b> si tu es en binôme c'est via l'espace membre du titulaire du binôme qu'il faut le faire.
@@ -134,6 +134,24 @@
               </div>
 
               <div class="grp_text">
+                <h3><b>Oubli de validation de service</b></h3>
+                <p>
+                  Si tu as effectué ton service mais que tu as oublié de le valider à l'entrée du magasin, il te faut remplir le formulaire suivant en indiquant la date de ton service.
+                </p>
+                <div class="faq_link_button_area">
+                  <a
+                    href="javascript:void(0);"
+                    target="_blank"
+                    type="button"
+                    class="btn--primary faq_link_button"
+                    id="request_form_link_btn2"
+                  >
+                    Oubli validation service
+                  </a>
+                </div>
+              </div>
+              
+              <div class="grp_text">
                 <h3><b>Demande de congé maladie ou parental</b></h3>
                 <p><b>
                   - Si tu t'absentes sur une période de 4 semaines autre que pour une raison de santé ou parentale il t'es demandé de déplacer ton service.<br/>
@@ -142,9 +160,8 @@
                 <p></b></p>
               </div>
               
-              <div class="grp_text">
-                <b><h3>Les congés maladie:</h3></b>
-                Les congés maladie peuvent être pris en cas d’impossibilité physique de réaliser son service pendant une longue durée qui prend en compte<b> minimum 2 services</b>. Si ton impossibilité est d'un mois il t'es demandé de déplacer ton service.<br />
+              <div class="grp_text"><h3><b>Les congés maladie</b></h3>
+                Les congés maladie peuvent être pris en cas d’impossibilité physique de réaliser son service pendant une longue durée qui prend en compte<b> minimum 2 services</b>. Si ton impossibilité est d'un mois il t'est demandé de déplacer ton service.<br />
                 En cas de rhume nous encourageons les coops à simplement déplacer leur service depuis l’espace membre. La coop a besoin de la participation de chacun.e !<br />
                 <b>ATTENTION:</b> Compte tenu de la facilité offerte par le statut de binôme, les personnes formant un binôme ne pourront pas<br />
                 bénéficier de congés maladie. Il sera alors possible de se désolidariser de son binôme en remplissant le formulaire adéquat sur l’espace membre afin d’en bénéficier.<br />
@@ -398,7 +415,9 @@
                 Nous avons créé des formulaires spécifiques pour la plupart des problèmes rencontrés par les membres. Changer de créneau, créer un binôme, ajouter un produit à la gamme, partir en vacances... <br />
                 Cela dit, nous en découvrons de nouveaux tous les jours.<br />
                 Si tu n'as pas su quel formulaire remplir, tu es au bon endroit. <br />
-                Vas-y dit nous tout !<br />
+                Vas-y dit nous tout !<br /><br />
+                Attention: si tu souhaites contacter le BDM pour prévenir que tu seras absent-e à ton service cela ne sert à rien! Il te faut déplacer ton service via ton espace membre. Il n'est cependant pas possible d'échanger un service qui commence dans moins de 24h pour des raisons de logistiques. Si tu ne peux pas venir tu seras donc comptabilisé-e absent-e. Tu basculeras en statut "Rattrapage" et ne pourras plus faire tes courses. Il te faudra sélectionner dans ton espace membre un rattrapage à faire dans les 6 prochains mois pour basculer en statut "Délai" et pouvoir faire de nouveau tes courses.<br />
+                Merci de ne pas contacter le Bureau des membres pour cela, il te donnera exactement la même réponse.<br />
                 <div class="faq_link_button_area">
                   <a
                     href="javascript:void(0);"

templates/orders/explanations.html

+<h3>Comment sont calculées les conso. moyennes / jour ?</h3>
+<div class="txtleft">
+    <p>
+        La fonction qui calcule les consommations moyennes prend en paramètre une date de départ. <br/>
+        Si elle n'est pas indiquée, la date prise en compte sera "<em>aujourd'hui - nb de jours paramétré dans Odoo</em>".<br/>
+        Ce nombre de jours paramétrable se trouve en suivant les menus suivants :<br>
+        Configuration > Technique > Paramètres > Paramètres systèmes. <br/>
+        La valeur est définie avec la clef "<em>lacagette_products.nb_past_days_to_compute_sales_average</em>".<br/>
+        Ce paramètre vaut <strong>{{nb_past_days_to_compute_sales_average}}</strong> au moment du chargement de cette page.<br/>
+        Les ventes du dimanche sont exclues du calcul.<br/>
+    </p>
+    <p>
+        Une requête est faite sur l'ensemble des passages en caisse, de la date de départ à hier, récupérant pour tous les jours de la période (dimanches exclus) les quantités vendues des articles achetés chez le fournisseur.<br/>
+        Pour chaque article, le nombre de jours considérés pour faire la moyenne est défini commme suit :<br/>
+        <em>Nb de jours ouvrés de la période - Nb de jours des périodes de rupture</em><br/>
+        Les périodes de ruptures sont caractérisées par un nombre de jours consécutifs assez important de jours sans vente de l'article.<br/>
+        Le nombre de jours consécutifs sans vente pour considérer l'article en rupture est actuellement de <strong>{{nb_of_consecutive_non_sale_days_considered_as_break}}</strong><br/>
+        (c'est le paramètre système avec la clef "<em>lacagette_products.nb_of_consecutive_non_sale_days_considered_as_break</em>").<br/>
+    </p>
+    <p>
+        Pour chaque article, la consommation moyenne par jour est obtenue en divisant la quantité totale vendue sur la période par le nombre de jours significatifs.
+    </p>
+</div>

templates/orders/helper.html

@@ -93,8 +93,13 @@
                 </select>
             </form>
             <form action="javascript:;" id="coverage_form" class="order_form_item">
+                <div class="input-wrapper">
                     <input type="number" name="coverage_days" id="coverage_days_input" placeholder="Nb jours de couverture" min="1">
-                <button type="submit" class='btn--primary'>Calculer les besoins</button>
+                    <input type="number" name="percent_adjustement" id="percent_adjust_input" placeholder="ajustement en %">
+                </div>
+                <div>
+                    <button  type="submit" class='btn--primary'>Calculer les besoins</button> <i class='main fa fa-info-circle fa-lg'></i>
+                </div>
             </form>
         </div>
     
@@ -270,14 +275,21 @@
             <hr/>
         </div>
 
-        <div id="modal_product_npa">
-            <h3>Attention !</h3>
+        <div id="modal_product_actions">
+            Actions sur <h3><span class="product_name"></span></h3>
             <p>
-                Vous vous apprêtez à passer le produit <span class="product_name"></span> en <span class="product_npa"></span>.<br/>
-                Dès que vous aurez cliqué sur "Valider", le produit sera retiré du tableau et l'information sera enregistrée dans Odoo.
+                <h4>NPA</h4>
+                <div class="npa-options">
+                    <label><input type="checkbox" name="npa-actions" value="simple-npa" /> Mettre le produit en NPA </label>
+                    <label><input type="checkbox" name="npa-actions" value="npa-in-name" /> Mettre le produit en NPA et afficher NPA</label>
+                    <label><input type="checkbox" name="npa-actions" value="fds-in-name" /> Mettre le produit en NPA et afficher FDS</label>
+                </div>
             </p>
-            <p>Êtez-vous sûr ?</p>
-            <hr/>
+            <p>
+                <h4>Stock minimum</h4>
+                <input type="number" name="minimal_stock" value="" />
+            </p>
+
         </div>
 
         <div id="modal_create_order">
@@ -294,7 +306,9 @@
             <br/>
             <hr/>
         </div>
-
+        <div id="explanations">
+            {% include "orders/explanations.html" %}
+        </div>
         <div id="modal_create_order__supplier_date_planned">
             <div class="modal_input_area">
                 <span class="modal_input_label supplier_name"></span>