Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
O
odoo
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • cooperatic-foodcoops
  • odoo
  • Merge Requests
  • !30

Merged
Opened Feb 23, 2022 by Félicie@felicie 
  • Report abuse
Report abuse

2175 search coop number

Mettre à jour les applications, vérifier dans la config/paramètres/paramètres systèmes/lacagette_cpo.reduce_pos_members_search que la valeur est à false

  • Discussion 4
  • Commits 5
  • Changes 3
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Feb 23, 2022
    Resolved by Damien Moulard Feb 23, 2022
    lacagette_addons/lacagette_custom_pos/static/src/js/backend.js
    2 2 "use strict";
    3 3
    4 4 var PosDB = require('point_of_sale.DB');
    5 var Model = require('web.DataModel');
    6 var config_parameter = new Model('ir.config_parameter');
    7 var reduce_pos_members_search = null;
    5 8
    6 9 PosDB.include({
    10 init: function(parent, options){
    11 this._super(parent,options);
    12 config_parameter.call('get_param', ['lacagette_cpo.reduce_pos_members_search'])
    13 .then(function(param){
    14 if (param != undefined) {
    15 reduce_pos_members_search = param;
    16 console.log('init', reduce_pos_members_search);
    • Damien Moulard @DamienM commented Feb 23, 2022
      Developer

      enlever le console.log

      enlever le console.log
    • Félicie @felicie

      changed this line in version 2 of the diff

      Feb 23, 2022

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1373&start_sha=128a653238452a908c404fec8415b91cd0aa601f#09c2a592b9faf1ff5e73d841a00df879e1bf7776_16_16)
      Toggle commit list
    Please register or sign in to reply
  • Félicie @felicie

    added 1 commit

    • 832245d2 - console log removed

    Compare with previous version

    Feb 23, 2022

    added 1 commit

    • 832245d2 - console log removed

    Compare with previous version

    added 1 commit <ul><li>832245d2 - console log removed</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1373&start_sha=128a653238452a908c404fec8415b91cd0aa601f)
    Toggle commit list
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Feb 23, 2022
    Resolved by Damien Moulard Feb 23, 2022
    lacagette_addons/lacagette_cpo/data/ir_config_parameter_data.xml
    4 4 <field name="key">lacagette_cpo.proxy_url</field>
    5 5 <field name="value">https://tools.lacagette-coop.fr/inventory/inventory/generate_inventory_list</field>
    6 6 </record>
    7 <record id="reduce_pos_members_search" model="ir.config_parameter">
    • Damien Moulard @DamienM commented Feb 23, 2022
      Developer

      Je me rend compte qu'on a fait une bêtise ! Il vaut mieux mettre la définition de ce paramètre dans le même module que là où il est utilisé.

      C'est-à-dire que le fichier dans lequel tu t'en sers est dans le module lacagette_custom_pos, alors que là on est dans lacagette_cpo.

      Il faudrait donc :

      • dans lacagette_custom_pos, créer un dossier data et dedans un fichier ir_config_parameter_data.xml avec la même strucure que celui-ci
      • Déplacer la définition de ce paramètre (balise ) dans ce nouveau fichier.

      Tu aura besoin de mettre à jour ces deux modules dans odoo.

      Edited Feb 23, 2022 by Damien Moulard
      Je me rend compte qu'on a fait une bêtise ! Il vaut mieux mettre la définition de ce paramètre dans le même module que là où il est utilisé. C'est-à-dire que le fichier dans lequel tu t'en sers est dans le module `lacagette_custom_pos`, alors que là on est dans `lacagette_cpo`. Il faudrait donc : * dans `lacagette_custom_pos`, créer un dossier `data` et dedans un fichier `ir_config_parameter_data.xml` avec la même strucure que celui-ci * Déplacer la définition de ce paramètre (balise <record>) dans ce nouveau fichier. Tu aura besoin de mettre à jour ces deux modules dans odoo.
    • Félicie @felicie

      changed this line in version 3 of the diff

      Feb 23, 2022

      changed this line in version 3 of the diff

      changed this line in [version 3 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1374&start_sha=832245d212ca29b4a52c1990a372e3a7c8b94d36#470971a2e9b8b0add25136f78ce326ffa69f015c_7_7)
      Toggle commit list
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Feb 23, 2022
    Resolved by Damien Moulard Feb 23, 2022
    lacagette_addons/lacagette_custom_pos/static/src/js/backend.js
    12 26 } catch(e) {
    13 27 console.log(e)
    14 28 }
    15 if(partner.barcode){
    29
    30 if (reduce_pos_members_search === 'True') {
    • Damien Moulard @DamienM commented Feb 23, 2022
      Developer

      c'est l'inverse : === 'False' (si on veut reduce, on veut ne pas passer à l'intérieur du if)

      c'est l'inverse : `=== 'False'` (si on veut reduce, on veut ne pas passer à l'intérieur du if)
    • Félicie @felicie commented Feb 23, 2022
      Developer

      Ah oui je vois, et dans la configuration d'odoo, il faudra mettre le paramètre à True si on ne veut pas passer à l'intérieur de ce if !

      Ah oui je vois, et dans la configuration d'odoo, il faudra mettre le paramètre à True si on ne veut pas passer à l'intérieur de ce if !
    • Félicie @felicie

      changed this line in version 3 of the diff

      Feb 23, 2022

      changed this line in version 3 of the diff

      changed this line in [version 3 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1374&start_sha=832245d212ca29b4a52c1990a372e3a7c8b94d36#09c2a592b9faf1ff5e73d841a00df879e1bf7776_29_29)
      Toggle commit list
    Please register or sign in to reply
  • Félicie @felicie

    added 1 commit

    • d70ec7c1 - parameter changed

    Compare with previous version

    Feb 23, 2022

    added 1 commit

    • d70ec7c1 - parameter changed

    Compare with previous version

    added 1 commit <ul><li>d70ec7c1 - parameter changed</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1374&start_sha=832245d212ca29b4a52c1990a372e3a7c8b94d36)
    Toggle commit list
  • Félicie @felicie

    added 3 commits

    • d70ec7c1...b414b37c - 2 commits from branch dev_cooperatic
    • 20ded4b7 - Merge branch 'dev_cooperatic' into 2175-serch-coop-number

    Compare with previous version

    Feb 23, 2022

    added 3 commits

    • d70ec7c1...b414b37c - 2 commits from branch dev_cooperatic
    • 20ded4b7 - Merge branch 'dev_cooperatic' into 2175-serch-coop-number

    Compare with previous version

    added 3 commits <ul><li>d70ec7c1...b414b37c - 2 commits from branch <code>dev_cooperatic</code></li><li>20ded4b7 - Merge branch &#39;dev_cooperatic&#39; into 2175-serch-coop-number</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/odoo/merge_requests/30/diffs?diff_id=1375&start_sha=d70ec7c11c6636a14cd69053325cafbbbac9e084)
    Toggle commit list
  • Damien Moulard @DamienM

    resolved all discussions

    Feb 23, 2022

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    Feb 23, 2022

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit bbde8038

    Feb 23, 2022

    mentioned in commit bbde8038

    mentioned in commit bbde80383d356f91b0fa27edba3118c94d7c54de
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: cooperatic-foodcoops/odoo!30
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.