Skip to content

Commit 654ce07

Browse files
yahondaclaude
andcommitted
Rename internal variable to provided_keys
More directly describes what the local holds — the keys the caller actually provided — than the previous candidates/competitors names. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 79a643f commit 654ce07

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

lib/plsql/connection.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,13 @@ class Connection
2020
# branch on :database vs :sid. Raises ArgumentError on conflicts or
2121
# invalid values.
2222
def self.resolve_database_aliases!(params)
23-
candidates = []
24-
candidates << ":database" if params[:database]
25-
candidates << ":service_name" if params[:service_name]
26-
candidates << ":sid" if params[:sid]
27-
if candidates.size > 1
23+
provided_keys = []
24+
provided_keys << ":database" if params[:database]
25+
provided_keys << ":service_name" if params[:service_name]
26+
provided_keys << ":sid" if params[:sid]
27+
if provided_keys.size > 1
2828
raise ArgumentError,
29-
"Cannot specify more than one of #{candidates.join(', ')}; they are mutually exclusive."
29+
"Cannot specify more than one of #{provided_keys.join(', ')}; they are mutually exclusive."
3030
end
3131

3232
if (svc = params[:service_name])

0 commit comments

Comments
 (0)