Produits Entr'ouvert » Combo

Il y a un d'un côté ça qui laisse penser que sans restriction, l'élément est accepté :

            if not self.transports_restrictions:
                filtered.append(item)

Mais il y a de l'autre côté du code qui fait l'inverse :

     def check_transport(self, transport):
         restrictions = self.get_transports_restrictions()
         if restrictions and transport in restrictions:
             return True
         return False

L'erreur est a priori dans ce dernier, parce que le check_resource est lui écrit de la sorte :

    def check_resource(self, resource):
        restrictions = self.get_resources_restrictions()
        if restrictions and self.simplify(resource) not in restrictions:
            return False
        return True 

Cela étant, je n'ai pas de vue globale et je n'ai par exemple pas cherché pourquoi get_newsletters() retournait autant de lignes pour une newsletter que celle-ci avait de transports, c'est-à-dire :

            for t in item['transports']:
                if self.check_transport(t['id']):
                    filtered.append(item)

Pourquoi pas de break ici ?