Skip to content

T
third-party
Opened by  

2459-search-associate-only-in-members

{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on the diff
    Resolved by Damien Moulard
    members/models.py
    775 775 cond = [['name', 'ilike', str(key)]]
    776 776 cond.append('|')
    777 777 cond.append(['is_member', '=', True])
    778 cond.append(['is_associated_people', '=', True])
    778 if search_type != 'members':
    779 cond.append(['is_associated_people', '=', True])
    • Damien Moulard @DamienM commented
      Developer

      non, car si on fait une recherche "full" on veut ne veut pas que les associated people !

    • Damien Moulard @DamienM commented
      Developer

      pourquoi pas juste :

      if search_type == 'members': cond.append(['is_associated_people', '=', False])

      ?

    • François C. @fracolo commented
      Master

      Attention, il y a le caractère '|' qui est inséré pour faire un OU : je ne sais pas ce que ça donne s'il n'y a pas 2 éléments qui suivent

    • Etienne Freiss @e.freiss commented
      Developer

      Sans les deux éléments ça ne fonctionne plus en effet

      Edit : En fait j'ai rien dit ça marche meme quand il n'y en a qu'un

      Edited by Etienne Freiss
    • Etienne Freiss @e.freiss commented
      Developer

      Et c'est un OR et pas un AND vue qu'il y a le | si j'ai bien compris. Avant c'était toujours avec les deux lignes. La je souhaite juste que dans le cas ou c'est "members" on n'inclut pas les suppléants.

    • Damien Moulard @DamienM commented
      Developer

      Effectivement je n'avais pas vu le OR, j'ai rien dit !

    Please register or sign in to reply
  • Damien Moulard @DamienM

    resolved all discussions

    resolved all discussions

    Toggle commit list
  • Damien Moulard @DamienM

    merged

    merged

    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 6aa05fdd

    mentioned in commit 6aa05fdd

    Toggle commit list
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 to comment