ClicShopping #375

    foreach ($models as $model) {
      $modelId = $model['id'];
      $provider = $model['provider'] ?? 'openai'; // Default to openai if missing
      // Validate provider name
      if (!in_array($provider, $validProviders)) {

    
in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    

                    
Time to fix: about 15 minutes
                Read doc
                                 Permalink
         Copy Prompt
    

            

            

                                    Last edited Tue, 17 Feb 2026 13:08:46 +0000 by clicshopping
                    
                            
        
    
        if (defined('CLICSHOPPING_APP_CHATGPT_CH_DEBUG') && CLICSHOPPING_APP_CHATGPT_CH_DEBUG === 'True') {
          error_log("WARNING: Invalid provider '$provider' for model '$modelId'. Using 'openai' as default.");
        }
        $provider = 'openai';
      }

        'open-mistral-nemo',
        'open-codestral-mamba',
        'mistral-moderation-latest'
      ];
      if (empty($model) || !in_array($model, $valid_models)) {

    
in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    

                    
Time to fix: about 15 minutes
                Read doc
                                 Permalink
         Copy Prompt
    

            

            

                                    Last edited Tue, 17 Feb 2026 13:08:46 +0000 by clicshopping
                    
                            
        
    
        $model = 'mistral-large-latest';
      }
      $config = new MistralAIChat();
      $config->apiKey = $api_key;

    if ($intent === null || !is_array($intent)) {
      return null;
    }
    // Validate required fields
    if (!isset($intent['intent']) || !in_array($intent['intent'], ['price_comparison', 'product_discovery', 'market_research', 'trend_analysis'])) {

    
in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    

                    
Time to fix: about 15 minutes
                Read doc
                                 Permalink
         Copy Prompt
    

            

            

                                    Last edited Wed, 13 May 2026 12:12:21 +0000 by clicshopping
                    
                            
        
    
      return null;
    }
    // Ensure all fields exist with defaults
    $validatedIntent = [

    }
    $prefix = CLICSHOPPING::getConfig('db_table_prefix');
    
    // Remove prefix and '_embedding' suffix to get entity type
    $entityType = str_replace([$prefix, '_embedding'], '', $tableName);

    
Use the first-class callable syntax ($this->method(...))
    

                    
Time to fix: about 15 minutes
                Read doc
                                 Permalink
         Copy Prompt
    

            

            

                                    Last edited Thu, 18 Dec 2025 01:44:05 +0000 by clicshopping
                    
                            
        
    
    
    // Get ID column for this entity type
    $idColumn = $this->getIdColumnForEntityType($entityType);
    
    // Cache the result

  public function getEntityTypeForTable(string $tableName): string
  {
    $prefix = CLICSHOPPING::getConfig('db_table_prefix');
    
    // Remove prefix and '_embedding' suffix
    $entityType = str_replace([$prefix, '_embedding'], '', $tableName);

    
Use the first-class callable syntax ($this->method(...))
    

                    
Time to fix: about 15 minutes
                Read doc
                                 Permalink
         Copy Prompt
    

            

            

                                    Last edited Thu, 18 Dec 2025 01:44:05 +0000 by clicshopping
                    
                            
        
    
    
    return $entityType;
  }
  /**