Skip to content

Commit f7578d6

Browse files
wphillipmoorewphillipmoore-claude
andcommitted
fix: require name parameter for DELETE Q* methods
Re-generate commands from updated mapping-data.json which adds name_required: true to DELETE QLOCAL, QREMOTE, QALIAS, and QMODEL. These methods now require a name parameter since you cannot delete a queue without specifying which one. Co-Authored-By: wphillipmoore-claude <255925739+wphillipmoore-claude@users.noreply.github.com>
1 parent 3978cd6 commit f7578d6

2 files changed

Lines changed: 12 additions & 8 deletions

File tree

src/pymqrest/commands.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1688,7 +1688,7 @@ def delete_psid(
16881688

16891689
def delete_qalias(
16901690
self,
1691-
name: str | None = None,
1691+
name: str,
16921692
request_parameters: Mapping[str, object] | None = None,
16931693
response_parameters: Sequence[str] | None = None,
16941694
) -> None:
@@ -1718,7 +1718,7 @@ def delete_qalias(
17181718

17191719
def delete_qlocal(
17201720
self,
1721-
name: str | None = None,
1721+
name: str,
17221722
request_parameters: Mapping[str, object] | None = None,
17231723
response_parameters: Sequence[str] | None = None,
17241724
) -> None:
@@ -1748,7 +1748,7 @@ def delete_qlocal(
17481748

17491749
def delete_qmodel(
17501750
self,
1751-
name: str | None = None,
1751+
name: str,
17521752
request_parameters: Mapping[str, object] | None = None,
17531753
response_parameters: Sequence[str] | None = None,
17541754
) -> None:
@@ -1778,7 +1778,7 @@ def delete_qmodel(
17781778

17791779
def delete_qremote(
17801780
self,
1781-
name: str | None = None,
1781+
name: str,
17821782
request_parameters: Mapping[str, object] | None = None,
17831783
response_parameters: Sequence[str] | None = None,
17841784
) -> None:

src/pymqrest/mapping-data.json

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -172,16 +172,20 @@
172172
"qualifier": "psid"
173173
},
174174
"DELETE QALIAS": {
175-
"qualifier": "queue"
175+
"qualifier": "queue",
176+
"name_required": true
176177
},
177178
"DELETE QLOCAL": {
178-
"qualifier": "queue"
179+
"qualifier": "queue",
180+
"name_required": true
179181
},
180182
"DELETE QMODEL": {
181-
"qualifier": "queue"
183+
"qualifier": "queue",
184+
"name_required": true
182185
},
183186
"DELETE QREMOTE": {
184-
"qualifier": "queue"
187+
"qualifier": "queue",
188+
"name_required": true
185189
},
186190
"DELETE QUEUE": {
187191
"qualifier": "queue",

0 commit comments

Comments
 (0)