[PR #22704] [CLOSED] fix: resolve UnboundLocalError for limit in oracle23ai.py get() #42453

New Issue

GiteaMirror · 2026-04-25T14:21:11-05:00

GiteaMirror commented

2026-04-25 14:21:11 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/22704
Author: @mvanhorn
Created: 3/15/2026
Status: ❌ Closed

Base: dev ← Head: osc/22664-oracle23ai-unbound-limit-v2

📝 Commits (1)

e1312a6 fix: resolve UnboundLocalError for limit in oracle23ai.py get()

📊 Changes

1 file changed (+3 additions, -4 deletions)

View changed files

📝 backend/open_webui/retrieval/vector/dbs/oracle23ai.py (+3 -4)

📄 Description

Summary

Moves the limit = 1000 assignment before the log.info() call that references it in the get() method of oracle23ai.py.

Changes

The get() method signature is def get(self, collection_name: str) with no limit parameter. The log.info() at line 714 referenced limit before it was assigned at line 718, causing an UnboundLocalError when Oracle 23ai hybrid search was activated.

Testing

Verified the function signature has no limit parameter
Verified the log statement now executes after limit is assigned

Fixes #22664
Closes #22616

This contribution was developed with AI assistance (Claude Code).

Contributor License Agreement

By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/open-webui/pull/22704 **Author:** [@mvanhorn](https://github.com/mvanhorn) **Created:** 3/15/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `osc/22664-oracle23ai-unbound-limit-v2` --- ### 📝 Commits (1) - [`e1312a6`](https://github.com/open-webui/open-webui/commit/e1312a60a45dd393001f95317e2e62a911c45b17) fix: resolve UnboundLocalError for limit in oracle23ai.py get() ### 📊 Changes **1 file changed** (+3 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/retrieval/vector/dbs/oracle23ai.py` (+3 -4) </details> ### 📄 Description ## Summary Moves the `limit = 1000` assignment before the `log.info()` call that references it in the `get()` method of `oracle23ai.py`. ## Changes The `get()` method signature is `def get(self, collection_name: str)` with no `limit` parameter. The `log.info()` at line 714 referenced `limit` before it was assigned at line 718, causing an `UnboundLocalError` when Oracle 23ai hybrid search was activated. ## Testing - Verified the function signature has no `limit` parameter - Verified the log statement now executes after `limit` is assigned Fixes #22664 Closes #22616 This contribution was developed with AI assistance (Claude Code). ### Contributor License Agreement - [x] By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-04-25 14:21:11 -05:00

GiteaMirror closed this issue

2026-04-25 14:21:14 -05:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/open-webui#42453