Skip to content

Commit 7ad6afa

Browse files
authored
Merge pull request #34 from backdrop-contrib/26-warning-undefined-array-key-storage
check for $form_state['storage'] existence
2 parents 49dec0b + c4c0339 commit 7ad6afa

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

modules/openai_chatgpt/openai_chatgpt.module

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -181,30 +181,33 @@ function openai_chatgpt_form_submit($form, &$form_state) {
181181

182182
$text = $form_state['values']['text'];
183183
$system = $form_state['values']['system'];
184-
$model = $form_state
185-
['values']['model'];
184+
$model = $form_state['values']['model'];
186185
$temperature = $form_state['values']['temperature'];
187186
$max_tokens = $form_state['values']['max_tokens'];
188187

189188
// Now pass the API key to the constructor
190189
$api = new OpenAIApi($apiKey);
191190

192-
$storage = $form_state['storage'];
191+
// Ensure storage is initialized
192+
if (!isset($form_state['storage']) || !is_array($form_state['storage'])) {
193+
$form_state['storage'] = [];
194+
}
193195

194-
if (!empty($storage['messages'])) {
195-
$messages = $storage['messages'];
196+
if (!empty($form_state['storage']['messages'])) {
197+
$messages = $form_state['storage']['messages'];
196198
$messages[] = ['role' => 'user', 'content' => trim($text)];
197199
} else {
198200
$messages = [
199201
['role' => 'system', 'content' => trim($system)],
200-
['role' => 'user', 'content' => trim($text)]
202+
['role' => 'user', 'content' => trim($text)],
201203
];
202204
}
203205

204206
$result = $api->chat($model, $messages, $temperature, $max_tokens);
205207
$messages[] = ['role' => 'assistant', 'content' => $result];
206-
$form_state['storage'] = ['messages' => $messages];
208+
$form_state['storage']['messages'] = $messages;
207209
$form_state['rebuild'] = TRUE;
208210
}
209211

210212

213+

0 commit comments

Comments
 (0)