Content Generator & openAI db logs

Created on 11 September 2023, 10 months ago

Updated 3 November 2023, 8 months ago

Problem/Motivation

Getting below errors on admin/reports/dblog/event/{id} page

 TypeError: nl2br(): Argument #1 ($string) must be of type string, null given in nl2br() (line 107 of modules/contrib/openai/modules/openai_dblog/src/Controller/OpenAIDbLogController.php).

Warning: Undefined variable $explanation in Drupal\openai_dblog\Controller\OpenAIDbLogController->eventDetails() (line 107 of modules/contrib/openai/modules/openai_dblog/src/Controller/OpenAIDbLogController.php).

Steps to reproduce

Enable Open AI and Open AI logs module
Configure API key and Organization
Add content using Open AI content generator

Proposed resolution

Proper check should be added for `$explanation`

$rows[] = [
        [
          'data' => $header,
          'header' => TRUE,
        ],
        [
          'data' => [
            '#markup' => nl2br($explanation) ?? 'No possible explanations were found, or the API service is not responding.',
          ],
        ],
      ];

in OpenAIDbLogControllercontroller

🐛 Bug report

Status

Fixed

Version

1.0

Component

Code

Created by

🇮🇳India sonvir249 Pune

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @sonvir249
@sonvir249 opened merge request.
Status changed to Needs review 10 months ago7:04am 11 September 2023
Comment 10 months ago →
🇮🇳India sonvir249 Pune
Issue was unassigned.
Status changed to RTBC 10 months ago9:04am 11 September 2023
Comment 10 months ago →
🇮🇳India Nitesh Sethia
Looks good and is ready to be merged to the dev instance.
Status changed to Fixed 8 months ago4:26pm 3 November 2023
Comment 8 months ago →
🇺🇸United States kevinquillen
Comment 7 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024