Skip to content

Commit 0787a77

Browse files
authored
Added checks for empty table names. (#54)
1 parent e96e5f7 commit 0787a77

File tree

7 files changed

+7
-7
lines changed

7 files changed

+7
-7
lines changed

driver/src/main/java/oracle/nosql/driver/ops/GetIndexesRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ public GetIndexesRequest setNamespace(String namespace) {
128128
*/
129129
@Override
130130
public void validate() {
131-
if (tableName == null) {
131+
if (tableName == null || tableName.isEmpty()) {
132132
throw new IllegalArgumentException(
133133
"GetIndexesRequest requires a table name");
134134
}

driver/src/main/java/oracle/nosql/driver/ops/GetTableRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ public GetTableRequest setNamespace(String namespace) {
131131
*/
132132
@Override
133133
public void validate() {
134-
if (tableName == null) {
134+
if (tableName == null || tableName.isEmpty()) {
135135
throw new IllegalArgumentException(
136136
"GetTableRequest requires a table name");
137137
}

driver/src/main/java/oracle/nosql/driver/ops/MultiDeleteRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ public String getTypeName() {
258258
*/
259259
@Override
260260
public void validate() {
261-
if (tableName == null) {
261+
if (tableName == null || tableName.isEmpty()) {
262262
throw new IllegalArgumentException(
263263
("MultiDeleteRequest requires table name"));
264264
}

driver/src/main/java/oracle/nosql/driver/ops/ReadRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public Request setDefaults(NoSQLHandleConfig config) {
5353
}
5454

5555
protected void validateReadRequest(String requestName) {
56-
if (tableName == null) {
56+
if (tableName == null || tableName.isEmpty()) {
5757
throw new IllegalArgumentException(
5858
(requestName +
5959
" requires table name"));

driver/src/main/java/oracle/nosql/driver/ops/TableRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@ public TableRequest setDefaults(NoSQLHandleConfig config) {
286286

287287
@Override
288288
public void validate() {
289-
if (statement == null && tableName == null) {
289+
if (statement == null && (tableName == null || tableName.isEmpty())) {
290290
throw new IllegalArgumentException(
291291
"TableRequest requires statement or TableLimits and name");
292292
}

driver/src/main/java/oracle/nosql/driver/ops/TableUsageRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,7 @@ public TableUsageRequest setTimeout(int timeoutMs) {
257257
*/
258258
@Override
259259
public void validate() {
260-
if (tableName == null) {
260+
if (tableName == null || tableName.isEmpty()) {
261261
throw new IllegalArgumentException(
262262
"TableUsageRequest requires a table name");
263263
}

driver/src/main/java/oracle/nosql/driver/ops/WriteRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public Request setDefaults(NoSQLHandleConfig config) {
4444
}
4545

4646
protected void validateWriteRequest(String requestName) {
47-
if (tableName == null) {
47+
if (tableName == null || tableName.isEmpty()) {
4848
throw new IllegalArgumentException(
4949
(requestName +
5050
" requires table name"));

0 commit comments

Comments
 (0)