Upgrade from pylint v2.8.3 to 2.9.3 raise some new issues::
  searx/search/checker/__main__.py:37:26: R1732: Consider using 'with' for resource-allocating operations (consider-using-with)
  searx/search/checker/__main__.py:38:26: R1732: Consider using 'with' for resource-allocating operations (consider-using-with)
  searx/search/processors/__init__.py:20:0: R0402: Use 'from searx import engines' instead (consider-using-from-import)
  searx/preferences.py:182:19: C0207: Use data.split('-', maxsplit=1)[0] instead (use-maxsplit-arg)
  searx/preferences.py:506:15: R1733: Unnecessary dictionary index lookup, use 'user_setting' instead (unnecessary-dict-index-lookup)
  searx/webapp.py:436:0: C0206: Consider iterating with .items() (consider-using-dict-items)
  searx/webapp.py:950:4: C0206: Consider iterating with .items() (consider-using-dict-items)
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
		
	
			
		
			
				
	
	
		
			71 lines
		
	
	
		
			2.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			71 lines
		
	
	
		
			2.1 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
# SPDX-License-Identifier: AGPL-3.0-or-later
 | 
						|
# lint: pylint
 | 
						|
 | 
						|
"""Implement request processores used by engine-types.
 | 
						|
 | 
						|
"""
 | 
						|
 | 
						|
__all__ = [
 | 
						|
    'EngineProcessor',
 | 
						|
    'OfflineProcessor',
 | 
						|
    'OnlineProcessor',
 | 
						|
    'OnlineDictionaryProcessor',
 | 
						|
    'OnlineCurrencyProcessor',
 | 
						|
    'PROCESSORS',
 | 
						|
]
 | 
						|
 | 
						|
import threading
 | 
						|
 | 
						|
from searx import logger
 | 
						|
from searx import engines
 | 
						|
 | 
						|
from .online import OnlineProcessor
 | 
						|
from .offline import OfflineProcessor
 | 
						|
from .online_dictionary import OnlineDictionaryProcessor
 | 
						|
from .online_currency import OnlineCurrencyProcessor
 | 
						|
from .abstract import EngineProcessor
 | 
						|
 | 
						|
logger = logger.getChild('search.processors')
 | 
						|
PROCESSORS = {}
 | 
						|
"""Cache request processores, stored by *engine-name* (:py:func:`initialize`)"""
 | 
						|
 | 
						|
def get_processor_class(engine_type):
 | 
						|
    """Return processor class according to the ``engine_type``"""
 | 
						|
    for c in [OnlineProcessor, OfflineProcessor, OnlineDictionaryProcessor, OnlineCurrencyProcessor]:
 | 
						|
        if c.engine_type == engine_type:
 | 
						|
            return c
 | 
						|
    return None
 | 
						|
 | 
						|
 | 
						|
def get_processor(engine, engine_name):
 | 
						|
    """Return processor instance that fits to ``engine.engine.type``)"""
 | 
						|
    engine_type = getattr(engine, 'engine_type', 'online')
 | 
						|
    processor_class = get_processor_class(engine_type)
 | 
						|
    if processor_class:
 | 
						|
        return processor_class(engine, engine_name)
 | 
						|
    return None
 | 
						|
 | 
						|
 | 
						|
def initialize_processor(processor):
 | 
						|
    """Initialize one processor
 | 
						|
 | 
						|
    Call the init function of the engine
 | 
						|
    """
 | 
						|
    if processor.has_initialize_function:
 | 
						|
        t = threading.Thread(target=processor.initialize, daemon=True)
 | 
						|
        t.start()
 | 
						|
 | 
						|
 | 
						|
def initialize(engine_list):
 | 
						|
    """Initialize all engines and store a processor for each engine in :py:obj:`PROCESSORS`."""
 | 
						|
    for engine_data in engine_list:
 | 
						|
        engine_name = engine_data['name']
 | 
						|
        engine = engines.engines.get(engine_name)
 | 
						|
        if engine:
 | 
						|
            processor = get_processor(engine, engine_name)
 | 
						|
            initialize_processor(processor)
 | 
						|
            if processor is None:
 | 
						|
                logger.error('Error get processor for engine %s', engine_name)
 | 
						|
            else:
 | 
						|
                PROCESSORS[engine_name] = processor
 |